Skip to content

[Mission] 요시 week9 완료#89

Open
LLUPINUS wants to merge 1 commit into
Yoshi/mainfrom
Yoshi/week9
Open

[Mission] 요시 week9 완료#89
LLUPINUS wants to merge 1 commit into
Yoshi/mainfrom
Yoshi/week9

Conversation

@LLUPINUS

Copy link
Copy Markdown

📌 PR 제목

요시 week9 완료

#️⃣ 연관된 이슈

closes #88


✅ 변경 사항

이번 PR에서 변경된 내용을 간략히 정리해주세요.

  • 마이페이지 UI 전부 구현하기
  • 서버에서 데이터를 받아와서 마이페이지 연결하기
  • 마이페이지 팔로잉 리스트 HorizontalPager로 구현하기

📷 영상 및 스크린샷

image

🔗 알게 된 사항

  • Compose의 side effect를 알 수 있었음
  • 스와이프 UI의 사용법에 대해 알 수 있었음
  • TabRow + HorizontalPager 연동법을 알 수 있었음

📝 질문 사항

워크북과 과제를 하며 궁금했던 점을 작성해주세요

@LLUPINUS LLUPINUS requested a review from hw4nx02 May 31, 2026 06:13
@LLUPINUS LLUPINUS self-assigned this May 31, 2026
@LLUPINUS LLUPINUS requested a review from a team May 31, 2026 06:13
@LLUPINUS LLUPINUS added the 🚀Week 9 9주차 워크북 미션 label May 31, 2026

@hw4nx02 hw4nx02 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

9주차 피드백

안녕하세요 요시! 축제 지나고 기말 시즌이 왔네요... 기말도 화이팅이고 시험 끝나고 봅시다!

총평

전반적으로 UI 요구사항을 맞춰 잘 구현해주셨습니다. 특히 로딩 상태, 네트워크 에러 상태를 사용자에게 표시하는 것까지 고려한 부분이 눈에 띄네요!

리뷰

Good!

1. 깔끔한 데이터 레이어 설계 및 DTO-도메인 모델 분리

UserRepository에서 UserDto.toUser() 변환 함수를 사용하셨습니다. 이를 통해 네트워크 응답 모델인 UserDto와 화면에서 사용하는 도메인 모델인 User를 명확히 분리하고, UserRepository를 통해 비즈니스 로직을 캡슐화한 점이 매우 좋습니다. 이를 통해 UI 레이어는 데이터 소스의 구체적인 구현(Retrofit 등)에 의존하지 않고 안정적으로 데이터를 소비할 수 있게 된 것 같네요!

2. API 키 관리 설정

val reqresApiKey = localProperties.getProperty("NIKE_API_KEY") ?: "" 및 buildConfigField를 설정하셨습니다. 이를 통해 API 키를 소스 코드에 하드코딩하지 않고 민감한 정보가 GitHub 등의 원격 저장소에 노출되는 것을 방지할 수 있을 것 같습니다.

To Improve

1. Gradle 버전 카탈로그 관리

현재 build.gradle.kts에 Coil, Retrofit, OkHttp 등의 라이브러리가 직접 문자열 버전으로 추가되어 있습니다. 프로젝트의 일관된 종속성 관리를 위해 Gradle의 Version Catalog(libs.versions.toml)를 활용하는 것이 좋습니다.
libs.versions.toml 파일에 정의된 coil-compose, retrofit 등의 라이브러리 참조를 build.gradle.kts에서 libs.coil.compose, libs.retrofit 형태로 일관되게 적용해보세요!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚀Week 9 9주차 워크북 미션

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants