[Release] v2.1.4 마이페이지 rating 문제 해결을 위한 배포 #328
Conversation
* fix: 마이페이지 rating null인 문제 수정 * refactor: gemini 리뷰 반영
* fix: 마이페이지 rating null인 문제 수정 * refactor: gemini 리뷰 반영 * fix: rating null나오는 문제 수정
* fix: 마이페이지 rating null인 문제 수정 * refactor: gemini 리뷰 반영 * fix: rating null나오는 문제 수정 * refactor: primitive형 rating 수정
Summary of ChangesHello @JjungminLee, 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은 마이페이지에서 사용자 리뷰의 평점 표시와 관련된 문제를 해결하기 위한 v2.1.4 릴리즈입니다. 기존 평점 처리 로직을 개선하고, 사용자 리뷰 조회 기능을 최적화하며, 디버깅을 위한 로깅을 추가하여 안정성과 정확성을 높였습니다. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. 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}
There was a problem hiding this comment.
Code Review
이번 PR은 마이페이지의 리뷰 평점 표시 문제를 해결하기 위한 변경으로 보입니다. 새로운 Ratings 객체의 평점을 우선적으로 사용하고, 없을 경우 기존 rating 필드를 사용하도록 로직을 수정한 점이 인상적입니다. 다만, 코드에 디버깅 목적으로 추가된 것으로 보이는 로깅 코드가 여러 곳에 남아있습니다. 프로덕션 환경에 배포되기 전에 이러한 코드들은 제거하는 것이 좋습니다. 또한, 사용되지 않는 새로운 repository 메소드가 추가되었는데, 이 역시 정리할 필요가 있어 보입니다. 자세한 내용은 각 파일에 남긴 리뷰 코멘트를 참고해주세요.
| sliceReviews.forEach(item -> { | ||
| Ratings r = item.getRatings(); | ||
| log.info( | ||
| "reviewId=" + item.getId() | ||
| + ", ratingCol=" + item.getRating() | ||
| + ", ratingsObj=" + (r == null ? "null" : "not-null") | ||
| + ", main=" + (r == null ? null : r.getMainRating()) | ||
| + ", amount=" + (r == null ? null : r.getAmountRating()) | ||
| ); | ||
| }); |
There was a problem hiding this comment.
findMyReviews 메소드에 추가된 로깅 코드는 디버깅 목적으로 보입니다. 프로덕션 코드에 포함될 경우 불필요한 로그가 대량으로 생성되어 성능에 영향을 줄 수 있고 로그 파일을 어지럽힐 수 있습니다. 병합 전에 이 코드를 제거하는 것을 권장합니다.
| log.info("ratings = {}", review.getRatings()); | ||
| if (review.getRatings() != null) { | ||
| log.info("mainRating = {}", review.getRatings().getMainRating()); | ||
| } | ||
| log.info("legacy rating = {}", review.getRating()); |
There was a problem hiding this comment.
from 메소드에 추가된 log.info 호출은 디버깅용 코드로 보입니다. 이러한 로그는 프로덕션 환경에서 불필요하며, 민감한 정보를 노출할 위험도 있습니다. 병합 전에 제거하는 것이 좋습니다.
| @Query(""" | ||
| select r | ||
| from Review r | ||
| where r.user = :user | ||
| and (:lastReviewId is null or r.id < :lastReviewId) | ||
| order by r.id desc | ||
| """) | ||
| List<Review> findMyReviews( | ||
| @Param("user") User user, | ||
| @Param("lastReviewId") Long lastReviewId, | ||
| Pageable pageable | ||
| ); |
There was a problem hiding this comment.
새로 추가된 findMyReviews 메소드가 코드 내에서 사용되지 않고 있습니다. 불필요한 코드는 혼란을 야기하고 유지보수성을 떨어뜨릴 수 있으므로, 사용되지 않는다면 제거하는 것이 좋습니다.
#️⃣ Issue Number
📝 요약(Summary)
💬 공유사항 to 리뷰어
✅ PR Checklist
PR이 다음 요구 사항을 충족하는지 확인하세요.