[BOOK-145] refactor: handleException 함수 개선

Author: easyhooon

core/common/build.gradle.kts

@@ -14,6 +14,7 @@ android {
 dependencies {
     implementations(
         projects.core.model,
+        projects.core.network,
 
         libs.logger,
     )

core/common/src/main/kotlin/com/ninecraft/booket/core/common/utils/HandleException.kt

@@ -1,38 +1,77 @@
 package com.ninecraft.booket.core.common.utils
 
+import com.ninecraft.booket.core.network.response.ErrorResponse
 import com.orhanobut.logger.Logger
+import kotlinx.serialization.SerializationException
+import kotlinx.serialization.json.Json
 import retrofit2.HttpException
+import java.io.IOException
 import java.net.ConnectException
 import java.net.SocketTimeoutException
 import java.net.UnknownHostException
 
 fun handleException(
     exception: Throwable,
+    onError: (String) -> Unit,
     onLoginRequired: () -> Unit,
-    onServerError: (String) -> Unit,
-    onNetworkError: (String) -> Unit,
 ) {
     when {
         exception is HttpException && exception.code() == 401 -> {
             onLoginRequired()
         }
 
-        exception is HttpException && exception.code() in 500..599 -> {
-            onServerError("서버 오류가 발생했습니다.")
+        exception is HttpException -> {
+            val serverMessage = exception.parseErrorMessage()
+            val message = serverMessage ?: getHttpErrorMessage(exception.code())
+            Logger.e("HTTP ${exception.code()}: $message")
+            onError(message)
         }
 
-        exception is UnknownHostException || exception is ConnectException -> {
-            onNetworkError("네트워크 연결을 확인해주세요.")
-        }
-
-        exception is SocketTimeoutException -> {
-            onServerError("서버 응답 시간이 초과되었습니다.")
+        exception.isNetworkError() -> {
+            onError("네트워크 연결을 확인해주세요.")
         }
 
         else -> {
-            val errorMessage = exception.message ?: "알 수 없는 오류가 발생했습니다."
+            val errorMessage = exception.message ?: "알 수 없는 오류가 발생했습니다"
             Logger.e(errorMessage)
-            onServerError(errorMessage)
+            onError(errorMessage)
         }
     }
 }
+
+@Suppress("TooGenericExceptionCaught")
+private fun HttpException.parseErrorMessage(): String? {
+    return try {
+        val errorBody = response()?.errorBody()?.string()
+        if (errorBody.isNullOrBlank()) return null
+        Json.decodeFromString<ErrorResponse>(errorBody).getErrorMessage()
+    } catch (e: SerializationException) {
+        Logger.e("JSON parsing failed: ${e.message}")
+        null
+    } catch (e: IOException) {
+        Logger.e("Failed to read error body: ${e.message}")
+        null
+    } catch (e: Exception) {
+        Logger.e("Unexpected error parsing response: ${e.message}")
+        null
+    }
+}
+
+private fun getHttpErrorMessage(statusCode: Int): String {
+    return when (statusCode) {
+        400 -> "요청이 올바르지 않습니다"
+        403 -> "접근 권한이 없습니다"
+        404 -> "존재하지 않는 데이터입니다"
+        429 -> "요청이 너무 많습니다. 잠시 후 다시 시도해주세요"
+        in 400..499 -> "요청 처리 중 오류가 발생했습니다"
+        in 500..599 -> "서버 오류가 발생했습니다"
+        else -> "알 수 없는 오류가 발생했습니다"
+    }
+}
+
+private fun Throwable.isNetworkError(): Boolean {
+    return this is UnknownHostException ||
+        this is ConnectException ||
+        this is SocketTimeoutException ||
+        this is IOException
+}

core/network/src/main/kotlin/com/ninecraft/booket/core/network/response/ErrorResponse.kt

@@ -0,0 +1,18 @@
+package com.ninecraft.booket.core.network.response
+
+import kotlinx.serialization.SerialName
+import kotlinx.serialization.Serializable
+
+@Serializable
+data class ErrorResponse(
+    @SerialName("status")
+    val status: String,
+    @SerialName("code")
+    val code: String,
+    @SerialName("message")
+    val message: String,
+) {
+    fun getErrorMessage(): String {
+        return message.takeIf { it.isNotBlank() } ?: "알 수 없는 오류가 발생했습니다"
+    }
+}

feature/library/src/main/kotlin/com/ninecraft/booket/feature/library/LibraryPresenter.kt

@@ -52,8 +52,7 @@ class LibraryPresenter @AssistedInject constructor(
 
                             handleException(
                                 exception = exception,
-                                onServerError = handleErrorMessage,
-                                onNetworkError = handleErrorMessage,
+                                onError = handleErrorMessage,
                                 onLoginRequired = {
                                     navigator.resetRoot(LoginScreen)
                                 },

feature/search/src/main/kotlin/com/ninecraft/booket/feature/search/SearchPresenter.kt

@@ -107,8 +107,7 @@ class SearchPresenter @AssistedInject constructor(
 
                         handleException(
                             exception = exception,
-                            onServerError = handleErrorMessage,
-                            onNetworkError = handleErrorMessage,
+                            onError = handleErrorMessage,
                             onLoginRequired = {
                                 navigator.resetRoot(LoginScreen)
                             },

feature/settings/src/main/kotlin/com/ninecraft/booket/feature/settings/SettingsPresenter.kt

@@ -87,8 +87,7 @@ class SettingsPresenter @AssistedInject constructor(
 
                                     handleException(
                                         exception = exception,
-                                        onServerError = handleErrorMessage,
-                                        onNetworkError = handleErrorMessage,
+                                        onError = handleErrorMessage,
                                         onLoginRequired = {
                                             navigator.resetRoot(LoginScreen)
                                         },