[WEEK7] 7차 필수과제 구현 #14

HAJIEUN02 · 2024-12-11T08:43:07Z

Related issue 🛠

closed [WEEK7] 7주차 필수과제 구현 #13

Work Description ✏️

MVI 패턴 적용 및 수정

Screenshot 📸

XRecorder_05112024_115538.mp4

4차/6차 과제와 같은 영상입니다!

Uncompleted Tasks 😅

N/A

To Reviewers 📢

안드로이드 초기 데이터 로드 해당 아티클 및 합동세미나를 거쳐 오면서 초기 데이터 로드에 LaunchedEffect(Unit)를 사용하던 것을 수정했는데요! MyViewModel을 보시면 다음과 같이 flow를 이용해서 emit하고, 이를 관찰하던 MyRoute에서 해당 state가 Success인 경우 MyScreen을 호출하는 것으로 구현을 해두었습니다.

이번에 Home 뷰도 My 뷰처럼 flow를 이용해서 수정하고 싶었는데 어떻게 수정해야 할지 감이 잘 안 오더라구요ㅜ.ㅜ 일단 LaunchedEffect(homeUiState)를 이용해서 Idle인 경우 뷰모델의 setHomeImgList(실제 앱이라면 서버통신 하는 함수)를 호출하는 것으로 구현을 해두었습니다. flow를 이용해 좀더 깔끔하고 좋은 방향으로 개선할 방법이 있을까요? 조언해주시면 감사하겠습니다!

@HiltViewModel
class MyViewModel @Inject constructor(
    private val patchUserHobbyUseCase: PatchUserHobbyUseCase
) : ViewModel() {
    val myUiState: StateFlow<MyUiState> =
        flow<MyUiState> {
            runCatching {
                patchUserHobbyUseCase()
            }.onSuccess { data ->
                emit(MyUiState.Success(data.hobby))
            }.onFailure { throwable ->
                emit(MyUiState.Error(throwable.message))
            }
        }.catch { throwable ->
            emit(MyUiState.Error(throwable.message))
        }.stateIn(
            scope = viewModelScope,
            started = SharingStarted.WhileSubscribed(5_000),
            initialValue = MyUiState.Loading
        )
}

@Composable
fun MyRoute(
    paddingValues: PaddingValues,
    myViewModel: MyViewModel = hiltViewModel()
) {
    val uiState by myViewModel.myUiState.collectAsStateWithLifecycle()

    when(val myUiState: MyUiState = uiState) {
        is MyUiState.Success -> {
            MyScreen(
                modifier = Modifier.padding(paddingValues),
                userHobby = myUiState.hobby
            )
        }
        else -> Unit
    }
}

1971123-seongmin

항상 보고 공부하고 배울점이 정말 많은 것 같아요 😚😚
과제하느라 고생하셨습니다.

1971123-seongmin · 2024-12-18T06:44:42Z

app/src/main/java/org/sopt/and/presentation/ui/home/HomeScreen.kt

    editorRecommendedImgList: List<String>,
    todayTopRankingImgList: List<String>,
-    modifier: Modifier = Modifier
+    modifier: Modifier = Modifier,
+    pagerState: PagerState = rememberPagerState(pageCount = { 0 }),


HomeScreen에서 rememberPagerState는 어떻게 사용되는건가요??

1971123-seongmin · 2024-12-18T06:45:31Z

app/src/main/java/org/sopt/and/presentation/ui/home/HomeRoute.kt


    LaunchedEffect(Unit) {
       homeViewModel.setHomeImgList()
    }

+    LaunchedEffect(pagerState) {
+        snapshotFlow { pagerState.currentPage }.collect { page ->


snapshotFlow는 혹시 사용하신 이유가 있나요??

1971123-seongmin · 2024-12-18T06:48:45Z

app/src/main/java/org/sopt/and/presentation/ui/auth/login/LoginViewModel.kt

                }.onFailure {
                    Timber.d("[로그인] 실패 -> $it")
-                    setEffect(LoginContract.LoginEffect.ShowFailSnackBar(failMessage = event.failMessage))
-                    setState(currentUiState.copy(loginStatus = LoginContract.LoginStatus.Fail))
+                    setEffect{LoginContract.LoginEffect.ShowFailSnackBar(failMessage = event.failMessage)}


저는 바로 Toast 메시지를 SideEffect로 보여줬는데 이렇게 하는게 더 맞는 것 같아요. 다음에는 할 때 적용해 보게습니다.

1971123-seongmin · 2024-12-18T06:50:52Z

app/src/main/java/org/sopt/and/presentation/ui/home/HomeContract.kt

+        data class Error(val message: String? = null) : HomeUiState()
+
+        data object Idle : HomeUiState()
+    }


UiState 구성을 단순히 데이터 말고 상태들을 넣어서 구현하셨네요. 이렇게 하면 Loading, Error에 맞게 더 유연하게 코드를 쓸수 있을 것 같아요. 짱 ☺️

인정합니다 저도 이부분은 항상 제너릭 타입으로 가져갔던 것 같은데 이런 방법으로도 다양하게 접근해볼수있구나 느끼고 갑니다 ...ㅠㅠ 짱짱

1971123-seongmin · 2024-12-18T06:52:37Z

app/src/main/java/org/sopt/and/presentation/ui/home/HomeRoute.kt

-    LaunchedEffect(pagerState) {
-        snapshotFlow { pagerState.currentPage }.collect { page ->
-            homeViewModel.setCurrentBannerPage(page)
+    when (homeUiState) {


이 부분은 LaunchedEffect에 안들어가 있는 건가요??

greedy0110 · 2024-12-18T10:15:30Z

app/src/main/java/org/sopt/and/presentation/ui/home/HomeRoute.kt

+            )
+        }
+
+        else -> Unit


Loading, Error, Idle 인 상황에도 화면을 그려주는 게 좋을 것 같습니다.
아무런 화면도 안나오는 안드로이드 앱을 상상해보세요. 사용자는 앱을 사용하지 못하므로 버그로 보일거에요.

greedy0110 · 2024-12-18T10:16:32Z

app/src/main/java/org/sopt/and/presentation/ui/home/HomeContract.kt

-        val editorRecommendedList: List<String> = immutableListOf(),
-        val todayTopRankingList: List<String> = immutableListOf(),
-        val homeStatus: HomeStatus = HomeStatus.Loading
+        val homeInitialState: HomeUiState = HomeUiState.Idle,


이 값은 계속 변화하는데요. initialState 라는 변수명이 맞는지 고민해볼 필요가 있어보입니다.

greedy0110 · 2024-12-18T10:17:24Z

app/src/main/java/org/sopt/and/presentation/ui/home/HomeRoute.kt

+    LaunchedEffect(homeUiState) {
+        if(homeState.homeInitialState == HomeContract.HomeUiState.Idle) {
+            homeViewModel.setHomeImgList()
+        }


LaunchedEffect(Unit) 에서 호출하는 것으로도 충분했던 것 같은데요.
homeInitialState를 비교해서 호출하도록 바꾸신 이유가 무엇일까요?

greedy0110 · 2024-12-18T10:20:43Z

app/src/main/java/org/sopt/and/presentation/ui/home/HomeViewModel.kt

+        setState {
+            copy(
+                homeInitialState = HomeContract.HomeUiState.Success(
+                    bannerImgList = mockBannerItem,
+                    editorRecommendedList = mockEditorRecommendedItem,
+                    todayTopRankingList = mockTodayTopRankingItem
+                )


제 생각에 Loading, Success, Error 케이스를 구분하셨으니 아래처럼 접근해보고 테스트해보시면 좋을 것 같습니다.

viewModelScope.launch { setLoading() // loading 중으로 state를 갱신합니다. delay(2000) // 임의로 로딩 중임을 표시해봅니다. try { val data = someFunction() setSuccess(data) // success로 갱신합니다. } catch (..) { setError(..) // error 로 갱신합니다. } }

someFunction 함수가 성공하도록, 그리고 실패하도록 수정해보면서 화면이 어떻게 보이는지 테스트해보시면 좋을 듯 합니다. 어색한 부분은 없는지 등을 체크해보셔요.

hyeeum

고생하셨어요..이거 마지막 코리인거죠? (눈물) 그동안 함께 과제해서 너무 감사했어요
지은언니 코드들을 보면서 참 꼼꼼하다고 생각했었습니다
언니가 to reviewer에서 언급한 내용을 생각하고있는데 마땅한 방법이 생각나면 바로 리뷰 달러올게요!!!

hyeeum · 2024-12-19T14:38:09Z

app/src/main/java/org/sopt/and/presentation/ui/home/HomeContract.kt

+        data class Error(val message: String? = null) : HomeUiState()
+
+        data object Idle : HomeUiState()
+    }


인정합니다 저도 이부분은 항상 제너릭 타입으로 가져갔던 것 같은데 이런 방법으로도 다양하게 접근해볼수있구나 느끼고 갑니다 ...ㅠㅠ 짱짱

jihyunniiii

고생하셨습니다 ~

jihyunniiii · 2024-12-25T04:49:11Z

app/src/main/java/org/sopt/and/presentation/ui/auth/login/LoginViewModel.kt

            }

            is LoginContract.LoginEvent.PasswordVisibilityChanged -> {
-                setState(currentUiState.copy(showPassword = !currentUiState.showPassword))
+                setState{ copy(showPassword = !currentUiState.showPassword) }


얘도 인자로 받아와도 좋을 것 같아요. handleEvent 함수 내에서는 단순히 state를 변경하는 작업만 담당할 수 있게요.

[mod] #13 PagerState 로직 수정

2154455

HAJIEUN02 added the 💚필수과제💚 label Dec 11, 2024

HAJIEUN02 self-assigned this Dec 11, 2024

HAJIEUN02 linked an issue Dec 11, 2024 that may be closed by this pull request

[WEEK7] 7주차 필수과제 구현 #13

Open

1 task

HAJIEUN02 added 2 commits December 13, 2024 15:42

[mod] #13 BaseViewModel 수정

ea9eda5

[mod] #13 HomeScreen에 loadState 추가

5914346

HAJIEUN02 requested review from hyeeum, kamja0510, 1971123-seongmin and jihyunniiii and removed request for hyeeum and kamja0510 December 13, 2024 06:58

[del] #13 불필요한 state 삭제

2bae0d9

1971123-seongmin approved these changes Dec 18, 2024

View reviewed changes

greedy0110 reviewed Dec 18, 2024

View reviewed changes

hyeeum approved these changes Dec 19, 2024

View reviewed changes

jihyunniiii reviewed Dec 25, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[WEEK7] 7차 필수과제 구현 #14

[WEEK7] 7차 필수과제 구현 #14

HAJIEUN02 commented Dec 11, 2024 •

edited

Loading

1971123-seongmin left a comment

1971123-seongmin Dec 18, 2024

1971123-seongmin Dec 18, 2024

1971123-seongmin Dec 18, 2024

1971123-seongmin Dec 18, 2024

hyeeum Dec 19, 2024

1971123-seongmin Dec 18, 2024

greedy0110 Dec 18, 2024

greedy0110 Dec 18, 2024

greedy0110 Dec 18, 2024

greedy0110 Dec 18, 2024

hyeeum left a comment

hyeeum Dec 19, 2024

jihyunniiii left a comment

jihyunniiii Dec 25, 2024

[WEEK7] 7차 필수과제 구현 #14

Are you sure you want to change the base?

[WEEK7] 7차 필수과제 구현 #14

Conversation

HAJIEUN02 commented Dec 11, 2024 • edited Loading

Related issue 🛠

Work Description ✏️

Screenshot 📸

Uncompleted Tasks 😅

To Reviewers 📢

1971123-seongmin left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hyeeum left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jihyunniiii left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

HAJIEUN02 commented Dec 11, 2024 •

edited

Loading