refactor : 게시글 이모지와 댓글 좋아요 누르기/취소하기 API 응답 DTO 반환하도록 수정 #635
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
limehee
approved these changes
Dec 20, 2024
| @Component | ||
| public class CommentLikeDtoMapper { | ||
|
|
||
| public CommentLikeToggleResponseDto toDto(Long boardId, Long commentLikes, Boolean isDeleted) { |
There was a problem hiding this comment.
도메인 객체를 DTO로 바꾸는 변환이 아니라 각 파라미터를 대입하는 것이기 때문에 메서드 이름을 of로 하는건 어떠신가요?
There was a problem hiding this comment.
명명규칙이 도메인 객체를 DTO로 바꾼다면 toDto이고 다른 파라미터들로 DTO를 생성하면 of로 되는걸까요?
There was a problem hiding this comment.
네네 DTO 생성의 개념에서 of를 사용해주시면 될 것 같아요
There was a problem hiding this comment.
해당 부분에 대한 컨벤션은 아래 링크의 5. 생성자명 변경 코멘트 참고해주시면 도움되실 것 같아요.
There was a problem hiding this comment.
넵 확인하도록 하겠습니다!
SongJaeHoonn
approved these changes
Dec 20, 2024
2 tasks
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.
Summary
기존에는 아래와 같이 응답이 되었습니다.
게시글 이모지 누르기/취소하기 API : board category 반환
댓글 좋아요 누르기/취소하기 API : 댓글의 총 좋아요 개수 반환
이를 아래와 같이 응답하도록 수정했습니다.
게시글 이모지 누르기/취소하기 API :
댓글 좋아요 누르기/취소하기 API :
Tasks
ETC
해당 작업에서 CommentLike 테이블의 경우 소프트딜리트가 적용되지 않음을 확인했습니다.
댓글의 좋아요와 게시글의 이모지는 비슷한 로직임에도 게시글의 이모지만 소프트딜리트가 적용되어 있었습니다.
따라서 누르기/취소하기 토글 행위에 대해 어떤 딜리트 방식이 효율적일지 성능 테스트를 게시글 이모지, 댓글 좋아요 토글의 삭제 방식에 따른 성능 개선 #634 에서 진행하고 개선하도록 하겠습니다.
@limehee님께서 댓글에서도 좋아요 대신 이모지를 사용하는 것을 제안해주셨습니다. 해당 부분은 members 프로젝트에서 현재 기획한 신규 기능을 마무리하고 추가 회의를 통해 다시 다루도록 @Jeong-Ag 님과 이야기 나눴습니다. 댓글 이모지 기능으로 노선을 수정할 경우, 핫게시글 선정 방식의 수정과 프론트에서 게시글 옆에 [댓글개수]를 표현하는 방식에 대해서도 추가 수정이 필요한 점이 있었습니다. 이러한 이유로 신규 기능 출시를 먼저 하고, 추후 다시 다뤄보도록 하겠습니다. 좋은 의견 감사합니다.