Merge pull request #75 from YAPP-Github/feature/moneymong-234-소속-찾기-가입된-소속-경고-효과

Feature/moneymong 234 소속 찾기 가입된 소속 경고 효과

Author: jhg3410

data/src/main/java/com/moneymong/moneymong/data/mapper/agency/AgenciesGetMapper.kt

@@ -12,8 +12,7 @@ fun AgencyGetResponse.toEntity() = AgencyGetEntity(
     type = when (type) {
         "IN_SCHOOL_CLUB" -> AgencyGetEntity.AgencyType.CLUB
         "STUDENT_COUNCIL" -> AgencyGetEntity.AgencyType.COUNCIL
-//        else -> throw IllegalArgumentException("Unknown type: $type")
-        else -> AgencyGetEntity.AgencyType.CLUB
+        else -> throw IllegalArgumentException("Unknown type: $type")
     }
 )
 

data/src/main/java/com/moneymong/moneymong/data/pagingsource/AgencyPagingSource.kt

@@ -18,13 +18,13 @@ class AgencyPagingSource(
 
     override suspend fun load(params: LoadParams<Int>): LoadResult<Int, AgencyGetResponse> {
         val page = params.key ?: START_PAGE
-        val size = params.loadSize
-        return dataSource.getAgencies(page = page, size = size).fold(
+        val loadSize = params.loadSize
+        return dataSource.getAgencies(page = page, size = loadSize).fold(
             onSuccess = {
                 LoadResult.Page(
                     data = it.agencies,
                     prevKey = null,
-                    nextKey = if (it.agencies.isEmpty()) null else page + 1
+                    nextKey = if (it.agencies.size < loadSize) null else page + 1
                 )
             },
             onFailure = {

feature/agency/src/main/java/com/moneymong/moneymong/feature/agency/navigation/AgencyRegisterNavigation.kt

@@ -1,8 +1,5 @@
 package com.moneymong.moneymong.feature.agency.navigation
 
-import androidx.compose.foundation.layout.PaddingValues
-import androidx.compose.foundation.layout.padding
-import androidx.compose.ui.Modifier
 import androidx.navigation.NavController
 import androidx.navigation.NavGraphBuilder
 import androidx.navigation.NavOptions

feature/agency/src/main/java/com/moneymong/moneymong/feature/agency/register/AgencyRegisterState.kt

@@ -2,7 +2,7 @@ package com.moneymong.moneymong.feature.agency.register
 
 import androidx.compose.ui.text.input.TextFieldValue
 import com.moneymong.moneymong.common.base.State
-import com.moneymong.moneymong.feature.agency.AgencyType
+import com.moneymong.moneymong.feature.agency.search.AgencyType
 
 data class AgencyRegisterState(
     val agencyName: TextFieldValue = TextFieldValue(),

feature/agency/src/main/java/com/moneymong/moneymong/feature/agency/register/AgencyRegisterViewModel.kt

@@ -6,7 +6,7 @@ import com.moneymong.moneymong.common.error.MoneyMongError
 import com.moneymong.moneymong.domain.param.agency.AgencyRegisterParam
 import com.moneymong.moneymong.domain.usecase.agency.RegisterAgencyUseCase
 import com.moneymong.moneymong.domain.usecase.agency.SaveAgencyIdUseCase
-import com.moneymong.moneymong.feature.agency.AgencyType
+import com.moneymong.moneymong.feature.agency.search.AgencyType
 import dagger.hilt.android.lifecycle.HiltViewModel
 import org.orbitmvi.orbit.annotation.OrbitExperimental
 import org.orbitmvi.orbit.syntax.simple.blockingIntent

feature/agency/src/main/java/com/moneymong/moneymong/feature/agency/register/complete/AgencyRegisterCompleteScreen.kt

@@ -1,5 +1,6 @@
 package com.moneymong.moneymong.feature.agency.register.complete
 
+import androidx.activity.compose.BackHandler
 import androidx.compose.foundation.Image
 import androidx.compose.foundation.background
 import androidx.compose.foundation.clickable
@@ -49,6 +50,8 @@ fun AgencyRegisterCompleteScreen(
         }
     }
 
+    BackHandler { viewModel.navigateToAgencySearch() }
+
     Column(
         modifier = modifier
             .fillMaxSize()

feature/agency/src/main/java/com/moneymong/moneymong/feature/agency/register/view/AgencyRegisterContentView.kt

@@ -28,7 +28,7 @@ import com.moneymong.moneymong.design_system.theme.Body2
 import com.moneymong.moneymong.design_system.theme.Gray06
 import com.moneymong.moneymong.design_system.theme.Gray10
 import com.moneymong.moneymong.design_system.theme.Heading2
-import com.moneymong.moneymong.feature.agency.AgencyType
+import com.moneymong.moneymong.feature.agency.search.AgencyType
 
 @Composable
 internal fun AgencyResisterContentView(

…ymong/moneymong/feature/agency/Agency.kt …oneymong/feature/agency/search/Agency.ktfeature/agency/src/main/java/com/moneymong/moneymong/feature/agency/Agency.kt renamed to feature/agency/src/main/java/com/moneymong/moneymong/feature/agency/search/Agency.kt feature/agency/src/main/java/com/moneymong/moneymong/feature/agency/Agency.kt renamed to feature/agency/src/main/java/com/moneymong/moneymong/feature/agency/search/Agency.kt

@@ -1,6 +1,7 @@
-package com.moneymong.moneymong.feature.agency
+package com.moneymong.moneymong.feature.agency.search
 
 import com.moneymong.moneymong.domain.entity.agency.AgencyGetEntity
+import com.moneymong.moneymong.domain.entity.agency.MyAgencyEntity
 import com.moneymong.moneymong.domain.param.agency.AgencyRegisterParam
 
 data class Agency(
@@ -22,6 +23,19 @@ fun AgencyGetEntity.toAgency(): Agency {
     )
 }
 
+fun MyAgencyEntity.toAgency(): Agency {
+    return Agency(
+        id = this.id.toLong(),
+        type = when (this.type) {
+            "IN_SCHOOL_CLUB" -> AgencyType.CLUB
+            "STUDENT_COUNCIL" -> AgencyType.COUNCIL
+            else -> throw IllegalArgumentException("Unknown type: $type")
+        },
+        name = this.name,
+        memberCount = this.headCount
+    )
+}
+
 enum class AgencyType(val text: String) {
     CLUB(text = "동아리"),
     COUNCIL(text = "학생회");

feature/agency/src/main/java/com/moneymong/moneymong/feature/agency/search/AgencySearchScreen.kt

@@ -15,6 +15,7 @@ import androidx.compose.foundation.layout.size
 import androidx.compose.foundation.lazy.LazyColumn
 import androidx.compose.material3.Text
 import androidx.compose.runtime.Composable
+import androidx.compose.runtime.getValue
 import androidx.compose.ui.Alignment
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.res.painterResource
@@ -25,10 +26,12 @@ import androidx.paging.LoadState
 import androidx.paging.compose.LazyPagingItems
 import androidx.paging.compose.collectAsLazyPagingItems
 import androidx.paging.compose.itemKey
+import com.moneymong.moneymong.common.error.MoneyMongError
 import com.moneymong.moneymong.design_system.R
 import com.moneymong.moneymong.design_system.component.button.MDSFloatingActionButton
 import com.moneymong.moneymong.design_system.component.tooltip.MDSToolTip
 import com.moneymong.moneymong.design_system.component.tooltip.MDSToolTipPosition
+import com.moneymong.moneymong.design_system.error.ErrorDialog
 import com.moneymong.moneymong.design_system.error.ErrorItem
 import com.moneymong.moneymong.design_system.error.ErrorScreen
 import com.moneymong.moneymong.design_system.loading.LoadingItem
@@ -38,9 +41,10 @@ import com.moneymong.moneymong.design_system.theme.Gray01
 import com.moneymong.moneymong.design_system.theme.Gray08
 import com.moneymong.moneymong.design_system.theme.MMHorizontalSpacing
 import com.moneymong.moneymong.design_system.theme.Red03
-import com.moneymong.moneymong.feature.agency.Agency
 import com.moneymong.moneymong.feature.agency.search.component.AgencySearchTopBar
 import com.moneymong.moneymong.feature.agency.search.item.AgencyItem
+import org.orbitmvi.orbit.compose.collectAsState
+import org.orbitmvi.orbit.compose.collectSideEffect
 
 @Composable
 fun AgencySearchScreen(
@@ -49,8 +53,27 @@ fun AgencySearchScreen(
     navigateToRegister: () -> Unit,
     navigateAgencyJoin: (agencyId: Long) -> Unit
 ) {
+    val state by viewModel.collectAsState()
     val pagingItems = viewModel.agencies.collectAsLazyPagingItems()
 
+    viewModel.collectSideEffect {
+        when (it) {
+            is AgencySearchSideEffect.NavigateToRegister -> {
+                navigateToRegister()
+            }
+
+            is AgencySearchSideEffect.NavigateToAgencyJoin -> {
+                navigateAgencyJoin(it.agencyId)
+            }
+        }
+    }
+
+    if (state.visibleWarningDialog) {
+        ErrorDialog(message = "이미 가입된 소속입니다") {
+            viewModel.changeVisibleWarningDialog(false)
+        }
+    }
+
     Box(
         modifier = modifier
             .fillMaxSize()
@@ -65,7 +88,17 @@ fun AgencySearchScreen(
             AgencySearchContentView(
                 modifier = Modifier.weight(1f),
                 pagingItems = pagingItems,
-                onClickItem = navigateAgencyJoin
+                onClickItem = { agencyId ->
+                    if (agencyId in state.joinedAgenciesIds) {
+                        viewModel.changeVisibleWarningDialog(true)
+                    } else {
+                        viewModel.navigateToJoin(agencyId)
+                    }
+                },
+                isLoading = state.isLoading,
+                isError = state.isError,
+                errorMessage = state.errorMessage,
+                fetchMyAgencyList = viewModel::fetchMyAgencyList
             )
         }
         Column(
@@ -82,7 +115,7 @@ fun AgencySearchScreen(
                 Spacer(modifier = Modifier.height(8.dp))
             }
             MDSFloatingActionButton(
-                onClick = navigateToRegister,
+                onClick = viewModel::navigateToRegister,
                 iconResource = R.drawable.ic_plus_default,
                 containerColor = Red03
             )
@@ -94,19 +127,43 @@ fun AgencySearchScreen(
 private fun AgencySearchContentView(
     modifier: Modifier = Modifier,
     pagingItems: LazyPagingItems<Agency>,
-    onClickItem: (agencyId: Long) -> Unit
+    onClickItem: (agencyId: Long) -> Unit,
+    isLoading: Boolean,
+    isError: Boolean,
+    errorMessage: String,
+    fetchMyAgencyList: () -> Unit
 ) {
-    if (pagingItems.itemCount == 0) {
-        ContentViewWithoutAgencies(
-            modifier = modifier,
-            pagingItems = pagingItems
+    val contentLoading = pagingItems.loadState.refresh is LoadState.Loading || isLoading
+    val contentError = pagingItems.loadState.refresh is LoadState.Error || isError
+    val contentErrorMessage = errorMessage.ifEmpty {
+        (pagingItems.loadState.refresh as? LoadState.Error)?.error?.message
+            ?: MoneyMongError.UnExpectedError.message
+    }
+
+    if (contentLoading) {
+        LoadingScreen(modifier = modifier.fillMaxSize())
+    } else if (contentError) {
+        ErrorScreen(
+            modifier = modifier.fillMaxSize(),
+            message = contentErrorMessage,
+            onRetry = {
+                pagingItems.retry()
+                fetchMyAgencyList()
+            },
         )
     } else {
-        ContentViewWithAgencies(
-            modifier = modifier,
-            pagingItems = pagingItems,
-            onClickItem = onClickItem
-        )
+        if (pagingItems.itemCount == 0) {
+            ContentViewWithoutAgencies(
+                modifier = modifier,
+                pagingItems = pagingItems
+            )
+        } else {
+            ContentViewWithAgencies(
+                modifier = modifier,
+                pagingItems = pagingItems,
+                onClickItem = onClickItem
+            )
+        }
     }
 }
 
@@ -151,7 +208,7 @@ private fun ContentViewWithAgencies(
                 }
             }
 
-            else -> Unit
+            is LoadState.NotLoading -> Unit
         }
     }
 }

feature/agency/src/main/java/com/moneymong/moneymong/feature/agency/search/AgencySearchSideEffect.kt

@@ -0,0 +1,8 @@
+package com.moneymong.moneymong.feature.agency.search
+
+import com.moneymong.moneymong.common.base.SideEffect
+
+sealed interface AgencySearchSideEffect : SideEffect {
+    data object NavigateToRegister : AgencySearchSideEffect
+    data class NavigateToAgencyJoin(val agencyId: Long) : AgencySearchSideEffect
+}