♻️ groupListFlow를 ViewModel에서 관측하고 상태를 저장할 수 있도록 변경

Author: soopeach

presentation/src/main/java/com/whyranoid/presentation/community/CommunityItemFragment.kt

@@ -5,15 +5,12 @@ import android.os.Bundle
 import android.view.View
 import androidx.core.view.isVisible
 import androidx.fragment.app.viewModels
-import androidx.lifecycle.lifecycleScope
 import androidx.navigation.fragment.findNavController
 import com.whyranoid.presentation.R
 import com.whyranoid.presentation.base.BaseFragment
 import com.whyranoid.presentation.databinding.FragmentCommunityItemBinding
-import com.whyranoid.presentation.model.toGroupInfoUiModel
 import com.whyranoid.presentation.util.repeatWhenUiStarted
 import dagger.hilt.android.AndroidEntryPoint
-import kotlinx.coroutines.launch
 
 @AndroidEntryPoint
 internal class CommunityItemFragment :
@@ -54,7 +51,7 @@ internal class CommunityItemFragment :
     }
 
     private fun observeState() {
-        repeatWhenUiStarted {
+        viewLifecycleOwner.repeatWhenUiStarted {
             viewModel.eventFlow.collect { event ->
                 handleEvent(event)
             }
@@ -65,7 +62,7 @@ internal class CommunityItemFragment :
         when (event) {
             is Event.CategoryItemClick -> {
                 val action =
-                    CommunityFragmentDirections.actionCommunityFragmentToGroupDetailFragment(event.groupInfo.toGroupInfoUiModel())
+                    CommunityFragmentDirections.actionCommunityFragmentToGroupDetailFragment(event.groupInfo)
                 findNavController().navigate(action)
             }
         }
@@ -78,8 +75,8 @@ internal class CommunityItemFragment :
         }
         binding.rvCommunity.adapter = myGroupAdapter
 
-        viewLifecycleOwner.lifecycleScope.launch {
-            viewModel.getMyGroupListUseCase().collect { groupList ->
+        viewLifecycleOwner.repeatWhenUiStarted {
+            viewModel.myGroupList.collect { groupList ->
                 removeShimmer()
                 myGroupAdapter.submitList(groupList.sortedBy { it.name })
             }
@@ -88,10 +85,8 @@ internal class CommunityItemFragment :
 
     private fun removeShimmer() {
         binding.shimmerCommunity.apply {
-            if (isShimmerStarted) {
-                stopShimmer()
-                isVisible = false
-            }
+            stopShimmer()
+            isVisible = false
         }
     }
 

presentation/src/main/java/com/whyranoid/presentation/community/CommunityViewModel.kt

@@ -2,25 +2,35 @@ package com.whyranoid.presentation.community
 
 import androidx.lifecycle.ViewModel
 import androidx.lifecycle.viewModelScope
-import com.whyranoid.domain.model.GroupInfo
 import com.whyranoid.domain.usecase.GetMyGroupListUseCase
+import com.whyranoid.presentation.model.GroupInfoUiModel
+import com.whyranoid.presentation.model.toGroupInfoUiModel
 import dagger.hilt.android.lifecycle.HiltViewModel
 import kotlinx.coroutines.flow.MutableSharedFlow
+import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.flow.SharedFlow
+import kotlinx.coroutines.flow.StateFlow
 import kotlinx.coroutines.flow.asSharedFlow
+import kotlinx.coroutines.flow.asStateFlow
+import kotlinx.coroutines.flow.launchIn
+import kotlinx.coroutines.flow.onEach
 import kotlinx.coroutines.launch
 import javax.inject.Inject
 
 @HiltViewModel
 class CommunityViewModel @Inject constructor(
-    val getMyGroupListUseCase: GetMyGroupListUseCase
+    getMyGroupListUseCase: GetMyGroupListUseCase
 ) : ViewModel() {
 
+    private val _myGroupList = MutableStateFlow<List<GroupInfoUiModel>>(emptyList())
+    val myGroupList: StateFlow<List<GroupInfoUiModel>>
+        get() = _myGroupList.asStateFlow()
+
     private val _eventFlow = MutableSharedFlow<Event>()
     val eventFlow: SharedFlow<Event>
         get() = _eventFlow.asSharedFlow()
 
-    fun onCategoryItemClicked(groupInfo: GroupInfo) {
+    fun onCategoryItemClicked(groupInfo: GroupInfoUiModel) {
         emitEvent(Event.CategoryItemClick(groupInfo))
     }
 
@@ -29,4 +39,12 @@ class CommunityViewModel @Inject constructor(
             _eventFlow.emit(event)
         }
     }
+
+    init {
+        getMyGroupListUseCase().onEach { groupInfoList ->
+            _myGroupList.value = groupInfoList.map { groupInfo ->
+                groupInfo.toGroupInfoUiModel()
+            }
+        }.launchIn(viewModelScope)
+    }
 }

presentation/src/main/java/com/whyranoid/presentation/community/Event.kt

@@ -1,7 +1,7 @@
 package com.whyranoid.presentation.community
 
-import com.whyranoid.domain.model.GroupInfo
+import com.whyranoid.presentation.model.GroupInfoUiModel
 
 sealed class Event {
-    data class CategoryItemClick(val groupInfo: GroupInfo) : Event()
+    data class CategoryItemClick(val groupInfo: GroupInfoUiModel) : Event()
 }

presentation/src/main/java/com/whyranoid/presentation/community/MyGroupAdapter.kt

@@ -5,25 +5,31 @@ import android.view.ViewGroup
 import androidx.recyclerview.widget.DiffUtil
 import androidx.recyclerview.widget.ListAdapter
 import androidx.recyclerview.widget.RecyclerView
-import com.whyranoid.domain.model.GroupInfo
 import com.whyranoid.presentation.databinding.MyGroupItemBinding
+import com.whyranoid.presentation.model.GroupInfoUiModel
 
-class MyGroupAdapter(private val onClickListener: (GroupInfo) -> Unit) :
-    ListAdapter<GroupInfo, MyGroupAdapter.MyGroupViewHolder>(diffUtil) {
+class MyGroupAdapter(private val onClickListener: (GroupInfoUiModel) -> Unit) :
+    ListAdapter<GroupInfoUiModel, MyGroupAdapter.MyGroupViewHolder>(diffUtil) {
 
     companion object {
-        val diffUtil = object : DiffUtil.ItemCallback<GroupInfo>() {
-            override fun areItemsTheSame(oldItem: GroupInfo, newItem: GroupInfo): Boolean =
+        val diffUtil = object : DiffUtil.ItemCallback<GroupInfoUiModel>() {
+            override fun areItemsTheSame(
+                oldItem: GroupInfoUiModel,
+                newItem: GroupInfoUiModel
+            ): Boolean =
                 oldItem.groupId == newItem.groupId
 
-            override fun areContentsTheSame(oldItem: GroupInfo, newItem: GroupInfo): Boolean =
+            override fun areContentsTheSame(
+                oldItem: GroupInfoUiModel,
+                newItem: GroupInfoUiModel
+            ): Boolean =
                 oldItem == newItem
         }
     }
 
     class MyGroupViewHolder(private val binding: MyGroupItemBinding) :
         RecyclerView.ViewHolder(binding.root) {
-        fun bind(groupInfo: GroupInfo) {
+        fun bind(groupInfo: GroupInfoUiModel) {
             binding.groupInfo = groupInfo
         }
     }

presentation/src/main/res/layout/my_group_item.xml

@@ -7,7 +7,7 @@
 
         <variable
             name="groupInfo"
-            type="com.whyranoid.domain.model.GroupInfo" />
+            type="com.whyranoid.presentation.model.GroupInfoUiModel" />
     </data>
 
     <androidx.constraintlayout.widget.ConstraintLayout