Skip to content

[우윤수] 7주차 과제 제출#31

Merged
jettieb merged 3 commits into
EFUB:member/wys0530from
wys0530:week7
May 20, 2026
Merged

[우윤수] 7주차 과제 제출#31
jettieb merged 3 commits into
EFUB:member/wys0530from
wys0530:week7

Conversation

@wys0530
Copy link
Copy Markdown

@wys0530 wys0530 commented May 15, 2026

📌 구현 기능

  • 댓글 삭제
  • 게시글 좋아요 생성
  • 게시글 좋아요 삭제

🗂️ 과제 정리

댓글 삭제

image

게시글 좋아요 생성

image

게시글 좋아요 삭제

image

⚠️ 어려웠던 점

  • 지난번에 만들어둔 제 명세서에서는 좋아요 생성에서 멤버id, post id, 생성일을 반환하도록 만들어두긴 해서 일단 그대로 구현했는데, 일반적으로 좋아요 생성과 같은 기능에서는 지난 세미나에서 만들었던 것처럼 noContent로 반환하면 되나요?? 딱히.. 어떻게 하던 상관 없는걸까요 ㅎㅎ..

@wys0530 wys0530 self-assigned this May 15, 2026
@KKANGCHONG
Copy link
Copy Markdown

KKANGCHONG commented May 17, 2026

image

CommentService에서 댓글 작성자 여부를 확인할 때 엔티티 객체 자체를 비교하고 있는데, Member 엔티티에서 equals/hashCode가 재정의되어 있지 않다면 의도와 다르게 비교될 가능성이 있습니다. PostService와 동일하게comment.getWriter().getMemberId().equals(member.getMemberId())처럼 식별자 기준으로 비교하면 더 안전하고 일관성 있을 것 같습니다~!

@KKANGCHONG
Copy link
Copy Markdown

그리고 질문 주신 내용은 둘 다 맞지만 목적이 다르다고 생각하면 좋을거 같습니다~
리소스 생성 관점에서는 지금처럼 구현하신 방식이 REST 원칙에 더 가깝지만 좋아요처럼 단순 상태 변경 기능은 204 No Content로 간단하게 처리하기도 합니다. 이렇게 하면 코드가 간결해지고 네트워크 비용이 미세하지만 줄어든다는 장점이 있습니다. 당연히 구현하신대로 하면 REST 원칙에 가까운거니 이것 또한 좋은 방식이구요!
다만 좋아요처럼 단순히 상태 변경에 가까운 기능이라면 200 이나 204로 처리하는 경우가 많은걸로 알고 있습니다! 그래서 개인적으론 상관은 없지만 간결한 코드를 추천 드립니다..!

@KKANGCHONG
Copy link
Copy Markdown

KKANGCHONG commented May 17, 2026

수고 많으셨습니다 리뷰할게 없을 정도로 너무 잘 하신거 같아요! 고생하셨어요~~😁👍🏻

@KKANGCHONG KKANGCHONG self-requested a review May 17, 2026 14:48
@jettieb jettieb merged commit cf8a0c2 into EFUB:member/wys0530 May 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants