[Fix] 내 리뷰에서 사진이 안보이는 현상 고치기 및 null일때 디폴트값 처리 #378
Conversation
Summary of ChangesHello @HI-JIN2, 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은 사용자가 '내 리뷰' 페이지에서 자신의 리뷰 사진을 볼 수 없었던 중요한 버그를 해결합니다. 이 버그는 사진이 다른 섹션에서는 올바르게 표시되었음에도 불구하고 특정 페이지에서만 발생했습니다. 또한, 데이터 모델 변환 과정에서 발생할 수 있는 null 값 관련 오류를 방지하기 위해 여러 필드에 대한 기본값 처리 로직을 추가하여 애플리케이션의 안정성을 향상시켰습니다. 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}
There was a problem hiding this comment.
Code Review
이 PR은 '내 리뷰' 화면에서 리뷰에 포함된 사진이 보이지 않는 버그를 수정하고, API 응답의 null 값에 대한 기본값 처리를 추가합니다.
MyReviewAdapter에서 CardView의 visibility를 올바르게 처리하여 사진이 표시되도록 수정한 점은 좋습니다. 또한 이미지 URL을 다루는 로직을 firstOrNull을 사용하여 더 안전하고 간결하게 개선한 점도 훌륭합니다.
MyReviewResponse에서 API 응답의 null 가능성에 대비한 것은 좋지만, 몇 가지 개선할 점이 보입니다. reviewId의 기본값으로 1을 사용하는 것은 중복 ID 문제를 일으킬 수 있으므로 더 안전한 값으로 변경하는 것을 제안합니다. 또한, 불필요한 코드가 있어 수정을 제안했습니다. 자세한 내용은 각 파일의 주석을 확인해주세요.
app/src/main/java/com/eatssu/android/data/dto/response/MyReviewResponse.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/com/eatssu/android/data/dto/response/MyReviewResponse.kt
Outdated
Show resolved
Hide resolved
…wResponse.kt Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…wResponse.kt Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
| tasteGrade = data.tasteRating ?: 0, | ||
| writeDate = data.writeDate ?: "", | ||
| content = data.content ?: "", | ||
| imgUrl = data.imgUrlList ?: arrayListOf(), |
There was a problem hiding this comment.
이미지를 하나만 올릴 수 있는걸로 알고 있었는데 리스트로 오는군요,,?
확장성을 위한 거였을까요? 궁금하네요..
There was a problem hiding this comment.
리스트로 옵니다! 말씀하신대로 확장성을 위해서 리스트로 리뷰작성/리스트 둘다 리스트로 되어있는데, 운영 단에서 1장 이상일 필요 없다랑 S3 리소스 문제로 1장으로 운영하고 있슴다
Summary
버그 인지 시나리오
저는 분명 사진을 포함해서 리뷰를 작성했습니다.
식단 > 리뷰에서는 보이는 잘 보이던 사진이, 내 리뷰에서는 보이지 않는 오류를 발견하였습니다.
kimumu하는 리뷰가 사진이 있는 리뷰가 맞습니다.
To reviewers
버그 수정만 하였습니다.
클린 어쩌구는 v2에서 이미 바꿔놔서 여기서 또 바꾸면 충돌납니다..