[Android][Toast] Can't toast on a thread that has not called Looper.prepare()

Crashlytics에서 이슈 해당 발견

 

문제 발생

마이너 이슈로 등장하고 있다. (고 판단하고 있다. 제발 마이너 이슈여라...)

 

이슈에 대한 설명들

https://stackoverflow.com/a/3875204

Can't create handler inside thread that has not called Looper.prepare()

https://gdlseed.tistory.com/70

안드로이드 코틀린 / can't toast on a thread that has not called looper.prepare()

즉, Toast 호출은 UI 작업이라 Main Thread에서 작업해야 하는데, 실제로 Main Thread가 아닌 쓰레드에서 Toast 호출이 일어난 것.

 

그럼 코드의 어디서, 왜 발생했는가

RetrofitService.INSTANCE.getMyApi()
	.observeOn(Scheduler.io())
    .subscribeOn(Scheduler.io())
    .subscribe(data -> handleData(data),
    	error -> if (error != null) {
        	ExceptionExtensionsKt.handleErrorIfMyCustomException(error);
        }
    );

자바로 작성된 코드이다.

 

에러 공통 처리 메소드(저 부분에서 토스틀 호출한다.) 호출이 Main Thread에서 이뤄지는 것이 아닌 것을 이제 확인했다.

 

저 에러 공통 처리 부분은 Main Thread에서 호출되는 경우가 더 많기는 하다. 

 

이슈를 어떻게 수정할 것인가

다음과 같은 방법들이 있다.

 

1. 저 에러 공통 처리 메소드를 호출하는 모든 곳을 뒤져서 Main Thread에서 실행하도록 수정한다.

이렇게 진행할 경우 예상되는 것들

• 수고롭다.
• 수정해야할 부분을 내가 놓칠 수 있다. 결국엔 또 같은 문제가 발생할 것이다.
• 저 에러 공통 처리 메소드에 토스트 호출이 있다는 걸 인식하지 못한 (히스토리를 잊어버린 미래의 나를 포함해서) 다른 개발자가 저 메소드를 재사용시 Main Thread가 아닌 곳에서 호출하도록 코드를 작성할 수 있다. 그럼 또 이슈가 발생한다.

 

2. 저 에러 공통 처리 메소드 안에서 토스트를 호출하는 부분을 Main Thread에서 실행하도록 수정한다.

이렇게 진행할 경우 예상되는 것들

• 1번보다는 훨씬 덜 수고롭다.
• 저 메소드를 재사용하는 경우 코드를 작성하는 개발자는 작업하는 Thread에 대해서 신경쓸 필요가 없다.

2번으로 선택한다.

 

 

이슈 수정

에러 공통 처리 코드 안을 까보고 나서 과거의 내가 놓친 부분을 발견했다.🤦‍♀️ 

에러 공통 처리 코드 안의 작업들이 Main Thread에서 작업해야하는 것(토스트 호출 등)과 Thread에서 작업해야하는 것(로컬 데이터 삭제 등)으로 나뉘어져 있는데, 코드 작성 당시 이것들을 고려하지 않았다. 🤦‍♀️ 

이번 이슈로 내가 놓친 부분을 발견해서 다행이다.

 

코드 수정은 대략 이러하다.

fun Throwable.handleErrorIfMyCustomException() {
	...
    
    Thread {
    	//UI
        Handler(Looper.getMainLooper()).post {
        	Timber.d("Toast Thread - ${Thread.currentThread().name}")
        	CustomToast.makeTest(context, R.string.msg, Toast.LENGTH_SHORT).show()
        }
        
        //Thread
        Timber.d("Logout Thread - ${Thread.currentThread().name}")
        LocalDataManager.deleteLocalData()
    }.start()
    ...
}

코틀린으로 작성한 코드이다.

 

로그로 동작 확인 완료.

 

로컬 데이터 삭제는 워커 쓰레드에서 동작됨을 확인함

토스트 노출은 메인 쓰레드에서 동작됨은 확인함