Conversation
hw4nx02
approved these changes
Jun 2, 2026
hw4nx02
left a comment
There was a problem hiding this comment.
9주차 피드백
안녕하세요 오리🦆! 축제 지나고 기말 시즌이 왔네요... 기말도 화이팅이고 시험 끝나고 봅시다!
총평
전반적으로 마이페이지 UI 구현과 함께 Retrofit을 활용한 API 연동, ViewModel을 통한 상태 관리를 깔끔하게 구조화해주셨습니다. 특히 비동기 데이터 로딩을 ViewModel의 init 블록에서 처리하여 Composable 생명주기 관련 부작용을 최소화한 점이 돋보입니다.
리뷰
Good!
1. ViewModel을 통한 비동기 데이터 로딩 처리
ProfileViewModel에서 init { getUsers() }를 사용함으로써 API 호출과 같은 비동기 작업을 Composable 본문이나 LaunchedEffect(Unit) 대신 ViewModel의 init 블록에서 실행하여, Recomposition 발생 시 불필요하게 API가 재호출되는 부작용을 근본적으로 방지해주신 것이 눈에 띕니다.
To Improve
1. HorizontalPager의 사용
현재 LazyRow로 팔로잉 리스트를 잘 구현해주셨으나, 본 미션에서는 HorizontalPager를 사용하는 것이 조건이었습니다. 이를 활용해서 다시 한 번 미션 내용을 구현해보시면 좋을 것 같습니다.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📌 PR 제목
9주차 마이페이지 구현 및 연동 실습입니다.
#️⃣ 연관된 이슈
closes #87
✅ 변경 사항
📷 영상 및 스크린샷
ori_week9.mp4
🔗 알게 된 사항
서버 이미지 연동 시
AsyncImage사용 ,,📝 질문 사항