Skip to content

Conversation

@ssosee
Copy link
Member

@ssosee ssosee commented Jul 13, 2025

제목 : [(Jira issue 번호)] 작업명

ex) [DP-123] pull request template 작성
(확인 후 지워주세요)

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요 (이미지 첨부 가능)

  • 아래 2개의 기능의 결합도가 높아서 같이 작업 했습니다.
  • 픽픽픽 댓글/답글 조회 API 수정
  • 픽픽픽 베스트 댓글 조회 API 수정

🔗 참고할만한 자료(선택)

슬랙이나 피그마, Jira 등 참고 링크를 첨부해주세요

  • [레포 이름 #이슈번호](이슈 주소)

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

@ssosee ssosee requested a review from yu-so-young2 July 13, 2025 10:21
@ssosee ssosee self-assigned this Jul 13, 2025
@ssosee ssosee added documentation Improvements or additions to documentation fix 기능 수정 feat 기능 추가 labels Jul 13, 2025
ralph-teuida and others added 6 commits July 13, 2025 19:26
- 익명회원 댓글 삭제 서비스 개발 및 테스트 코드 작성
# Conflicts:
#	src/main/java/com/dreamypatisiel/devdevdev/domain/service/pick/GuestPickCommentServiceV2.java
#	src/test/java/com/dreamypatisiel/devdevdev/domain/service/pick/GuestPickCommentServiceV2Test.java
# Conflicts:
#	src/test/java/com/dreamypatisiel/devdevdev/domain/service/pick/GuestPickCommentServiceV2Test.java

// 댓글을 회원이 작성한 경우
// 부모 댓글을 익명회원이 작성한 경우
if (parentCreatedBy == null) {
Copy link
Member

@yu-so-young2 yu-so-young2 Jul 16, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

조건문이 조금 복잡(?)한 것 같은데 평탄화하면 어떨까요? 예를 들어 이런식으루...

boolean isRepliedByMember = repliedCreatedBy != null;
boolean isParentByMember = parentCreatedBy != null;

if (!isRepliedByMember && !isParentByMember) {
    return createResponseForAnonymousReplyToAnonymous(...);
}
if (!isRepliedByMember) {
    return createResponseForAnonymousReplyToMember(...);
}
if (!isParentByMember) {
    return createResponseForMemberReplyToAnonymous(...);
}
return createResponseForMemberReplyToMember(...);

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오 좋은 아이디어 감사합니다.
츄라츄라이 해보겠습니다! 👍

@yu-so-young2 yu-so-young2 merged commit 589c369 into develop Jul 20, 2025
1 check passed
@ssosee ssosee deleted the DP-543 branch July 20, 2025 04:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation feat 기능 추가 fix 기능 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants