#117 [fix] 모델 회원가입 로직 변경 #121
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
KWY0218
reviewed
Jan 14, 2024
Comment on lines
247
to
259
| User user = userRepository.findById(userId).orElseThrow(() -> new NotFoundException(ErrorCode.USER_NOT_FOUND_EXCEPTION)); | ||
|
|
||
| Model model = Model.builder() | ||
| .kakaoId(user.getKakaoId()) | ||
| .name(request.name()) | ||
| .year(request.year()) | ||
| .gender(request.gender()) | ||
| .phoneNumber(request.phoneNumber()) | ||
| .isMarketingAgree(request.isMarketingAgree()) | ||
| .profileImgUrl(s3Service.getDefaultProfileImageUrl()) | ||
| .kakaoId(kakaoId) | ||
| .role(Role.MODEL) | ||
| .build(); | ||
| .year(request.year()).build(); | ||
|
|
||
| modelJpaRepository.save(model); |
There was a problem hiding this comment.
p2
현재 로직은 새로운 model을 하나 더 만드는 로직입니다.
find by id를 통해 기존 user를 불러온 뒤, 해당 setter를 통해 해당 user의 정보를 update 한 뒤,
Model 객체를 builder를 통해 만들어서 save 하도록 로직을 수정해야 할 것 같습니다.
There was a problem hiding this comment.
Builder를 통해 Model 객체를 만들어서 저장을 하게 되면 기존에 만들어진 userId에 대한 정보를 이어받아서 Model만 생성되는 것이 아니라 새로운 User 정보도 생성된다는 문제가 있습니다! 기존의 User가 1번 이였다면, 2번 유저를 가르키고 있는 Model이 만들어 지는데 혹시 이러한 방법을 하는게 맞을까요..?
-- 제 코드가 문제가 있을 수도 있어서 방금 수정 코드 다시 올려두었습니다!
User 2 번이 생성안되고 Model 객체가 User 1을 참조할 수 있는 방법 좀 더 생각해서 PR 올릴께욤!
KWY0218
reviewed
Jan 15, 2024
| @ApiResponses(value = { | ||
| @ApiResponse(responseCode = "200", description = "모델 회원가입 성공"), | ||
| @ApiResponse(responseCode = "400", description = "유효하지 않은 카카오 코드를 입력했습니다.", content = @Content(schema = @Schema(implementation = ErrorResponse.class))), | ||
| @ApiResponse(responseCode = "400", description = "인증오류 입니다.", content = @Content(schema = @Schema(implementation = ErrorResponse.class))), |
There was a problem hiding this comment.
p2
access token 관련 오류이기 때문에 400이 아닌 401 에러인 것 같습니다..!
| @@ -74,21 +79,19 @@ public class DesignerService { | |||
|
|
|||
|
|
|||
| @Transactional | |||
| public DesignerMainResponse getDesignerMainInfo(Long userId, int page, int size){ | |||
| public DesignerMainResponse getDesignerMainInfo(final long userId, final int page, final int size){ | |||
There was a problem hiding this comment.
p2
아래 User user = new User() 는 사용하지 않는 듯 합니다..!
사용하지 않는 구문은 제거해주세요
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
관련 이슈번호
해결하는 데 얼마나 걸렸나요? (예상 작업 시간 / 실제 작업 시간)
해결하려는 문제가 무엇인가요?
기존의 문제 상황
원하는 상황
현재상황, 문제 상황
기존의 문제 상황을 해결함 어떻게?
해결 방법
JPQL 쿼리를 사용해서 Model 객체만 insert 하는 방법으로 구현 완료~!