• 사용자는 선택한 댓글에 '좋아요'를 할 수 있습니다.

• 사용자가 이미 '좋아요'한 댓글에 다시 '좋아요' 요청을 하면 '좋아요'를 했던 기록이 취소됩니다.

• 요청이 성공하면 Client로 성공했다는 메시지, 상태코드 반환하기

• 댓글 수정, 댓글 조회, 댓글 등록 시에 댓글 반환하면서 좋아요 개수도 반환하기

• Post, User Entity 관계에서 Post가 User의 id를 FK로 참조하게끔 수정.
• PostController에서 받는 요청 중에 전체 게시글 목록 조회와 선택 게시글 목록 조회에 대해서 인증을 받지 않아도 되게끔 수정.
• 이에 따라 service, controller, entity 등 수정 필요.

테스트 해봐야 할 것

• Post 기록
  -> 회원 가능 & 관리자 가능 & 비회원 불가능
• Post 수정 삭제
  -> 작성자 가능 & 관리자 가능 & 비회원 불가 & 작성자, 관리자 외 회원 불가
• 선택한 게시글 조회 및 전체 게시글 목록 조회
  -> 회원, 비회원, 관리자 모두 가능(인증 받는지 안 받는지 테스트)

• Comment Entity, CommentController, CommentService, CommentRepository 추가

• 게시글 조회할 때 게시글에 달린 댓글들 같이 조회하게끔 수정.
  -> Post Entity와 PostResonseDto 수정

• 댓글 작성, 수정, 삭제 기능 구현
  -> 댓글 작성 : DB에 포스트 저장 유무 확인
  -> 댓글 수정, 삭제 : DB에 댓글 저장 유무 확인
  -> 관리자와 작성자만 댓글 수정, 삭제 가능

코드리뷰입니다.

Overview

• issue 단위로 브랜치 관리하시는 부분이 인상적입니다 👍

#13 (comment) 관련

• 현재 구성해주신 패키지 구조도 많이 사용되는 구조입니다.
• 중요한 점은 이전에 현웅 튜터님께서 올려주신 것처럼, 도메인에 입각해 패키지를 구성해야 한다는 점입니다.
• 현 프로젝트는 복잡도가 많이 높지 않기 때문에, 세세하게 분류할 필요가 없지만 프로젝트 복잡도가 올라가면 user.controller / user.entity와 같은 형태나, entity.user / entitiy.post와 같은 형태를 사용하기도 합니다.

#12 관련

• 개인적으로 AOP보다 참고 자료의 방법으로 exception handling하는 걸 선호하긴 합니다.

  • 일전에 승현 튜터님께서도 비슷한 코멘트 (https://github.com/JisooPyo/Spring-Blog-Assignment/issues/1#issuecomment-1632943463)를 주신 것으로 보이네요 😄

• HTTP Status Code로도 많은 정보를 보낼 수 있는데요. 참고 자료를 통해 에러 코드를 보내는 것도 추천 드립니다.

• ERD 첨부해주신 것을 보면 likes 테이블에 특정 id 관련 row수를 구함으로써 count를 구할 수 있는데, 별도 컬럼으로 중복 저장해주신 것으로 보입니다.

  • 물론, 반정규화를 의도하셨을 수 있지만, orm에 따라 반정규화를 권장하지 않기도 해서 유의해주시길 바랍니다!

• 지금 커밋 메시지를 잘 작성해주시는 편이긴 한데, AngularJS commit conventions 이 현재 커밋하시는 스타일과 유사해서 참고해보시면 좋을 것 같습니다.

• 세부 피드백은 아래 코멘트에 남겨두겠습니다. 🙇

• 사용자는 선택한 게시글에 '좋아요'를 할 수 있다.
• 사용자가 이미 '좋아요'한 게시글에 다시 '좋아요' 요청을 하면 '좋아요'를 했던 기록이 취소된다.
• 요청이 성공되면 Client로 성공했다는 메시지, 상태코드를 반환한다.

+) 게시글 조회 시 좋아요 개수도 반환한다.

• SpringSecurity를 이용하여 WebSecurityConfig를 설정한다.
• 기존에 쓰던 PasswordConfig 기능을 WebSecurityConfig에 넣어서 사용한다.
• 기존에 쓰던 AuthFilter를 사용하지 않는다.(Security를 사용하므로)
• Jwt를 쿠키에 담아서 보내거나 받는 형식이 아니라 헤더에 담아서 주고받는 식으로 바꾼다.
  --> JwtAuthenticationFilter, JwtAuthorizationFilter, UserDetailsImpl, UserDetailsServiceImpl 파일 만들기
• 포스트맨으로 회원가입 및 로그인 요청해보기
  --> JWT가 헤더에 잘 담겨 오는지 확인, jwt를 jwt,io에서 확인해보기.

• 패키지 관련 있는 것끼리 묶어 볼 것 ex. User는 User와 관련된 파일들끼리

ERD, 기능 수정

• AOP를 활용하여 예외처리하기

• 예외 처리
  -> 토큰이 필요한 API 요청에서 토큰을 전달하지 않았거나 정상 토큰이 아닐 때는 "토큰이 유효하지 않습니다." 라는 에러메시지와 statusCode: 400을 Client에 반환하기
  -> 토큰이 있고, 유효한 토큰이지만 해당 사용자가 작성한 게시글/댓글이 아닌 경우에는 “작성자만 삭제/수정할 수 있습니다.”라는 에러메시지와 statusCode: 400을 Client에 반환하기
  -> DB에 이미 존재하는 username으로 회원가입을 요청한 경우 "중복된 username 입니다." 라는 에러메시지와 statusCode: 400을 Client에 반환하기
  -> 로그인 시, 전달된 username과 password 중 맞지 않는 정보가 있다면 "회원을 찾을 수 없습니다."라는 에러메시지와 statusCode: 400을 Client에 반환하기
  -> 회원가입 시 username과 password의 구성이 알맞지 않으면 에러메시지와 statusCode: 400을 Client에 반환하기

• Service에서 겹치는 예외가 많다. 잘 핸들링 해서 밖으로 빼볼 것