Conversation
Board.java의 전체적인 코드 구조를 개선하였습니다. createdAt이 null 값으로 찍히는 문제와 불필요한 주석 정리는 추후 수정 예정입니다.
기존 코드를 dto를 사용하여 수정하고 createdAt이 null값으로 찍히는 문제를 해결하였습니다.
기존 h2 기반의 db를 MySQL로 변경하였습니다.
기존 update API를 PUT 메소드에서 PATCH 메소드로 변경하였습니다.
Collaborator
|
기본기가 매우 탄탄하신 것 같습니다. 그리고 전반적인 JPA와 계층 분리,DTO에 대한 이해도가 높으신것 같습니다. @Setter를 지양하고 팩토리 메서드나 빌더를 활용하신 점이 인상 깊습니다. 특히 @NoArgsConstructor(access = AccessLevel.PROTECTED) 와 빌더 패턴을 통해서 개발자가 지정한 형식으로만 객체를 생성하도록 하여 데이터 정합성을 견고히 하는 부분을 새로 배워갑니다. |
Collaborator
|
본인이 궁금한 걸 바로바로 찾아내시는 모습을 꼭 본받아야 할거 같습니다! 수고하셨습니다! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
-> 빌더패턴 사용, update 메서드를 추가해서 사용
-> BoardCreateRequestDTO, BoardUpdateRequestDTO 생성
@RequestMapping 대신 @PostMapping, @GetMapping 등을 사용
PUT과 PATCH의 차이