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
hyxklee
reviewed
Nov 4, 2024
| @RestController | ||
| @RequestMapping("/api/v1/posts") | ||
| @RequiredArgsConstructor | ||
| public class PostController { |
There was a problem hiding this comment.
@tag, @operation 어노테이션을 이용해 스웨거 설명도 추가해주시면 좋을 것 같아요!
| // 게시물 ID로 조회 | ||
| @GetMapping("/{id}") | ||
| public ResponseDto<PostResponseDto> getPost(@PathVariable Long id) { | ||
| return postService.getPostResponse(id); |
There was a problem hiding this comment.
응답 객체, 즉 ResponseDto를 만들고, 안에 코드를 넣고, 메시지를 넣는 책임. 즉 응답에 관련된 책임은 controller 단에서 하는 것이 좋다고 생각해요!
따라서 서비스 단에서 ResponseDto를 만들어서 그대로 반환하기 보다,
@GetMapping("/{id}")
public ResponseDto<PostResponseDto> getPost(@PathVariable Long id) {
return ResponseDto.response(응답 코드, 응답 메시지, postService.getPostResponse(id));
}
이렇게 구현하는 것은 어떨까요??
| private List<Image> images = new ArrayList<>(); | ||
|
|
||
| @OneToMany(mappedBy = "post", cascade = CascadeType.REMOVE, orphanRemoval = true) | ||
| private List<Comment> commentList = new ArrayList<>(); |
There was a problem hiding this comment.
ERD가 수정이 되어서 Comment가 사라지고, Post의 자기 참조로 reply를 구현하게 되어 있어서 Comment 도메인은 제거해도 될 것 같아요!
hyxklee
approved these changes
Nov 5, 2024
| public PostResponseDto createPost(PostRequestDTO postRequestDTO, String email) { | ||
| // 이메일로 사용자 조회 | ||
| User user = userService.find(email); // JWT에서 추출한 이메일 사용 | ||
| if (user == null) { |
There was a problem hiding this comment.
userService.fin() 메서드에서 이미 null checking을 해주고 있기 때문에 따로 해주지 않아도 됩니다! if문 제거 부탁드릴게요
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.
1. 무슨 이유로 코드를 변경했나요?
첫 기능 개발시, 엔티티 클래스 구조를 제대로 잡지 않고 개발을 시작하여 갈수록 꼬이는 현상이 발생하여서
처음부터 엔티티 클래스 구현-> 관계 매핑 ->DTO 구현 서비스 로직 구현 -> 컨트롤러 구현 이런 순서로 개발을 진행하였습니다!
2. 어떤 위험이나 장애를 발견했나요?
현재 push 된 코드에서 포스트맨에서 테스트를 해 본 결과, 좋아요 추가 기능에서 추가는 잘 돌아갔다고 뜨나, 좋아요를 취소하는 요청을 테스트 해보면, 좋아요가 눌리지 않은 게시물이라고 뜹니다.
원래라면 좋아요가 취소 되었습니다. 라는 응답이 떠야 정상입니다.
이 부분에 대해서 피드백 주시면 수정 하겠습니다
3. 관련 스크린샷을 첨부해주세요.
4. 완료 사항
현재 글 생성, 글 조회, 좋아요 수로 정렬한 게시물 조회, 최신 게시물 조회, 좋아요 추가 , 좋아요 삭제, 글 삭제 등등의 기능을 구현하였으며
글 인용 ,리트윗 기능을 추가로 구현하여야합니다.
5. 추가 사항