모임 댓글 기능 구현 #169

ay-eonii · 2024-08-01T01:33:30Z

PR의 목적이 무엇인가요?

모임 댓글 기능을 구현합니다.

이슈 ID는 무엇인가요?

closed 댓글 기능 구현 #151

설명

상세 조회 API response에 comments 추가

{
    "data": {
        "title": "string",
        ...
        "comments": [
            {
                "commentId": 3,
                "nickname": "string",
                "content": "string",
                "dateTime": "2024-07-31 19:24",
                "childs": [
                    {
                        "commentId": 13,
                        "nickname": "string",
                        "content": "자식",
                        "dateTime": "2024-07-31 00:00"
                    },
                    {
                        "commentId": 14,
                        "nickname": "string",
                        "content": "자식",
                        "dateTime": "2024-07-31 00:00"
                    }
                ]
            },
            {
                "commentId": 4,
                "nickname": "string",
                "content": "string",
                "dateTime": "2024-07-31 19:24",
                "childs": []
            }
        ]
    }
}

댓글 작성 기능

질문 혹은 공유 사항 (Optional)

Mingyum-Kim

수고했어요 아연씨 ~
리뷰 가볍게 체크 부탁드려요 😋

Mingyum-Kim · 2024-08-01T01:43:45Z

backend/src/main/java/mouda/backend/comment/domain/Comment.java

-	@JoinColumn(name = "moim_id")
 	private Moim moim;

 	@ManyToOne
-	@JoinColumn(name = "member_id")
 	private Member member;


이거는 @Column(nullable = false) 를 안해줘도 되나요?

필요한 것 같아요@ManyToOne가 있어 @JoinColum(nullable = false)로 하겠습니다~!

Mingyum-Kim · 2024-08-01T01:44:25Z

backend/src/main/java/mouda/backend/comment/domain/Comment.java

+	}
+
+	private void validateContent(String content) {
+		if (content == null || content.isBlank()) {


content.isBlank() 에 null check가 포함되어있는 걸로 아는데요 😁

정보 공유용~

헤헷콩 감사합니다

Mingyum-Kim · 2024-08-01T01:45:03Z

backend/src/main/java/mouda/backend/comment/domain/Comment.java

+	private void validateMoim(Moim moim) {
+		if (moim == null) {
+			throw new MoimException(HttpStatus.NOT_FOUND, MoimErrorMessage.NOT_FOUND);
+		}
+	}


댓글 저장 시 잘못된 모임을 입력해서 에러가 발생했는데, MoimException을 발생시키는 게 적절할까요? 고민해봅시다!

Mingyum-Kim · 2024-08-01T01:48:20Z

backend/src/main/java/mouda/backend/comment/dto/response/ChildCommentResponse.java

+	@JsonFormat(pattern = "yyyy-MM-dd HH:mm")
+	LocalDateTime dateTime


데이터베이스에서는 date time으로 따로 저장되어있는데
여기는 한번에 보내네요! 이유가 있을까요?

LocalDateTime으로 보내면 상돌이 설정한 JacksonConfig가 적용되지 않는군요 .. 한번에 JacksonConfig에서 포맷팅할 수 없을까요?

Mingyum-Kim · 2024-08-01T01:49:52Z

backend/src/main/java/mouda/backend/comment/domain/Comment.java

+
+	private void validateMember(Member member) {
+		if (member == null) {
+			throw new CommentException(HttpStatus.NOT_FOUND, CommentErrorMessage.MEMBER_NOT_FOUND);


여기에서는 회원이 없을 때 CommentException이 발생하는데, 위와 컨벤션이 맞지 않는 것 같네요.

Mingyum-Kim · 2024-08-01T01:51:24Z

backend/src/main/java/mouda/backend/moim/service/MoimService.java

+		List<Comment> comments = commentRepository.findAllByMoimIdOrderByCreatedAt(id);
+		Map<Long, List<Comment>> childComments = comments.stream()
+			.filter(Comment::isChild)
+			.collect(groupingBy(Comment::getParentId));
+
+		List<CommentResponse> commentResponses = comments.stream()
+			.filter(Comment::isParent)
+			.map(comment -> CommentResponse.toResponse(comment, findChildComments(comment, childComments)))
+			.toList();


이 부분이 무슨 역할을 담당하는 지 이해하기 힘든 것 같아요. 메서드 분리해서 가독성을 높여주면 좋겠어용

Mingyum-Kim · 2024-08-01T01:52:47Z

backend/src/test/java/mouda/backend/comment/domain/CommentTest.java

나중에 테스트 픽스처를 사용하는 것으로 수정해주면 좋겠어요!
다른 도메인들도 픽스처를 사용하기로 해서요.

Mingyum-Kim · 2024-08-01T01:53:09Z

backend/src/test/java/mouda/backend/fixture/CommentFixture.java

오잉 픽스처가 있네요 .. 😀
왜 하나만 생성해두었는 지 궁금하네요 👍

null 같은 값이 부분적으로 들어가야해서 픽스쳐 사용하지 않았어용 픽스쳐는 자식 댓글 하나 추가할게용

ksk0605

간단한 코멘트니까 넘어가도 돼요!

ksk0605 · 2024-08-01T01:40:49Z

backend/src/main/java/mouda/backend/comment/domain/Comment.java

+	@Column(nullable = false)
 	private String content;

 	@ManyToOne
-	@JoinColumn(name = "moim_id")
 	private Moim moim;

 	@ManyToOne
-	@JoinColumn(name = "member_id")
 	private Member member;

+	@Column(nullable = false)
+	private LocalDateTime createdAt;
+
 	private Long parentId;


널이 가능한 컬럼과 아닌 컬럼이 나뉘어 있는데 제가 볼 때는 모든 정보다 다 null 이면 안되는 값 같아보여서요!

ksk0605 · 2024-08-01T01:44:04Z

backend/src/main/java/mouda/backend/comment/dto/response/CommentResponse.java

+	@JsonFormat(pattern = "yyyy-MM-dd HH:mm")
+	LocalDateTime dateTime,
+
+	List<ChildCommentResponse> childs


ㅋㅋ? child의 복수형은 children 아닌가요?

ksk0605 · 2024-08-01T01:49:47Z

backend/src/main/java/mouda/backend/moim/service/MoimService.java

+
 	public Moim createMoim(MoimCreateRequest moimCreateRequest) {
 		Member author = new Member(moimCreateRequest.authorNickname());
 		Moim moim = moimRepository.save(moimCreateRequest.toEntity());


builder 로 생성하지 않고 생성자를 선택하신 이유가 있나요?

hoyeonyy · 2024-08-01T01:53:21Z

backend/src/main/java/mouda/backend/comment/dto/request/CommentCreateRequest.java

+
+public record CommentCreateRequest(
+	Long parentId,
+


파라미터 요거 개행하나요?

hoyeonyy · 2024-08-01T01:58:13Z

backend/src/main/java/mouda/backend/moim/controller/MoimController.java

+
+	@Override
+	@PostMapping("/{moimId}")
+	public ResponseEntity<Void> createComment(@LoginMember Member member, @PathVariable Long moimId,


파라미터 3개 이상이면 줄바꿈 해야할 것 같습니다! 컨벤션에 적혀있더라구요!

감삼당 👍

hoyeonyy

고생많았습니다 테니! 코드량이 쫌 많긴 하네요!

사소한 컨벤션 몇개 남겨놨습니다!

Mingyum-Kim

피드백 잘 반영해주셔서 감사합니다><
어푸르브 드려요 ><

ay-eonii added 13 commits July 31, 2024 17:01

feat: 댓글 작성 날짜 필드 추가

7236595

feat: 댓글 응답 DTO 추가

95536c5

feat: 댓글 조회 기능 구현

62ce712

Merge branch 'develop-backend' into feature/#151

0a49daf

feat: 댓글 작성 일시 포맷

8a607c7

feat: 댓글 작성 일자 오름차순 정렬

0ab7205

feat: 댓글 작성자 닉네임 반환 메서드

2e1acf0

feat: 댓글 작성 기능

629bc62

refactor: 불필요한 어노테이션 삭제

4780285

refactor: 댓글 작성자 반환 메서드 사용

f26293b

feat: 제약조건 추가

f33ee8f

feat: 부모댓글 여부 예외처리

4fab80c

test: 댓글 테스트

fac5b8d

ay-eonii added BE 백엔드 관련 이슈입니다. 🌱 기능추가 feature (새로운 기능 구현) labels Aug 1, 2024

ay-eonii requested review from Mingyum-Kim, ksk0605, hoyeonyy and pricelees August 1, 2024 01:33

ay-eonii self-assigned this Aug 1, 2024

ay-eonii linked an issue Aug 1, 2024 that may be closed by this pull request

댓글 기능 구현 #151

Closed

5 tasks

Mingyum-Kim requested changes Aug 1, 2024

View reviewed changes

ksk0605 approved these changes Aug 1, 2024

View reviewed changes

hoyeonyy approved these changes Aug 1, 2024

View reviewed changes

ay-eonii added 5 commits August 1, 2024 11:17

feat: CommentException 모임 부재 예외 추가

37a0d74

feat: 모임, 멤버 nullable 불가능

9482f48

test: 자식 댓글 픽스쳐 추가

c70504b

style: 파라미터 개행 추가

ca96914

refactor: 모임 생성 빌더 사용

9796336

ay-eonii added 3 commits August 1, 2024 13:37

refactor: CommentResponse에서 childCommentResponse 생성

cf8e868

Merge branch 'develop-backend' into feature/#151

41c95b7

test: 예외 클래스 변경으로 인한 테스트 수정

11e4ad9

Mingyum-Kim approved these changes Aug 1, 2024

View reviewed changes

ay-eonii merged commit d4ce086 into develop-backend Aug 1, 2024
1 check passed

ay-eonii deleted the feature/#151 branch August 1, 2024 05:01

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

모임 댓글 기능 구현 #169

모임 댓글 기능 구현 #169

ay-eonii commented Aug 1, 2024

Mingyum-Kim left a comment

Mingyum-Kim Aug 1, 2024

ay-eonii Aug 1, 2024

Mingyum-Kim Aug 1, 2024

ay-eonii Aug 1, 2024

Mingyum-Kim Aug 1, 2024

Mingyum-Kim Aug 1, 2024

Mingyum-Kim Aug 1, 2024

Mingyum-Kim Aug 1, 2024

Mingyum-Kim Aug 1, 2024

Mingyum-Kim Aug 1, 2024

Mingyum-Kim Aug 1, 2024

ay-eonii Aug 1, 2024 •

edited

Loading

ksk0605 left a comment

ksk0605 Aug 1, 2024

ksk0605 Aug 1, 2024

ksk0605 Aug 1, 2024

ay-eonii Aug 1, 2024

hoyeonyy Aug 1, 2024

hoyeonyy Aug 1, 2024

ay-eonii Aug 1, 2024

hoyeonyy left a comment

Mingyum-Kim left a comment

		@JsonFormat(pattern = "yyyy-MM-dd HH:mm")
		LocalDateTime dateTime

모임 댓글 기능 구현 #169

모임 댓글 기능 구현 #169

Conversation

ay-eonii commented Aug 1, 2024

PR의 목적이 무엇인가요?

이슈 ID는 무엇인가요?

설명

질문 혹은 공유 사항 (Optional)

Mingyum-Kim left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

ay-eonii Aug 1, 2024 • edited Loading

Choose a reason for hiding this comment

ksk0605 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hoyeonyy left a comment

Choose a reason for hiding this comment

Mingyum-Kim left a comment

Choose a reason for hiding this comment

ay-eonii Aug 1, 2024 •

edited

Loading