프로젝트 피드백 - 작성자: 이성국 튜터

[tjdrnr0557]

Overview

• Preview ERD 등이 깔끔하게 정리되어서 좋습니다.
• PR을 이용해서 내역을 정리하고 브런치를 머지한 것등은 좋았습니다. 아쉬운 점은 PR에서 명확히 어떠한 작업을 했는지 나타냈으면 좋았을 것 같고 다른 팀원분들의 리뷰, 다른분들도 PR을 통해 머지하는 습관을 기르시면 더욱 좋을 것 같습니다
• ERD와 엔티티 내역이 좀 다르네요. ERD 만 보면 follow 테이블에 팔로잉하는 사람, 팔로워 하는 사람이 있어야 할 것 같은데 빠져있네요
• 전체적으로 restful api 설계에 대해 이해가 많이 생긴 것 같아서 좋습니다. 아쉬운 점은 comment_id -> commentId 로 카멜케이스로 적어주시면 좋겠네요

• Code review

  https://github.com/hojooglim/umm/blob/cc3eec901ad58777a06e97878dc36dc99c192228/src/main/java/com/example/umm/follow/entity/Follow.java#L17-L29

• user -> FollowerUser 로 쓰는게 더 이해하기 좋을 것 같네요

https://github.com/hojooglim/umm/blob/cc3eec901ad58777a06e97878dc36dc99c192228/src/main/java/com/example/umm/follow/repository/FollowRepository.java#L8-L12

• @Repository 어노테이션 추가해주세요

https://github.com/hojooglim/umm/blob/cc3eec901ad58777a06e97878dc36dc99c192228/src/main/java/com/example/umm/follow/service/FollowService.java#L23-L33

• 좋아요를 누른 것과 팔로우하는 것과 어떤 연관이 있는것인가요!? 비즈니스 로직에 대한 설명이 없어서 아쉽습니다 보통의 SNS는 좋아요와 팔로우가 분리되어있습니다
• ifPresentOrElse 메소드를 이용해 로직 구성 좋습니다
• 팔로우를 요청하는 함수인데 왜 삭제 로직이 들어가는지 모르겠네요
• 전반적으로 sns내 게시글을 umm으로 표시한 것 같은데 이는 도메인 이름에 맞지 않아서 Post 같은 것이 더 적절해보입니다.

https://github.com/hojooglim/umm/blob/cc3eec901ad58777a06e97878dc36dc99c192228/src/main/java/com/example/umm/follow/repository/FollowRepository.java#L9-L11

• repository 메소드 적절하게 잘 구성해주셨네요.

https://github.com/hojooglim/umm/blob/cc3eec901ad58777a06e97878dc36dc99c192228/src/main/java/com/example/umm/comment/controller/ReCommentController.java#L15-L29

• /reComments -> /re-comments 로 쓰는 것이 일반적입니다
• comment_id -> commentId

https://github.com/hojooglim/umm/blob/cc3eec901ad58777a06e97878dc36dc99c192228/src/main/java/com/example/umm/comment/entity/Comment.java#L47-L50

• 띄어쓰기 린트 맞춰주세요

https://github.com/hojooglim/umm/blob/cc3eec901ad58777a06e97878dc36dc99c192228/src/main/java/com/example/umm/comment/entity/Comment.java#L28-L39

• 연관관계 설정 좋습니다

https://github.com/hojooglim/umm/blob/cc3eec901ad58777a06e97878dc36dc99c192228/src/main/java/com/example/umm/comment/entity/Comment.java#L20-L23

• incremental pk 설정 좋네요

https://github.com/hojooglim/umm/blob/cc3eec901ad58777a06e97878dc36dc99c192228/src/main/java/com/example/umm/comment/entity/ReComment.java#L19-L21

• ReComment -> reComment
• 그리고 ReComment 테이블 내에 내용이라면 content 이정도로만 써도 코멘트 내용이라는것을 알 수 있겠네요

https://github.com/hojooglim/umm/blob/cc3eec901ad58777a06e97878dc36dc99c192228/src/main/java/com/example/umm/comment/service/ReCommentService.java#L19-L24

• npe 보다는 IllegalArgumentException 같은 것이 더 적절해보입니다. 아니면 의미가 맞는 다른 예외도 가능합니다
• 또한 트랜잭셔널 설정 필요합니다. 로직 구성은 좋네요

https://github.com/hojooglim/umm/blob/cc3eec901ad58777a06e97878dc36dc99c192228/src/main/java/com/example/umm/likeit/controller/LikeItController.java#L13-L28

• 게시글에 대한 좋아요라면 /umm/{ummId}/likes POST 와 같이 쓰는 것이 더 일반적입니다.
• /comments/{commentId}/likes

전반적으로 실력이 많이 성장한 것 같아서 보기 좋습니다. 로직 구성은 전반적으로 좋아보이고 리뷰 내용들 다음부터 유의해서 구성해보면 더 좋을 것 같습니다!

[hojooglim]

감사합니다