[BE] 약속 생성 결과 조회 API 구현 #52
Conversation
ikjo39
requested review from
ehBeak,
hw0603,
seokmyungham and
seunghye218
as code owners
Test Results17 tests 17 ✅ 3s ⏱️ Results for commit 662653a. ♻️ This comment has been updated with latest results. |
| public MomoApiResponse<MeetingCompletedResponse> findCompleted(@PathVariable String uuid) { | ||
| MeetingCompletedResponse response = meetingService.findCompleted(uuid); |
There was a problem hiding this comment.
컨트롤러, 서비스 메서드 명을 Completed 라는 단어보다는 Created나 Generated는 어떤가요?
Complete는 뭔가 약속이 최종 완료된 느낌이 드는 것 같아요.
There was a problem hiding this comment.
약속이 생성된 후 보여주어야 할 메서드니 재즈 말대로 Created가 적절해보이네요!
반영해보도록 할게요!
| public Attendee create(Meeting meeting) { | ||
| return new Attendee(meeting, new AttendeeName(name), new AttendeePassword(password), role); | ||
| } |
There was a problem hiding this comment.
연관관계가 있는 fixture을 어떻게 만들어야 하나 궁금했는데, 이렇게 Fixture을 만들 수 있네요! 👍👍
| void findCreatedResult() { | ||
| Meeting meeting = meetingRepository.save(MeetingFixture.GAME.create()); | ||
|
|
||
| MeetingCompletedResponse result = meetingService.findCreatedResult(meeting.getUuid()); |
There was a problem hiding this comment.
result라는 변수명 보다는 createdMeeting과 같이 생성이 완료된 약속이란 정보를 넣으면 좋을 것 같아요!
| public static MeetingCompletedResponse from(Meeting meeting) { | ||
| return new MeetingCompletedResponse(meeting.getUuid()); | ||
| } |
There was a problem hiding this comment.
MeetingCompletedResponse 클래스명을 수정하면 어떨까요?
제가 느끼기에 약속을 생성한 다음, 생성했다고 응답할 때 쓰이는 클래스 같아요..!
저도 지금 이름이 잘 생각이 안 나서😅 제안만 해봅니다!
There was a problem hiding this comment.
MeetingSharingResponse으로 변경해보았습니다 :)
| @@ -6,7 +6,8 @@ public enum AttendeeErrorCode implements ErrorCodeType { | |||
|
|
|||
| INVALID_PASSWORD_LENGTH(HttpStatus.BAD_REQUEST, "비밀번호 길이는 최대 20글자까지 가능합니다."), | |||
| INVALID_UUID(HttpStatus.BAD_REQUEST, "유효하지 않은 UUID 입니다."), | |||
| PASSWORD_MISMATCHED(HttpStatus.BAD_REQUEST, "비밀번호가 일치하지 않습니다."); | |||
| PASSWORD_MISMATCHED(HttpStatus.BAD_REQUEST, "비밀번호가 일치하지 않습니다."), | |||
| INVALID_ACCESS(HttpStatus.BAD_REQUEST, "유효하지 않는 접근입니다. 관리자에게 문의해주세요."); | |||
There was a problem hiding this comment.
유효하지 '않은' 접근이 좀 더 자연스럽게 읽히는 것 같아요!
There was a problem hiding this comment.
ENUM 명을 신경쓰지 못했네요!
수정하려 했지만 다시 보니 해당 Enum 값은 사용하고 있지 않아 삭제하였습니다!
| import org.springframework.stereotype.Service; | ||
| import org.springframework.transaction.annotation.Transactional; | ||
|
|
||
| @Slf4j |
관련 이슈
작업 내용
특이 사항
리뷰 요구사항 (선택)