[4기 전선희] Spring Boot JPA 게시판 구현 미션 제출합니다! #220
Conversation
funnysunny08
requested review from
hseong3243,
Seongju-Lee,
bjo6300 and
seongHyun-Min
| .andExpect(status().isOk()) | ||
| .andDo(print()) | ||
| .andDo(document("get-all-post", | ||
| responseFields( |
| UpdatePostRequest updatePostRequest = new UpdatePostRequest("바뀐 제목", null); | ||
|
|
||
| mockMvc.perform(patch("/posts/{postId}", post1.getPostId()) | ||
| .header("userId", post1.getUser().getUserId()) |
|
|
||
| @ManyToOne(fetch = FetchType.EAGER, cascade = CascadeType.ALL) | ||
| @JoinColumn(name = "user_id") | ||
| private User user; |
There was a problem hiding this comment.
페치 타입은 eager, cascade 타입은 ALL로 설정하신 이유가 무엇인가요?
| String title, | ||
| String content | ||
| ) { | ||
| } |
There was a problem hiding this comment.
CreatePostRequest와 UpdatePostRequest가 같다면 하나로 통합해서 사용하면 어떠신가요??
|
|
||
| @Column | ||
| @Setter | ||
| private String hobby; |
There was a problem hiding this comment.
Setter 보다 비즈니스적으로 의미있는 네이밍으로 변경 메서드를 제공하는 것은 어떨까요?
| String title, | ||
| String content |
There was a problem hiding this comment.
값의 대한 검증이 추가되면 어떨까요? Spring Validation에 대해 공부해보시면 좋을 것 같아요~
| private final PostRepository postRepository; | ||
| private final UserRepository userRepository; | ||
|
|
||
| private static final int PAGE_SIZE = 10; |
There was a problem hiding this comment.
PAGE_SIZE를 서버에서 고정시키는 이유가 궁금합니다~ 매개 변수로 받을 수 있는 편이 좀 더 유연하지 않을까요?
| @RestControllerAdvice | ||
| @Component | ||
| @RequiredArgsConstructor | ||
| public class ControllerExceptionAdvice { | ||
| /** |
There was a problem hiding this comment.
@Component, @RequiredArgsConstructor 불필요한 것 같습니다~
|
|
||
| @Column | ||
| @Setter | ||
| private String hobby; |
There was a problem hiding this comment.
Setter 보다 비즈니스적으로 의미있는 네이밍으로 변경 메서드를 제공하는 것은 어떨까요?
| .content(request.content()) | ||
| .user(user) | ||
| .build(); | ||
| post.setCreatedBy(user.getName()); |
There was a problem hiding this comment.
JPA Auditing을 사용하셨는데 이를 활용해도 되지 않나요?
| String createdBy, | ||
| LocalDateTime createdAt | ||
| ) { | ||
| public static PostResponse of(Long postId, String title, String content, String createdBy, LocalDateTime createdAt) { |
There was a problem hiding this comment.
주성님 말씀처럼 Post를 통해 받는 것이 파라미터 수를 줄일 수 있을 것 같네요 ㅎ
| public ApiResponse<PostsResponse> getAllPost( | ||
| @RequestParam int page | ||
| ) { | ||
| if (page < 1) { | ||
| return ApiResponse.error(Error.PAGE_REQUEST_VALIDATION_EXCEPTION, Error.PAGE_REQUEST_VALIDATION_EXCEPTION.getMessage()); | ||
| } |
There was a problem hiding this comment.
0은 포함하지 않나요..? Pageable을 파라미터로 전달받지 않으시는 이유가 궁금합니다~
| @Getter | ||
| @RequiredArgsConstructor(access = AccessLevel.PRIVATE) | ||
| @AllArgsConstructor(access = AccessLevel.PRIVATE) | ||
| public class ApiResponse<T> { |
There was a problem hiding this comment.
ApiResponse를 따로 만드시는 이유가 궁금합니다~ code의 경우 HTTP Code와 동일한데 굳이 필요한가요..?
| return ApiResponse.error(Error.PAGE_REQUEST_VALIDATION_EXCEPTION, Error.PAGE_REQUEST_VALIDATION_EXCEPTION.getMessage()); | ||
| } | ||
|
|
||
| return ApiResponse.success(Success.GET_POST_LIST_SUCCESS, postService.getAllPost(page)); |
There was a problem hiding this comment.
다음과 같이 postService.getAllPost(page)에서 반환되는 값이 한줄로 표현되는 경우는 오히려 가독성을 해칠 수 있어요.
반환 값을 먼저 뺀 다음에 success의 파라미터로 넣어두는게 좋을 것 같아요.
| } | ||
|
|
||
| @GetMapping("/{postId}") | ||
| @ResponseStatus(HttpStatus.OK) |
There was a problem hiding this comment.
Success.GET_POST_SUCCESS를 통해서 status를 반환하고 있는데, @ResponseStatus를 붙여줄 필요가 있나요?
| String createdBy, | ||
| LocalDateTime createdAt | ||
| ) { | ||
| public static PostResponse of(Long postId, String title, String content, String createdBy, LocalDateTime createdAt) { |
There was a problem hiding this comment.
주성님 말씀처럼 Post를 통해 받는 것이 파라미터 수를 줄일 수 있을 것 같네요 ㅎ
| @@ -0,0 +1,23 @@ | |||
| package com.board.server.domain.post.entity; | |||
There was a problem hiding this comment.
repository가 entity 패키지에 들어있는 이유는 무엇인가요?
| // CREATE | ||
|
|
||
| // READ | ||
| Page<Post> findAllByOrderByCreatedAtDesc(Pageable pageable); | ||
|
|
||
| @Query("select p from Post p join fetch p.user where p.postId = :postId") | ||
| Optional<Post> findById(Long postId); | ||
|
|
||
| // UPDATE | ||
|
|
||
| // DELETE |
There was a problem hiding this comment.
불필요한 주석은 혼란만 야기할 수 있어요 ㅎ
커밋하기 전에 확인하고 지워주는 습관을 가지는게 좋슴다
| public class NotFoundException extends CustomException { | ||
| public NotFoundException(Error error, String message) { | ||
| super(error, message); | ||
| } | ||
| } |
There was a problem hiding this comment.
커스텀 예외를 만드셨는데, 어떠한 이유로 만드시게 되었나요?
📌 과제 설명
JPA 강의를 통해 배운 JPA, rest-docs 등을 적용하여 간단한 게시판을 구현하였습니다!
👩💻 요구 사항과 구현 내용
SpringDataJPA 를 설정한다.
엔티티를 구성한다
회원(User)
게시글(Post)
회원과 게시글에 대한 연관관계를 설정한다.
게시글 Repository를 구현한다. (PostRepository)
API를 구현한다.
게시글 조회
게시글 작성 (POST "/posts")
게시글 수정 (POST "/posts/{id}")
REST-DOCS를 이용해서 문서화한다.