Rhedi/week9#93
Open
hajinmn-stack wants to merge 7 commits into
Open
Conversation
hw4nx02
reviewed
Jun 2, 2026
There was a problem hiding this comment.
9주차 피드백
안녕하세요 레디🔥! 축제 지나고 기말 시즌이 왔네요... 기말도 화이팅이고 시험 끝나고 봅시다!
총평
API 연동을 잘 수행해주신 것 같아요. 현재 UI가 요구사항을 모두 만족하지는 못한 상황 같습니다. 시험 끝나고 여유가 생기면 해당 UI를 마저 구현해보시는 것을 추천드립니다.
이전 주차 PR들에 대해 merge 대상을 UMC-10th:main이 아닌, UMC-10th:Rhedi/main으로 변경해주세요
리뷰
Good!
1. 개념 이해도
PR의 알게 된 사항으로 정리해주신 학습 내용을 통해 Compose Side Effect API(LaunchedEffect, DisposableEffect, rememberCoroutineScope 등)의 역할과 차이점을 명확히 인지하고 있으며, Coil 이미지 로딩 라이브러리의 비동기 처리 및 캐싱 전략에 대한 이해도를 확인할 수 있었습니다!
To Improve
1. 컴포저블 내부 변수 선언
컴포저블은 상태가 바뀔 때 다시 실행됩니다. 때문에 ProfileScreen의 팔로워 리스트 구현부에서 선언된 pagerState 변수 등은 컴포저블이 상태가 바뀔 때 계속해서 초기화되어 원하는 결과를 내지 못하는 원인이 될 수 있습니다. 또한, 데이터 등을 상위 컴포저블로 끌어올리고, 하위 컴포저블은 오로지 UI만 그리도록 분리하여 유지 보수성 및 확장성 측면에서 이점을 가져갈 수도 있으니 참고하시면 좋을 것 같습니다.
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 제목
#️⃣ 연관된 이슈
closes #(issue_num)
✅ 변경 사항
📷 영상 및 스크린샷
week9.webm
🔗 알게 된 사항
Pager & TabRow 연동 UI: HorizontalPager와 VerticalPager는 상태(PagerState)를 통해 스와이프 UI를 제어하며, 이를 TabRow와 결합하면 탭 클릭과 화면 스크롤이 양방향으로 동기화될수있다. TabRow <-> HorizontalPager
Coil 이미지 로딩: 코루틴 기반으로 설계되어 Jetpack Compose와 공식 호환되는 라이브러리로, AsyncImage 컴포저블을 통해 네트워크 이미지의 비동기 로딩, 예외 처리(placeholder/error), 크기 조정(ContentScale) 및 캐싱 전략을 효율적으로 처리합니다...
📝 질문 사항