프로젝트 피드백 - 작성자: 김태선

[thesun4sky]

Overview

• 이전 프로젝트에 부가기능을 잘 추가려고 노력해주신 부분들이 보여서 칭찬드립니다.
• 특히 Advice 를 통한 예외 공통화 부분이라던가 커스텀 익셉션 부분은 잘 구현해주셨습니다.
• 다만, API 응답이나 서비스 인터페이스 분리, 부분적으로 완성도를 높이기 위한 부분이 필요합니다.
• API 응답은 ResponseEntity 를 써서 응답코드를 함께 리턴하는것이 중요합니다.
• 서비스 인터페이스는 주석이 중요합니다. 서비스 코드 외에도 전체적으로 주석도 잘 추가해주시면 좋을 것 같습니다.
• 마지막으로 Post 엔티티에서 User 의 username을 포함하고 있는데 해당 필드는 User 와 별개로 관리되기 때문에 동기화가 깨질 수 있습니다.

코드 레벨에 대한 피드백은 아래에 남기도록 하겠습니다.

[thesun4sky]

GlobalExceptionHandler.java

https://github.com/skah1061/finalPractice/blob/040e0226f71c7df61bda5877c2fee47d985ea058/src/main/java/com/example/plusweek/exception/GlobalExceptionHandler.java#L8-L20

• 위 코드처럼 글로벌 예외처리를 잘 해주셔서 좋았습니다.
• 특히 각각의 예외에 대해서 주석도 남겨주시고 응답코드를 정리하신것 칭찬드립니다.

[thesun4sky]

AccountOfTheWrongRules.java

https://github.com/skah1061/finalPractice/blob/040e0226f71c7df61bda5877c2fee47d985ea058/src/main/java/com/example/plusweek/exception/AccountOfTheWrongRules.java#L3-L7

• 다만 위와 같이 커스텀 예외의 경우 클래스명뒤에 Exception 이 들어감으로써 해당 클래스가 예외 클래스임을 명시해주셔야 합니다.
• 그리고 필요하다면 커스텀 예외안에 필드를 추가해서 추가정보를 담을 수 있습니다.

[thesun4sky]

CommentController.java

https://github.com/skah1061/finalPractice/blob/040e0226f71c7df61bda5877c2fee47d985ea058/src/main/java/com/example/plusweek/contoller/CommentController.java#L27-L31

• 위와 같이 댓글 삭제 컨트롤러는 잘 작성해주셨는데 ResponseEntity 를 써서 응답코드를 함께 리턴하는것이 중요합니다.
• 그렇게 해야 클라이언트에서 사전에 정의된 응답코드별 처리를 할 수 있습니다.

[thesun4sky]

Post.java

https://github.com/skah1061/finalPractice/blob/040e0226f71c7df61bda5877c2fee47d985ea058/src/main/java/com/example/plusweek/entiry/Post.java#L23-L24

• Post 엔티티에서 User 의 username을 포함하고 있는데 해당 필드는 User 와 별개로 관리되기 때문에 동기화가 깨질 수 있습니다.
• User 의 id 를 포함하도록 해서 연관관계를 만들어줘야 username 이 수정되어도 Post 조회시점에 동기화가 될 수 있습니다.