release-1.0.0 fix: ocr 예외처리

MONEYMONG · Feb 8, 2024 · a3e6b4a · a3e6b4a
1 parent 77a0379
commit a3e6b4a
Show file tree

Hide file tree

Showing 4 changed files with 29 additions and 5 deletions.
diff --git a/core/common/src/main/java/com/moneymong/moneymong/common/ui/PrimitiveExt.kt b/core/common/src/main/java/com/moneymong/moneymong/common/ui/PrimitiveExt.kt
@@ -12,10 +12,14 @@ fun String.toWonFormat(unit: Boolean = false): String {
     return if (this.isEmpty()) {
         this
     } else {
-        if (unit) {
-            "${DecimalFormat("#,###").format(this.toLong())}원"
-        } else {
-            DecimalFormat("#,###").format(this.toLong())
+        try {
+            if (unit) {
+                "${DecimalFormat("#,###").format(this.toLong())}원"
+            } else {
+                DecimalFormat("#,###").format(this.toLong())
+            }
+        } catch (e: Exception) {
+            ""
         }
     }
 }

diff --git a/feature/ocr/src/main/java/com/moneymong/moneymong/ocr/OCRScreen.kt b/feature/ocr/src/main/java/com/moneymong/moneymong/ocr/OCRScreen.kt
@@ -36,6 +36,7 @@ import com.moneymong.moneymong.design_system.theme.White
 import com.moneymong.moneymong.common.ext.hasPermission
 import com.moneymong.moneymong.common.ui.noRippleClickable
 import com.moneymong.moneymong.common.util.DisposableEffectWithLifeCycle
+import com.moneymong.moneymong.design_system.error.ErrorDialog
 import com.moneymong.moneymong.design_system.theme.Black
 import com.moneymong.moneymong.ocr.view.OCRCameraPermissionDeniedView
 import com.moneymong.moneymong.ocr.view.OCRCaptureView
@@ -105,6 +106,12 @@ fun OCRScreen(
         )
     }
 
+    if (state.visibleErrorDialog) {
+        ErrorDialog(message = state.errorMessage) {
+            viewModel.onChangeVisibleErrorDialog(false)
+        }
+    }
+
     Scaffold {
         Box(
             modifier = modifier

diff --git a/feature/ocr/src/main/java/com/moneymong/moneymong/ocr/OCRState.kt b/feature/ocr/src/main/java/com/moneymong/moneymong/ocr/OCRState.kt
@@ -10,7 +10,9 @@ data class OCRState(
     val permissionDialogStatus: Boolean = false,
     val isDeniedCamera: Boolean = false,
     val visibleHelper: Boolean = true,
-    val isLoading: Boolean = false
+    val isLoading: Boolean = false,
+    val visibleErrorDialog: Boolean = false,
+    val errorMessage: String = ""
 ): State {
 
     val isReceipt: Boolean

diff --git a/feature/ocr/src/main/java/com/moneymong/moneymong/ocr/OCRViewModel.kt b/feature/ocr/src/main/java/com/moneymong/moneymong/ocr/OCRViewModel.kt
@@ -49,6 +49,13 @@ class OCRViewModel @Inject constructor(
                 .onSuccess {
                     reduce { state.copy(document = it) }
                     possibleNavigateToOCRResult(it.images.first().inferResult.orEmpty())
+                }.onFailure {
+                    reduce {
+                        state.copy(
+                            visibleErrorDialog = true,
+                            errorMessage = "영수증을 인식하지 못했습니다."
+                        )
+                    }
                 }.also { reduce { state.copy(isLoading = false) } }
         }
     }
@@ -110,4 +117,8 @@ class OCRViewModel @Inject constructor(
     fun onClickHelper() = intent {
         reduce { state.copy(visibleHelper = !state.visibleHelper) }
     }
+
+    fun onChangeVisibleErrorDialog(visible: Boolean) = intent {
+        reduce { state.copy(visibleErrorDialog = visible) }
+    }
 }