merge to main #71
merge to main #71
Conversation
Summary of ChangesHello @jiminnimij, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 사용자가 자신의 '응원' 및 '질문' 게시물을 조회할 수 있는 기능을 도입합니다. 이 기능은 새로운 데이터 전송 객체(DTO) 추가, 해당 컨트롤러에 새로운 API 엔드포인트 구현, 그리고 이러한 게시물의 검색 및 페이지네이션을 처리하는 서비스 메서드 업데이트를 포함합니다. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
| @@ -1,14 +1,19 @@ | |||
| package com.example.tikitaka.domain.post.controller; | |||
|
|
|||
| import com.example.tikitaka.domain.post.dto.ExhibitionPreview; | |||
There was a problem hiding this comment.
사용되지 않는 ExhibitionPreview import 구문이 있습니다. 불필요한 import는 코드를 정리하고 혼란을 줄이기 위해 제거하는 것이 좋습니다.
| @@ -1,14 +1,19 @@ | |||
| package com.example.tikitaka.domain.post.controller; | |||
|
|
|||
| import com.example.tikitaka.domain.post.dto.ExhibitionPreview; | |||
There was a problem hiding this comment.
사용되지 않는 ExhibitionPreview import 구문이 있습니다. 불필요한 import는 코드를 정리하고 혼란을 줄이기 위해 제거하는 것이 좋습니다.
| public MyPreviewGetResponse myCheerList( | ||
| @AuthenticationPrincipal Long memberId, | ||
| @RequestParam(required = true) int pageNum, | ||
| @RequestParam(required = true) int limit | ||
| ) { | ||
| return previewService.getMyPreviews(memberId, PostType.QUESTION, pageNum, limit); | ||
| } |
There was a problem hiding this comment.
메소드 이름이 myCheerList로 되어 있어 QuestionController의 맥락과 맞지 않습니다. 이 컨트롤러는 질문(Question)에 관련된 기능을 다루므로, 메소드 이름을 myQuestionList로 변경하여 코드의 가독성과 명확성을 높이는 것이 좋겠습니다.
| public MyPreviewGetResponse myCheerList( | |
| @AuthenticationPrincipal Long memberId, | |
| @RequestParam(required = true) int pageNum, | |
| @RequestParam(required = true) int limit | |
| ) { | |
| return previewService.getMyPreviews(memberId, PostType.QUESTION, pageNum, limit); | |
| } | |
| public MyPreviewGetResponse myQuestionList( | |
| @AuthenticationPrincipal Long memberId, | |
| @RequestParam(required = true) int pageNum, | |
| @RequestParam(required = true) int limit | |
| ) { | |
| return previewService.getMyPreviews(memberId, PostType.QUESTION, pageNum, limit); | |
| } |
| import java.util.List; | ||
| import java.util.Objects; | ||
|
|
||
| import static java.util.stream.Collectors.toList; |
There was a problem hiding this comment.
코드에서 Stream.toList() (Java 16+ 기능)를 사용하고 있으므로, java.util.stream.Collectors.toList의 static import는 필요하지 않습니다. 불필요한 import를 제거하여 코드를 더 깔끔하게 유지하는 것이 좋습니다.
| List<MyPreview> previesDto = previews.getContent().stream().map( | ||
| MyPreview::of | ||
| ).toList(); | ||
|
|
||
| return MyPreviewGetResponse.of(previesDto, pageInfo); |
There was a problem hiding this comment.
변수명에 오타가 있습니다. previesDto를 previewsDto로 수정하여 다른 변수명과 일관성을 맞추고 코드의 가독성을 높이는 것을 권장합니다.
| List<MyPreview> previesDto = previews.getContent().stream().map( | |
| MyPreview::of | |
| ).toList(); | |
| return MyPreviewGetResponse.of(previesDto, pageInfo); | |
| List<MyPreview> previewsDto = previews.getContent().stream().map( | |
| MyPreview::of | |
| ).toList(); | |
| return MyPreviewGetResponse.of(previewsDto, pageInfo); |
#️⃣ 연관된 이슈
#️⃣ 작업 내용
#️⃣ 테스트 결과
#️⃣ 변경 사항 체크리스트
#️⃣ 스크린샷 (선택)
#️⃣ 리뷰 요구사항 (선택)
📎 참고 자료 (선택)