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

[thesun4sky]

Overview

• 프로젝트 전체적인 완성도가 높아서 매우 놀랬습니다. 잘하셨습니다.
• AOP, 글로벌 예외처리, 서비스 인터페이스, 거기다가 Swagger 와 S3 까지 추가구현 해주고 계신부분들 칭찬드립니다.
• 도메인 단위로 패키지 구분하신것 좋은 시도 입니다.
• QueryDSL 도 잘 활용해서 구현해주신것 확인 하였으며 피드백 남기겠습니다.
• post, postImage, comment 간에 연관관계 설정도 적절히 잘 해주셨습니다.

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

[thesun4sky]

QComment.java

https://github.com/wisdomSG/myBlog/blob/5c8376526e94732ca4855fa33b0d2202280ea02c/src/main/generated/com/example/myblog/entity/QComment.java#L13-L18

• Q Class 는 자동생성되는 클래스로 코드 수정시 빌드할때마다 변경되는 코드입니다.
• 그렇기 때문에 /generated 하위 클래스는 깃에 커밋하지 않습니다.

[thesun4sky]

RoleCheckAop.java

https://github.com/wisdomSG/myBlog/blob/5c8376526e94732ca4855fa33b0d2202280ea02c/src/main/java/com/example/myblog/aop/RoleCheckAop.java#L53-L54

• AOP 를 통해 권한체크 공통화 잘해주셨습니다.
• 다만, 예외 던질시 예외 메세지를 넣어서 어떤 예외인지 명확히 전달하는것이 더 좋습니다.

[thesun4sky]

CommentController.java

https://github.com/wisdomSG/myBlog/blob/5c8376526e94732ca4855fa33b0d2202280ea02c/src/main/java/com/example/myblog/comment/CommentController.java#L29-L33

• AOP 에서 예외를 명확히 해주고, 글로벌 예외처리에서 잘 구분해서 처리해주면 위 코드를 대체할 수 있습니다.
• 잘 고민해서 수정해주세요.

[thesun4sky]

PostImage.java

https://github.com/wisdomSG/myBlog/blob/5c8376526e94732ca4855fa33b0d2202280ea02c/src/main/java/com/example/myblog/entity/PostImage.java#L19-L22

• 연관관계 설정은 전체적으로 잘 해주셨습니다.
• 다만, fetchType 을 LAZY 로 해주었다고 하더라도 log 출력과 같은곳에서 toString() 메서드를 호출하게되면 쿼리를 날리게 됨으로 주의해주세요.
• 해결방법 중 하나로 @Exclude 을 붙여서 문자열 출력시 제외시키는 방법이 있습니다. 아래 참조링크를 참고해주세요. https://jaime-note.tistory.com/384

[thesun4sky]

PostController.java

https://github.com/wisdomSG/myBlog/blob/5c8376526e94732ca4855fa33b0d2202280ea02c/src/main/java/com/example/myblog/post/PostController.java#L38-L57

• 전체 조회, 페이징 조회, 검색 조회를 모두 구분하여 개발해주셨는데요.
• 위 API 3개는 하나로 합쳐질 수 있습니다. 각각의 다른 조건 필드를 사용하면 되는거라서요.
• SearchCond 객체로 요청정보에 대한 내용을 모두 포함하게 하고 QueryDSL 로 개별적으로 처리할 수 있습니다.
• 아래 링크를 참조해주세요. https://m.blog.naver.com/qjawnswkd/222269026595