Refactor: 커뮤니티 상단 팔로잉 유저 아이콘 관련 null 값 수정 #175
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Summary of Changes
Hello @2anizirong, 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!
이 풀 리퀘스트는 커뮤니티 상단에 표시되는 팔로잉 유저 아이콘과 관련된 데이터 문제를 해결합니다. 기존에는 특정 사용자 프로필 정보(닉네임, 소개)가 null로 로드되어 아이콘에 제대로 표시되지 않았습니다. 이번 변경을 통해 해당 필드들이 올바르게 포함되도록 수정하여, 사용자 경험을 개선하고 데이터 무결성을 확보했습니다. 또한, 코드 가독성을 위한 사소한 포맷 조정도 포함되어 있습니다.
Highlights
- 사용자 프로필 아이콘 DTO의 null 값 수정: 커뮤니티 상단에 표시되는 팔로잉 유저 아이콘의 프로필 정보(닉네임, 소개)가
null로 표시되던 문제를 해결하기 위해UserConverter의toProfileIconDto메서드에nickname과introduce필드를 추가했습니다. 이로써 사용자 프로필 아이콘이 완전한 정보를 갖게 됩니다. - 사소한 코드 포맷 조정:
UserResponseDTO.java파일에 공백 라인이 추가되었고,UserServiceImpl.java파일의followingUserIconList메서드 시그니처에 파라미터 간 공백이 추가되었습니다. 이 변경사항들은 기능적인 영향 없이 코드 가독성을 향상시키는 사소한 포맷 조정입니다.
Using Gemini Code Assist
The 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 in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.
| Feature | Command | Description |
|---|---|---|
| Code Review | /gemini review |
Performs a code review for the current pull request in its current state. |
| Pull Request Summary | /gemini summary |
Provides a summary of the current pull request in its current state. |
| Comment | @gemini-code-assist | Responds in comments when explicitly tagged, both in issue comments and review comments. |
| Help | /gemini help |
Displays a list of available commands. |
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 .gemini/ folder in the base of the repository. Detailed instructions can be found here.
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
-
Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩
There was a problem hiding this comment.
Code Review
안녕하세요. 팔로잉 유저 아이콘 관련 null 값을 수정하는 PR 잘 보았습니다.
UserConverter.java에서 toProfileIconDto에 nickname과 introduce 필드를 추가하여 문제를 해결하신 것을 확인했습니다.
다만, 이 수정으로 인해 유사한 기능을 하는 toProfileDto 메서드와 반환 값에 불일치가 발생하게 되었습니다. 코드의 일관성과 유지보수성을 높이기 위해 두 메서드를 리팩토링하는 것을 제안하는 리뷰 의견을 남겼습니다. 관련 todo 주석도 있었던 만큼 이번 기회에 함께 개선하면 더 좋은 코드가 될 것 같습니다.
자세한 내용은 개별 코멘트를 확인해주세요.
| .nickname(user.getNickname()) | ||
| .introduce(user.getIntroduce()) |
There was a problem hiding this comment.
좋은 수정입니다. nickname과 introduce를 추가하여 null 문제를 해결하셨네요.
다만, 바로 아래 toProfileDto 메서드와 기능이 거의 중복되며, 이번 수정으로 인해 두 메서드가 반환하는 ProfileDto 객체의 내용에 불일치가 발생했습니다. (toProfileIconDto는 introduce를 포함하지만 toProfileDto는 포함하지 않음).
// todo 주석에서도 언급되었듯이 두 메서드를 통합하는 리팩토링을 진행하는 것이 좋아 보입니다. 예를 들어, toProfileDto가 introduce 필드도 설정하도록 수정하고, toProfileIconDto는 toProfileDto(user, true)를 호출하도록 변경하면 코드 중복을 줄이고 일관성을 유지할 수 있습니다.
이번 PR에서 함께 수정하는 것을 고려해 주세요.
|
@kjhh2605 같은 dto 사용으로 해당 API에서 필요없는 부분은 null로 받아오는게 맞다고 합니다. |
#️⃣연관된 이슈
📝작업 내용