🚀 :: (#418) 전체 예외 처리 #423

parkuiery · 2024-11-07T07:07:27Z

개요

전체 예외 처리를 추가하였습니다.

작업 내용

할 말

없음

Summary by CodeRabbit

새로운 기능
- UnknownException 및 ReissueException 예외 추가로 오류 처리 기능 강화.
- 앱에서 발생하는 예외에 대한 사용자 친화적인 알림 메시지 추가.
버그 수정
- 토큰 갱신 실패 시 사용자에게 알림을 제공하는 오류 처리 로직 개선.
문서화
- 새로운 상수 추가로 오류 메시지 제공 방식 개선.

coderabbitai · 2024-11-07T07:07:33Z

Warning

Rate limit exceeded

@parkuiery has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 9 minutes and 28 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between d06bf72 and ff313eb.

Walkthrough

이 변경 사항은 team.retum.common.exception 패키지에 두 개의 새로운 데이터 객체인 UnknownException과 ReissueException을 추가합니다. 이로 인해 기존의 예외 처리 구조가 확장됩니다. 또한, RefreshTokenService.kt와 RequestHandler.kt 파일에서 새로운 예외를 활용하여 오류 처리 로직이 수정되었습니다. 마지막으로, JobisApplication.kt 파일에서는 uncaught 예외 처리기를 추가하여 Firebase Crashlytics와의 통합 및 사용자에게 오류 메시지를 표시하는 기능이 강화되었습니다.

Changes

파일 경로	변경 요약
core/common/src/main/java/team/retum/common/exception/RequestExceptions.kt	`UnknownException` 및 `ReissueException` 데이터 객체 추가.
core/network/src/main/java/team/retum/network/util/RefreshTokenService.kt	`refreshToken` 함수에서 `IllegalStateException` 대신 `ReissueException`을 던지도록 수정.
core/network/src/main/java/team/retum/network/util/RequestHandler.kt	`UnknownException`을 사용하여 일반 `Throwable` 예외를 처리하도록 수정.
app/src/main/java/team/retum/jobisandroidv2/JobisApplication.kt	uncaught 예외 처리기를 추가하고, Firebase Crashlytics와 통합하여 사용자에게 오류 메시지를 표시하는 기능 추가.

Possibly related PRs

🚀 :: v2.2.2 #364: RequestHandler.kt 파일에서 UnknownException을 도입하여 오류 처리를 강화한 변경 사항.
🔗 :: (#380) 모집의뢰서 스켈레톤 버그 #381: 채용 기능의 오류 처리를 강화하는 PR로, UnknownException 및 ReissueException의 도입과 관련이 있음.
🔗 :: (#411) 모집의뢰서 필터 버그 #417: RecruitmentsViewModel.kt에서 오류 처리 개선을 다루며, 메인 PR에서 도입한 새로운 예외 유형과 관련됨.

Suggested reviewers

Tmdhoon2

Poem

🐇 변화의 바람이 불어오네,
두 예외가 새로이 태어나네.
오류는 이제 더 명확하게,
사용자에게 메시지 전하리.
토스트와 함께, 기쁨을 나누며,
우리는 더 나은 세상을 만들어가네! 🍞✨

Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

❤️ Share

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

Review comments: Directly reply to a review comment made by CodeRabbit. Example:
- I pushed a fix in commit <commit_id>, please review it.
- Generate unit testing code for this file.
- Open a follow-up GitHub issue for this discussion.
Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
- @coderabbitai generate unit testing code for this file.
- @coderabbitai modularize this function.
PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
- @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
- @coderabbitai read src/utils.ts and generate unit testing code.
- @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
- @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

@coderabbitai pause to pause the reviews on a PR.
@coderabbitai resume to resume the paused reviews.
@coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
@coderabbitai full review to do a full review from scratch and review all the files again.
@coderabbitai summary to regenerate the summary of the PR.
@coderabbitai resolve resolve all the CodeRabbit review comments.
@coderabbitai configuration to show the current CodeRabbit configuration for the repository.
@coderabbitai help to get help.

Other keywords and placeholders

Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (`.coderabbit.yaml`)

You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
Please see the configuration documentation for more information.
If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

Visit our Documentation for detailed information on how to use CodeRabbit.
Join our Discord Community to get help, request features, and share feedback.
Follow us on X/Twitter for updates and announcements.

coderabbitai

Actionable comments posted: 4

🧹 Outside diff range and nitpick comments (2)

core/common/src/main/java/team/retum/common/exception/RequestExceptions.kt (1)
13-14: 새로운 예외 클래스들이 기존 구조와 잘 통합되었습니다.

UnknownException은 예상치 못한 예외 상황을 처리하는 데 유용할 것 같습니다.

ReissueException은 토큰 갱신 실패를 더 명확하게 표현할 수 있게 되었습니다.

다만, 각 예외 클래스에 대한 문서화(KDoc)를 추가하면 더 좋을 것 같습니다.

예시:
/**
 * 예상치 못한 예외가 발생했을 때 사용되는 예외 클래스입니다.
 */
data object UnknownException : RuntimeException()

/**
 * 토큰 갱신 과정에서 오류가 발생했을 때 사용되는 예외 클래스입니다.
 */
data object ReissueException : RuntimeException()
app/src/main/java/team/retum/jobisandroidv2/JobisExceptionHandler.kt (1)
11-14: 문자열 리소스 파일로 오류 메시지 이동 필요

현재 하드코딩된 오류 메시지들을 res/values/strings.xml로 이동하는 것이 좋습니다. 이렇게 하면:

다국어 지원이 용이해집니다

문자열 관리가 더 체계적이 됩니다

다른 화면에서도 재사용이 가능해집니다

다음과 같이 변경하는 것을 제안합니다:
-private const val InternetErrorMsg = "인터넷 연결을 확인해주세요."
-private const val TimeoutErrorMsg = "요청 시간이 초과되었습니다.\n다시 시도해주세요."
-private const val LoginErrorMsg = "유저정보가 확인되지 않습니다.\n다시 로그인 해주세요."
-private const val UnknownErrorMsg = "처리 중 문제가 발생했습니다.\n앱을 재시작 해주세요."
strings.xml에 추가:
<resources>
    <string name="error_internet">인터넷 연결을 확인해주세요.</string>
    <string name="error_timeout">요청 시간이 초과되었습니다.\n다시 시도해주세요.</string>
    <string name="error_login">유저정보가 확인되지 않습니다.\n다시 로그인 해주세요.</string>
    <string name="error_unknown">처리 중 문제가 발생했습니다.\n앱을 재시작 해주세요.</string>
</resources>

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Reviewing files that changed from the base of the PR and between 0bb929e and b0e61a1.

📒 Files selected for processing (5)

app/src/main/java/team/retum/jobisandroidv2/JobisExceptionHandler.kt (1 hunks)
app/src/main/java/team/retum/jobisandroidv2/MainActivity.kt (1 hunks)
core/common/src/main/java/team/retum/common/exception/RequestExceptions.kt (1 hunks)
core/network/src/main/java/team/retum/network/util/RefreshTokenService.kt (2 hunks)
core/network/src/main/java/team/retum/network/util/RequestHandler.kt (2 hunks)

🔇 Additional comments (5)

core/common/src/main/java/team/retum/common/exception/RequestExceptions.kt (1)

Line range hint 1-12: 예외 처리 구조가 잘 설계되어 있습니다! 👍

각각의 예외 상황에 대해 명확하게 구분된 예외 클래스들이 잘 정의되어 있으며, 일관된 패턴을 따르고 있습니다.

core/network/src/main/java/team/retum/network/util/RefreshTokenService.kt (1)

10-10: LGTM!

새로운 예외 타입의 import가 올바르게 추가되었습니다.

core/network/src/main/java/team/retum/network/util/RequestHandler.kt (2)

14-14: 예외 처리를 위한 import 추가가 적절합니다.

UnknownException import 추가는 전체적인 예외 처리 개선 방향과 일치합니다.

67-67: 🛠️ Refactor suggestion

예외 처리 동작 변경에 대한 검증이 필요합니다.

기존 예외를 UnknownException으로 래핑하는 것은 에러 추적과 디버깅에 영향을 미칠 수 있습니다. 다음 사항들을 확인해 주세요:

원본 예외 정보가 손실되지 않도록 UnknownException 생성자에 원인 예외를 전달하는 것이 좋습니다.
로깅 시스템에서 이러한 예외들이 적절히 기록되는지 확인이 필요합니다.

다음과 같이 수정하는 것을 제안드립니다:

- throw UnknownException
+ throw UnknownException(cause = e)

예외 처리 변경의 영향을 확인하기 위해 다음 스크립트를 실행해보세요:

app/src/main/java/team/retum/jobisandroidv2/MainActivity.kt (1)

52-55: JobisExceptionHandler의 구현을 확인해주세요.

예외 처리기가 Context를 적절히 처리하는지 확인이 필요합니다.

다음 스크립트로 JobisExceptionHandler의 구현을 확인하겠습니다:

✅ Verification successful

Let me try another search to find the JobisExceptionHandler implementation:

Let me check the complete implementation including the makeToast method:

JobisExceptionHandler의 Context 처리가 적절합니다.

JobisExceptionHandler의 구현을 검토한 결과:

Context를 private 필드로 적절히 캡슐화하고 있습니다
Context는 오직 Toast 메시지 표시에만 사용되며, 이는 일반적인 안드로이드 패턴입니다
메모리 누수를 일으킬 수 있는 static 참조나 장기 보관이 없습니다

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Description: Verify JobisExceptionHandler implementation for proper context handling

# Check how context is stored and used in JobisExceptionHandler
ast-grep --pattern 'class JobisExceptionHandler($context: Context) : Thread.UncaughtExceptionHandler {
  $$$
}'

# Check for potential memory leaks by looking for strong references to context
rg -A 5 'context\.'

Length of output: 36918

Script:

#!/bin/bash
# Search for JobisExceptionHandler class implementation
rg -A 10 "class JobisExceptionHandler"

# Search for files that might contain JobisExceptionHandler
fd "JobisExceptionHandler"

Length of output: 1350

Script:

#!/bin/bash
# Get the complete implementation of JobisExceptionHandler
cat app/src/main/java/team/retum/jobisandroidv2/JobisExceptionHandler.kt

# Search for any strong references to context in the same file
rg "context\." app/src/main/java/team/retum/jobisandroidv2/JobisExceptionHandler.kt

Length of output: 1457

coderabbitai · 2024-11-07T07:11:44Z

app/src/main/java/team/retum/jobisandroidv2/JobisExceptionHandler.kt

+    fun makeToast(message: String) {
+        JobisToast.create(
+            context = context,
+            message = message,
+            drawable = JobisIcon.Error,
+        )
+    }


🛠️ Refactor suggestion

Toast 메서드 가시성 및 기능 개선 필요

makeToast 메서드에 대한 개선사항:

메서드가 public으로 선언되어 있어 불필요한 외부 노출이 있습니다

Toast 지속 시간을 설정할 수 없습니다

다음과 같이 개선하는 것을 제안합니다:

- fun makeToast(message: String) { + private fun makeToast( + message: String, + duration: Int = Toast.LENGTH_SHORT + ) { JobisToast.create( context = context, message = message, drawable = JobisIcon.Error, + duration = duration, ) }

Committable suggestion skipped: line range outside the PR's diff.

coderabbitai · 2024-11-07T07:11:44Z

app/src/main/java/team/retum/jobisandroidv2/JobisExceptionHandler.kt

+    override fun uncaughtException(t: Thread, e: Throwable) {
+        when (e) {
+            is OfflineException -> makeToast(InternetErrorMsg)
+            is ConnectionTimeOutException -> makeToast(TimeoutErrorMsg)
+            is ReissueException -> makeToast(LoginErrorMsg)
+            is UnknownException -> makeToast(UnknownErrorMsg)
+        }
+    }


⚠️ Potential issue

예외 처리 개선 필요

현재 구현에서 몇 가지 개선이 필요합니다:

when 식에 else 분기가 없어 처리되지 않는 예외가 발생할 수 있습니다

디버깅을 위한 로깅이 없습니다

더 구체적인 예외 처리가 필요할 수 있습니다

다음과 같이 개선하는 것을 제안합니다:

override fun uncaughtException(t: Thread, e: Throwable) { + Timber.e(e, "Uncaught exception occurred") when (e) { is OfflineException -> makeToast(InternetErrorMsg) is ConnectionTimeOutException -> makeToast(TimeoutErrorMsg) is ReissueException -> makeToast(LoginErrorMsg) is UnknownException -> makeToast(UnknownErrorMsg) + else -> { + Timber.e(e, "Unhandled exception") + makeToast(UnknownErrorMsg) + } } }

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

override fun uncaughtException(t: Thread, e: Throwable) {

when (e) {

is OfflineException -> makeToast(InternetErrorMsg)

is ConnectionTimeOutException -> makeToast(TimeoutErrorMsg)

is ReissueException -> makeToast(LoginErrorMsg)

is UnknownException -> makeToast(UnknownErrorMsg)

}

}

override fun uncaughtException(t: Thread, e: Throwable) {

Timber.e(e, "Uncaught exception occurred")

when (e) {

is OfflineException -> makeToast(InternetErrorMsg)

is ConnectionTimeOutException -> makeToast(TimeoutErrorMsg)

is ReissueException -> makeToast(LoginErrorMsg)

is UnknownException -> makeToast(UnknownErrorMsg)

else -> {

Timber.e(e, "Unhandled exception")

makeToast(UnknownErrorMsg)

}

}

}

coderabbitai · 2024-11-07T07:11:44Z

core/network/src/main/java/team/retum/network/util/RefreshTokenService.kt

@@ -40,7 +41,7 @@ object RefreshTokenService {
        }.onSuccess { token ->
            return token
        }.onFailure {
-            throw IllegalStateException("Fail refresh: ${it.message}")
+            throw ReissueException


🛠️ Refactor suggestion

예외 처리를 개선해 주세요.

현재 구현은 원래 발생한 예외의 정보를 모두 잃어버립니다. 디버깅과 문제 해결을 위해 다음과 같은 개선을 제안합니다:

원인이 되는 예외를 보존

의미 있는 오류 메시지 추가

로깅 추가

다음과 같이 수정하는 것을 추천드립니다:

- throw ReissueException + throw ReissueException("토큰 갱신 중 오류가 발생했습니다", it)

또한 로깅 구현을 위해 Timber나 다른 로깅 라이브러리를 사용하는 것을 고려해보세요.

Committable suggestion skipped: line range outside the PR's diff.

app/src/main/java/team/retum/jobisandroidv2/MainActivity.kt

coderabbitai

Actionable comments posted: 2

🧹 Outside diff range and nitpick comments (2)

app/src/main/java/team/retum/jobisandroidv2/JobisExceptionHandler.kt (2)
11-14: 문자열 리소스 파일로 이동 권장

오류 메시지를 strings.xml 리소스 파일로 이동하는 것이 좋습니다:

다국어 지원이 용이해집니다

문자열 재사용이 가능해집니다

일관된 메시지 관리가 가능해집니다

예시:

<resources>
    <string name="error_internet">인터넷 연결을 확인해주세요.</string>
    <string name="error_timeout">요청 시간이 초과되었습니다.\n다시 시도해주세요.</string>
    <string name="error_login">유저정보가 확인되지 않습니다.\n다시 로그인 해주세요.</string>
    <string name="error_unknown">처리 중 문제가 발생했습니다.\n앱을 재시작 해주세요.</string>
</resources>
16-18: Context 참조 관리 개선 필요

메모리 누수를 방지하기 위해 다음 사항을 고려해주세요:

ApplicationContext를 사용하도록 명시적으로 제한

Context 참조가 적절히 해제되는지 확인
class JobisExceptionHandler(
-    private val context: Context,
+    context: Context,
) : Thread.UncaughtExceptionHandler {
+    private val applicationContext = context.applicationContext

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Reviewing files that changed from the base of the PR and between b0e61a1 and 6209b0e.

📒 Files selected for processing (2)

app/src/main/java/team/retum/jobisandroidv2/JobisExceptionHandler.kt (1 hunks)
app/src/main/java/team/retum/jobisandroidv2/MainActivity.kt (1 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

app/src/main/java/team/retum/jobisandroidv2/MainActivity.kt

coderabbitai · 2024-11-07T08:30:09Z

app/src/main/java/team/retum/jobisandroidv2/JobisExceptionHandler.kt

+    private fun makeToast(message: String) {
+        JobisToast.create(
+            context = context,
+            message = message,
+            drawable = JobisIcon.Error,
+        )
+    }


🛠️ Refactor suggestion

Toast 표시 안정성 개선 필요

Toast 메시지 표시의 안정성을 높이기 위한 개선사항:

메인 스레드에서 실행되도록 보장

Toast 지속 시간 설정 옵션 추가

실패 시 대체 처리 방안 고려

private fun makeToast(message: String) { + Handler(Looper.getMainLooper()).post { + try { JobisToast.create( context = context, message = message, drawable = JobisIcon.Error, + duration = Toast.LENGTH_LONG ) + } catch (e: Exception) { + Timber.e(e, "Failed to show toast") + } + } }

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

private fun makeToast(message: String) {

JobisToast.create(

context = context,

message = message,

drawable = JobisIcon.Error,

)

}

private fun makeToast(message: String) {

Handler(Looper.getMainLooper()).post {

try {

JobisToast.create(

context = context,

message = message,

drawable = JobisIcon.Error,

duration = Toast.LENGTH_LONG

)

} catch (e: Exception) {

Timber.e(e, "Failed to show toast")

}

}

}

coderabbitai · 2024-11-07T08:30:09Z

app/src/main/java/team/retum/jobisandroidv2/JobisExceptionHandler.kt

+    override fun uncaughtException(t: Thread, e: Throwable) {
+        when (e) {
+            is OfflineException -> makeToast(InternetErrorMsg)
+            is ConnectionTimeOutException -> makeToast(TimeoutErrorMsg)
+            is ReissueException -> makeToast(LoginErrorMsg)
+            is UnknownException -> makeToast(UnknownErrorMsg)
+            else -> makeToast(UnknownErrorMsg)
+        }
+    }


🛠️ Refactor suggestion

예외 로깅 추가 필요

디버깅과 모니터링을 위해 로깅을 추가하는 것이 좋습니다:

예외 발생 시점과 상세 정보 기록

스레드 정보 활용

Crashlytics 등의 크래시 리포팅 도구 통합

override fun uncaughtException(t: Thread, e: Throwable) { + Timber.e(e, "Uncaught exception in thread ${t.name}") when (e) { is OfflineException -> makeToast(InternetErrorMsg) is ConnectionTimeOutException -> makeToast(TimeoutErrorMsg) is ReissueException -> makeToast(LoginErrorMsg) is UnknownException -> makeToast(UnknownErrorMsg) - else -> makeToast(UnknownErrorMsg) + else -> { + Timber.e(e, "Unhandled exception type: ${e.javaClass.simpleName}") + makeToast(UnknownErrorMsg) + } } }

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

override fun uncaughtException(t: Thread, e: Throwable) {

when (e) {

is OfflineException -> makeToast(InternetErrorMsg)

is ConnectionTimeOutException -> makeToast(TimeoutErrorMsg)

is ReissueException -> makeToast(LoginErrorMsg)

is UnknownException -> makeToast(UnknownErrorMsg)

else -> makeToast(UnknownErrorMsg)

}

}

override fun uncaughtException(t: Thread, e: Throwable) {

Timber.e(e, "Uncaught exception in thread ${t.name}")

when (e) {

is OfflineException -> makeToast(InternetErrorMsg)

is ConnectionTimeOutException -> makeToast(TimeoutErrorMsg)

is ReissueException -> makeToast(LoginErrorMsg)

is UnknownException -> makeToast(UnknownErrorMsg)

else -> {

Timber.e(e, "Unhandled exception type: ${e.javaClass.simpleName}")

makeToast(UnknownErrorMsg)

}

}

}

app/src/main/java/team/retum/jobisandroidv2/JobisExceptionHandler.kt

coderabbitai

Actionable comments posted: 1

🧹 Outside diff range and nitpick comments (2)

app/src/main/java/team/retum/jobisandroidv2/JobisApplication.kt (2)
25-41: 예외 처리 로직 개선이 필요합니다.

현재 구현은 다음과 같은 잠재적인 문제가 있습니다:

exitProcess(0)를 즉시 호출하면 사용자 데이터가 손실될 수 있습니다

모든 예외를 동일한 방식으로 처리하는 것은 사용자 경험을 저하시킬 수 있습니다

다음과 같은 개선을 제안드립니다:

치명적인 오류와 복구 가능한 오류를 구분하여 처리

앱 종료 전에 사용자 데이터를 저장하는 로직 추가

복구 가능한 경우 다이얼로그를 통해 사용자에게 선택권 제공

예시 구현:
private fun handleFatalError(thread: Thread, exception: Throwable) {
    Firebase.crashlytics.recordException(exception)
    
    // 사용자 데이터 저장
    saveApplicationState()
    
    // 복구 가능한 오류인 경우
    if (isRecoverableError(exception)) {
        showRecoveryDialog(exception)
        return
    }
    
    // 치명적인 오류인 경우
    showFatalErrorDialog {
        startActivity(
            Intent(this, MainActivity::class.java).apply {
                addFlags(Intent.FLAG_ACTIVITY_NEW_TASK or Intent.FLAG_ACTIVITY_CLEAR_TASK)
            }
        )
        exitProcess(0)
    }
}
44-50: Toast 메시지 표시 방식 개선이 필요합니다.

현재 구현은 앱이 재시작되기 전에 Toast 메시지가 사라질 수 있습니다.

다음과 같은 개선을 제안드립니다:
private fun showErrorMessage(message: String) {
    // 더 긴 시간 동안 표시
    Toast.makeText(this, message, Toast.LENGTH_LONG).show()
    
    // 또는 Snackbar 사용
    val rootView = findViewById<View>(android.R.id.content)
    Snackbar.make(rootView, message, Snackbar.LENGTH_LONG).show()
}

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Reviewing files that changed from the base of the PR and between 489c421 and d06bf72.

📒 Files selected for processing (1)

app/src/main/java/team/retum/jobisandroidv2/JobisApplication.kt (1 hunks)

🔇 Additional comments (1)

app/src/main/java/team/retum/jobisandroidv2/JobisApplication.kt (1)

4-13: 필요한 import 문이 모두 잘 추가되었습니다!

예외 처리와 Firebase Crashlytics 통합을 위한 import 문이 적절하게 포함되어 있습니다.

coderabbitai · 2024-11-07T13:02:50Z

app/src/main/java/team/retum/jobisandroidv2/JobisApplication.kt

+private const val InternetErrorMsg = "인터넷 연결을 확인해주세요."
+private const val TimeoutErrorMsg = "요청 시간이 초과되었습니다.\n다시 시도해주세요."
+private const val LoginErrorMsg = "유저정보가 확인되지 않습니다.\n다시 로그인 해주세요."
+private const val UnknownErrorMsg = "처리 중 문제가 발생했습니다."


🛠️ Refactor suggestion

문자열 리소스 관리 개선이 필요합니다.

하드코딩된 에러 메시지를 res/values/strings.xml로 이동하는 것이 좋습니다. 이렇게 하면:

다국어 지원이 용이해집니다

문자열 재사용이 가능해집니다

일관된 메시지 관리가 가능해집니다

다음과 같이 변경하는 것을 제안드립니다:

-private const val InternetErrorMsg = "인터넷 연결을 확인해주세요." -private const val TimeoutErrorMsg = "요청 시간이 초과되었습니다.\n다시 시도해주세요." -private const val LoginErrorMsg = "유저정보가 확인되지 않습니다.\n다시 로그인 해주세요." -private const val UnknownErrorMsg = "처리 중 문제가 발생했습니다."

strings.xml에 추가:

<resources> <string name="error_internet">인터넷 연결을 확인해주세요.</string> <string name="error_timeout">요청 시간이 초과되었습니다.\n다시 시도해주세요.</string> <string name="error_login">유저정보가 확인되지 않습니다.\n다시 로그인 해주세요.</string> <string name="error_unknown">처리 중 문제가 발생했습니다.</string> </resources>

parkuiery added 2 commits November 7, 2024 16:06

feat :: JobisExceptionHandler 구현

56d274a

refactor :: exception 추가

b0e61a1

parkuiery added 버그 버그를 수정합니다. 리팩토링 코드를 리팩토링 합니다. labels Nov 7, 2024

parkuiery self-assigned this Nov 7, 2024

parkuiery requested a review from a team as a code owner November 7, 2024 07:07

parkuiery linked an issue Nov 7, 2024 that may be closed by this pull request

전체 예외 처리 #418

Closed

coderabbitai bot reviewed Nov 7, 2024

View reviewed changes

refactor :: applicationContext으로 변경

6209b0e

coderabbitai bot reviewed Nov 7, 2024

View reviewed changes

Tmdhoon2 requested changes Nov 7, 2024

View reviewed changes

app/src/main/java/team/retum/jobisandroidv2/JobisExceptionHandler.kt Outdated Show resolved Hide resolved

refactor :: toast 변경

489c421

parkuiery requested a review from Tmdhoon2 November 7, 2024 12:07

parkuiery added 2 commits November 7, 2024 21:51

refactor :: 데이터 수집 및 재시작 로직 추가

46f94a8

refactor :: ktlint

d06bf72

coderabbitai bot reviewed Nov 7, 2024

View reviewed changes

refactor :: lint

d6b72ee

Tmdhoon2 approved these changes Nov 7, 2024

View reviewed changes

parkuiery added 2 commits November 7, 2024 22:35

refactor :: 재실행 로직 변경

e2f3af9

refactor :: ktlint

ff313eb

parkuiery merged commit 68ca1ef into develop Nov 7, 2024
2 checks passed

parkuiery deleted the feature/418-전체-예외-처리 branch November 7, 2024 13:45

coderabbitai bot mentioned this pull request Nov 8, 2024

🔗 :: (#430) 비밀번호 찾기 이메일 인증 버그 #431

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 :: (#418) 전체 예외 처리 #423

🚀 :: (#418) 전체 예외 처리 #423

parkuiery commented Nov 7, 2024 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Nov 7, 2024 •

edited

Loading

Rate limit exceeded

Chat

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (`.coderabbit.yaml`)

Documentation and Community

coderabbitai bot left a comment

coderabbitai bot Nov 7, 2024

coderabbitai bot Nov 7, 2024

coderabbitai bot Nov 7, 2024

coderabbitai bot left a comment

coderabbitai bot Nov 7, 2024

coderabbitai bot Nov 7, 2024

coderabbitai bot left a comment

coderabbitai bot Nov 7, 2024

🚀 :: (#418) 전체 예외 처리 #423

🚀 :: (#418) 전체 예외 처리 #423

Conversation

parkuiery commented Nov 7, 2024 • edited by coderabbitai bot Loading

개요

작업 내용

할 말

Summary by CodeRabbit

coderabbitai bot commented Nov 7, 2024 • edited Loading

Rate limit exceeded

Walkthrough

Changes

Possibly related PRs

Suggested reviewers

Poem

Chat

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (.coderabbit.yaml)

Documentation and Community

coderabbitai bot left a comment

Choose a reason for hiding this comment

coderabbitai bot Nov 7, 2024

Choose a reason for hiding this comment

coderabbitai bot Nov 7, 2024

Choose a reason for hiding this comment

coderabbitai bot Nov 7, 2024

Choose a reason for hiding this comment

coderabbitai bot left a comment

Choose a reason for hiding this comment

coderabbitai bot Nov 7, 2024

Choose a reason for hiding this comment

coderabbitai bot Nov 7, 2024

Choose a reason for hiding this comment

coderabbitai bot left a comment

Choose a reason for hiding this comment

coderabbitai bot Nov 7, 2024

Choose a reason for hiding this comment

parkuiery commented Nov 7, 2024 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Nov 7, 2024 •

edited

Loading

CodeRabbit Configuration File (`.coderabbit.yaml`)