Conversation
There was a problem hiding this comment.
제일 아래(38번 라인) 줄 : } 아래에 1칸 더 공백을 만들어 주시는 것을 권장드립니다. 저도 얼마 전 동구의 피드백으로 알게 되었는데 해당 빨간 - 표시는 "EOL(개행) 경고"로 파일의 끝(종료)에 개행이 없을 때 발생합니다.
만약 파일의 끝에 개행이 없다면 예기치 못한 문제가 발생할 수 도 있고 또한 파일을 구분할 때 도움이 되기도 합니다(아마 C언어에서 File I/O 부분 쪽에서 배웠을 겁니당). 그래서 인텔리제이에서 파일을 생성하실 때 자동으로 제일 아래에 개행이 추가되는 것을 보실 수 있으실 텐데 그것은 이 같은 문제를 미연에 방지하고자입니다.
There was a problem hiding this comment.
새롭게 알게된 사실이네요! 수정하겠습니다
There was a problem hiding this comment.
이번 변경사항에 관한 것은 아닌데 혹시 빌더 메서드랑 객체 생성 메서드 내부에 Invitation도 포함되어야 하지 않나요? Comments 엔티티가 Invitation 엔티티를 참조하니
There was a problem hiding this comment.
깜빡하고 넣지 않은 것 같습니다ㅠㅠ 수정하겠습니다!
| @PathVariable Long invitationId, | ||
| @RequestParam(defaultValue = "0") int page){ | ||
|
|
||
| Page<CommentResponseDTO> comments = commentService.findAllCommentsByInvitationId(invitationId, page); |
There was a problem hiding this comment.
Page 객체로 받아오면 필요없는 정보들도 모두 가져와져서 isLast, pageNumber 등 page 객체에서 추출할 수 있는 부분을 추출한 List 형태로 변경하는게 좋을 것 같습니다! 그리고 page 사이즈는 1부터 시작하도록 하는 것이 사용하는 데 있어 정확할 것 같아요!
There was a problem hiding this comment.
해당 사항 반영해보았습니다! 확인 부탁드립니다
src/main/java/com/wedit/weditapp/domain/comments/controller/CommentController.java
Show resolved
Hide resolved
src/main/java/com/wedit/weditapp/domain/comments/service/CommentService.java
Show resolved
Hide resolved
src/main/java/com/wedit/weditapp/domain/comments/service/CommentService.java
Show resolved
Hide resolved
개요
특정 청첩장의 방명록 조회 API 구현했습니다.
PR 유형
어떤 변경 사항이 있나요?
PR Checklist
PR이 다음 요구 사항을 충족하는지 확인하세요.
📣 To Reviewers
CommentService의 예외처리 코드는 아직 invitationRepository가 구현되기 전이라 주석처리 해뒀습니다.