SSAFY 관통 프로젝트 Refactoring

[Leehyoungwoo]

일정: 2023-11-27 ~ 2023-12-01

토요일 스터디 이전에 혼자 코드 수정 해보기

• 테이블 설계 (테이블 설계 경험 無 )
  • 게시판 part Refactoring을 위한 member, board관련 테이블 설계해보기
  • key 식별관계 공부 및 고려하면서
  • Enjoytrip의 비즈니스 로직 코드는 단순히 작업양만 많아지기 때문에 스킵
  • 게시판 글, 댓글, 좋아요, 댓글 좋아요 api Refactoring
  • 예외 처리를 컨트롤러 단에서 할지, Service단에서 할지 여부에 관한 고민
    • 예외 처리를 할때 단순하게 RuntimeException만 던졌는데 세부적으로 예외를 던질 필요에 대한 고민
    • 다양한 예외 상황을 고려하지 않고 되는 상황만 가정한 코드 수정
  • API 별로 컨트롤러에서 response로 반환할 타입에 대한 고려, 프론트에서 데이터를 받았을 때, 처리가 쉬워야 하지 않나
  • JWT 토큰 적용해보기
  • JWT가 프로젝트에 들어있지만, 내가 시큐리티를 몰라서 시큐리티 컨피그에서 막아버림, 토큰으로 각 API엔드포인트 접근 제한하는 방법
  • Dto 분리, 컨트롤러 분리 고민?
  • 관통 후반부에 시간 부족으로 dto는 하나로 돌려 쓰고, controller도 충분히 고민하지 않고 섞어 썻는데 보다 좋은 코드를 위해 무엇이 나은지 고민해보고 수정하기
• 프론트엔드
  • 웬만하면 하나도 안건드리고 싶지만 기능이 동작하지 않는 부분이 생겨서 refactoring 필요, 돌아가게만 하자..
  • 하면서 axios 개념 공부, 코딩해보기

[Leehyoungwoo]

쓰고나니 체크 박스로 만들지 않았는데 진행 여부를 한눈에 보기 위해서 내일 checkbox로 수정해야겠다

[all-cloudz]

https://github.com/Leehyoungwoo/vacation-study/issues/8#issuecomment-1831376551

• 에러 메세지가 원하는대로 response에 담기지 않아요
  • StringUtils.isEmpty()때문일까?
• 삭제 성공 시에는 HttpStatus.NO_CONTENT
• get요청에는 HttpStatus.OK, post요청에는 HttpStatus.CREATED

[all-cloudz]

https://github.com/Leehyoungwoo/vacation-study/issues/8#issuecomment-1831669820

• 19:55분 Board CRUD refactoring 완료
  • Service층에서 최대한 발생할 수 있는 여러 개의 예외를 던져 Controlle층에서 ResponseMessage의 메세지에 담아 ResponseEntity에 넣음
  • HttpStatus도 최대한 상황에 맞게 넣으려고 함

[all-cloudz]

https://github.com/Leehyoungwoo/vacation-study/issues/9#issuecomment-1833284372

게시물 목록에서 페이지 size를 RequestParam으로 받아오는게 맞나..? 내부적으로 페이지 당 게시물 수를 정해놓으면 받아올 필요가 없지 않나? @RequestParam(name = "pageSize", defaultValue = "8") int pageSize)

[all-cloudz]

https://github.com/Leehyoungwoo/vacation-study/issues/9#issuecomment-1833876348

댓글 작성 요청 url이 http://localhost:8080/board/4/comments인데 요청 body에 { "boardId": 4, "content": "댓글 테스트", "memberId": 1 } 이러면 boardId가 중복되는 감이 있는데 이렇게 해도 되는건가?

[all-cloudz]

https://github.com/Leehyoungwoo/vacation-study/issues/9#issuecomment-1833932129

• 글쓰기 테이블엔 text로 되어있어도 잘 읽어지는데 댓글은 왜 text로 되어있다고 안읽어지지 content가

[all-cloudz]

https://github.com/Leehyoungwoo/vacation-study/issues/9#issuecomment-1834076358

• postman으로 Refactoring api 요청 정상 처리 확인

[all-cloudz]

#8 (comment)

• 에러 메세지가 원하는대로 response에 담기지 않아요
  • StringUtils.isEmpty()때문일까?
• 삭제 성공 시에는 HttpStatus.NO_CONTENT
• get요청에는 HttpStatus.OK, post요청에는 HttpStatus.CREATED

@Valid로 인해 미리 체크하기 때문이야. 추가로 고민해봐야 하는 건, '값 검증을 어느 레이어에서 하는 게 적절한가?' 일 것 같아. 마지막으로 @ControllerAdvice, @RestControllerAdvice를 공부해야 할 것 같아!

[all-cloudz]

#9 (comment)

게시물 목록에서 페이지 size를 RequestParam으로 받아오는게 맞나..? 내부적으로 페이지 당 게시물 수를 정해놓으면 받아올 필요가 없지 않나? @RequestParam(name = "pageSize", defaultValue = "8") int pageSize)

유연성을 위해 프론트에서 값을 받아오는 걸 추천!

[Leehyoungwoo]

#9 (comment)

댓글 작성 요청 url이 http://localhost:8080/board/4/comments인데 요청 body에 { "boardId": 4, "content": "댓글 테스트", "memberId": 1 } 이러면 boardId가 중복되는 감이 있는데 이렇게 해도 되는건가?

정책에 따라 다를 수 있음

1. boardId, WritingDto를 같이 넘겨주기
2. WritingDto의 boardId에 setter로 컨트롤러에서 완성해서 service로 넘겨주기

[all-cloudz]

'2023-12-02' 까지 해야할 것

1. 클래스 Member리팩토링
2. 코드 중복 제거 해주세요, 예외 핸들링 하면 될 것 같아요
3. ResponseMessage의 의미를 만들어주세요, 아니면 없애주세요
4. Service에 트랜잭션 적용
5. Optional 적용
6. final 키워드 적용