[신윤섭] 3차 과제 제출 #12
Open
[신윤섭] 3차 과제 제출 #12
Conversation
This file contains 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
- key 값이 공유 되지 않도록 application.properties 제외
- SaveSinglePostRequest.java - SavePostsByExcelRequest.java
- id를 기반으로 게시글 생성 - request dto을 record로 변경 시 생긴 코드 수정 반영
- AUTO -> IDENTITY
- JdbcTemplate를 이용한 Batch Insert
- id를 기반으로 게시글 조회 및 영속성 컨텍스트 이용한 조회수 증가
- 오타 수정 및 직관적인 네이밍 반영
- likes를 기반으로한 Pageable 정렬 기능 추가 위함
- 페이지네이션과 추가 기능(현재, 전체 페이지 수) 구현 - likes 값에 따른 DESC 정렬
- 메서드명을 더 정확하게 수정
- URI 경로를 RESTful 원칙에 맞게 수정
- /search → /{id}
- /single → /
- /delete → /{id}
| page.getTotalPages(), | ||
| page.getTotalElements() | ||
| ); | ||
| } |
There was a problem hiding this comment.
오 이렇게 따로 PageResponse를 만들어서 사용하시는군요!
맨날 Page 응답하면 Warning 뜨던데 이렇게 처리하면 좋을 거 같네요! 배워갑니다
| @GeneratedValue(strategy = GenerationType.IDENTITY) | ||
| private Long id; | ||
|
|
||
| @NonNull |
There was a problem hiding this comment.
@NonNull을 필드와 생성자에서 모두 사용하시는 군요!
생성자에 붙이면 lombok이 처리해줄텐데 중복되는 느낌으로 필드에도 붙여주신 이유가 있을까요?
이 부분에서 @Column(nullable=false) 대신 @NonNull을 붙이신 이유가 궁금합니다!
| return new Post(title, content, name, VIEWS_INIT_VALUE, LIKES_INIT_VALUE); | ||
| }) | ||
| .toList(); | ||
| postJdbcRepository.saveAllPost(posts); |
| } catch (Exception e) { | ||
| log.error("Failed to save single post", e); | ||
| throw ApiException.from(INTERNAL_SERVER_ERROR); | ||
| } |
There was a problem hiding this comment.
이 부분에서 try catch예외처리를 해주신 이유가 궁급합니다!
| @Override | ||
| public PostDto findPostById(Long id) { | ||
| try { | ||
| Post findPost = postRepository.findById(id).orElseThrow(); |
There was a problem hiding this comment.
혹시 여기서 orElseThrow에서 어떤 예외를 던질지 명시해주지 않은 이유가 있으신가요!?
chaen-ing
reviewed
Nov 27, 2024
Comment on lines
+7
to
+13
| @Builder | ||
| public record PostDto(@NonNull String title, @NonNull String content, @NonNull String name, int views, int likes) { | ||
|
|
||
| public static PostDto from(Post post) { | ||
| return new PostDto(post.getTitle(), post.getContent(), post.getName(), post.getViews(), post.getLikes()); | ||
| } | ||
| } |
There was a problem hiding this comment.
레코드에 빌더를 쓰니까 더 코드가 깔끔해지네요! 저도 사용해봐야겠어요!
Comment on lines
+32
to
+33
| private final static int VIEWS_INIT_VALUE = 0; | ||
| private final static int LIKES_INIT_VALUE = 0; |
| import cotato.backend.domains.post.dto.request.SavePostsByExcelRequest; | ||
| import cotato.backend.domains.post.dto.request.SaveSinglePostRequest; | ||
|
|
||
| public interface PostService { |
There was a problem hiding this comment.
PostService 로 PostServiceImpl 을 한번 감싼 형태의 구조를 채택하신 이유가 궁금합니다!
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.
Post API 구현 체크리스트 및 API 명세
1. Post 엔티티 설계 및 구현
2. 게시글 단일 생성 기능
POST /api/posts3. 게시글 다중 생성 기능 (엑셀)
POST /api/posts/excel4. 게시글 조회 기능
GET /api/posts/{id}5. 게시글 목록 조회 기능
GET /api/posts6. 게시글 삭제 기능
DELETE /api/posts/{id}고민되는 점
Pageable도 한 번 더 살펴볼 필요가 있을 것 같다.