[Refactor] 안쓰는 리소스 삭제 #445
Conversation
Summary of ChangesHello @PeraSite, 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은 레거시 코드 정리의 일환으로, 현재 애플리케이션에서 사용되지 않는 다양한 리소스 파일을 제거하여 코드베이스를 경량화하고 정리하는 것을 목표로 합니다. 이는 주로 Android Studio의 리소스 정리 기능을 통해 이루어졌으며, 향후 개발 및 유지보수 효율성을 높이는 데 기여할 것입니다. 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은 사용되지 않는 안드로이드 리소스를 대거 삭제하여 프로젝트를 정리하는 것을 목표로 하고 있습니다. 자동화된 도구를 사용하여 리소스를 정리한 것으로 보이며, 이로 인해 몇 가지 잠재적인 문제가 발생할 수 있습니다. 주요 피드백은 다음과 같습니다:
strings.xml에서 삭제된 많은 문자열들이 코드 내에 하드코딩되어 있습니다. 이는 다국어 지원 및 유지보수를 어렵게 하므로, 다시 리소스로 추출해야 합니다.provider_paths.xml파일이 삭제되었는데, 이 파일은 카메라로 사진을 찍는 등 파일 공유 기능에 필수적일 수 있습니다. 리뷰에 사진을 첨부하는 기능이 있다면, 이 파일의 삭제는 앱의 오작동을 유발할 수 있으므로 확인이 필요합니다.
전반적으로 리소스 정리는 좋은 방향이지만, 자동화 도구의 결과를 맹신하기보다는 실제 코드에서의 사용 여부를 면밀히 검토해야 합니다.
I am having trouble creating individual review comments. Click here to see my feedback.
app/src/main/res/values/strings.xml (69-84)
많은 문자열 리소스가 삭제되었지만, 코드에서 하드코딩된 형태로 여전히 사용되고 있습니다. 이는 유지보수성을 저해하고 향후 다국어 지원을 어렵게 만듭니다. 자동화된 리소스 정리 도구가 Compose 코드 내의 사용을 감지하지 못한 것으로 보입니다. 삭제된 문자열들을 다시 strings.xml에 추가하고 코드에서는 stringResource()를 사용하여 참조하도록 수정해야 합니다.
다음은 하드코딩된 문자열의 몇 가지 예시입니다:
ReviewListScreen.kt:"리뷰","리뷰 작성하기","오늘의 메뉴","아직 작성된 리뷰가 없어요","메뉴에 가장 먼저 리뷰를 남겨주세요!"
ModifyReviewScreen.kt:"리뷰 수정하기","화면을 준비하는 중입니다.","완료하기","오늘의 식사는 어땠나요?","추천하고 싶은 메뉴가 있나요?","메뉴에 대한 상세한 리뷰를 작성해주세요"
ModifyViewModel.kt&ReviewListViewModel.kt:"리뷰 수정이 실패했습니다.","리뷰를 수정했습니다.","리뷰를 불러오지 못했습니다.","리뷰 삭제에 실패했습니다.","리뷰를 삭제했습니다."
이 외에도 여러 곳에서 하드코딩된 문자열이 발견되었습니다. 전체적으로 검토가 필요합니다.
app/src/main/res/xml/provider_paths.xml (1-6)
provider_paths.xml 파일이 삭제되었습니다. 이 파일은 FileProvider를 통해 앱 외부(예: 카메라 앱)와 파일을 공유하는 기능에 필수적입니다. 앱에 리뷰 사진을 촬영하여 첨부하는 기능이 있다면, 이 파일의 삭제로 인해 해당 기능이 오작동하거나 앱이 충돌할 수 있습니다. ic_baseline_camera_alt_24.xml 아이콘이 함께 삭제된 것으로 보아 관련 기능이 제거되었을 수 있지만, 만약 기능이 여전히 존재한다면 문제가 될 수 있습니다. Android Studio의 '사용하지 않는 리소스 제거' 기능은 AndroidManifest.xml의 <meta-data> 태그를 통해 참조되는 경우에도 이 파일을 실수로 제거할 수 있습니다. FileProvider 사용 여부를 확인하고, 필요하다면 이 파일을 복원하는 것을 권장합니다.
Summary
레거시로 더 이상 사용하지 않는 리소스(이미지, 레이아웃, 문자열 등) 를 삭제합니다.
다국어 지원을 위해 문자열 XML를 정리하다 한번에 정리해주는 기능이 안드로이드 스튜디오에 있어서 돌렸습니다!
Describe your changes
Issue
To reviewers
삭제된 이미지들 중 사용하는 것을 몇 번 본 것 같은건 남겨두었으나, 당장만 안쓰는건지 나중에 필요할 때가 생기는건지 검토 부탁드립니다!