[Android 삽질 교훈] RecyclerView에 RxJava는 쓰지 말아야 합니다.

리사이클러뷰의 리스트 아이템에 연속되는 클릭 이벤트를 막고자 RxJava의 throttleFirst를 활용했습니다.

(저는 이 작업이 RxJava의 응용에 제가 한 발짝 더 나아갔다는 생각에 사무실에서 내적 기쁨을 누렸어요.)

 

메모리 누수도 잊지 말고 막아야지😉 이러고  RecyclerView에서 RxJava의 dispose를 진행할 적당한 콜백 함수가 있나 구글에게 물어봤습니다.

StackOverFlow에서 어댑터 안의 onViewDetachedFromWindow를 이용하면 된다는 글을 확인했구요.

 

다음과 같이 코드를 짰어요.

override fun onViewDetachedFromWindow(holder: RecyclerView.ViewHolder) {
    if (holder is ViewHolder) {
        holder.disposable.dispose()
    }
    super.onViewDetachedFromWindow(holder)
}

 

그리고 이슈가 발생했습니다. 😧

해당 RecyclerView의 리스트를 스크롤해서 잠깐 가려졌다가 다시 보이게 된 리스트 아이템에서는 클릭 이벤트가 먹히질 않았습니다.

 

RecyclerView의 아이템이 화면에 노출되지 않았을 때 onViewDetachedFromWindow()가 호출되었기에 이미 dispose() 코드가 실행되었던 것이였어요. 가려진 그 아이템이 다시 화면에 노출되어 생성된 클릭 이벤트 스트림은 그 누구도 받아주지 않고 흘러가게 되었던 거였어요.... 😢(눈물 또르르)

 

잠시 반성의 시간을 가지고...😞 onViewDetachedFromWindow에 대해 잘 모르고 코드를 작성했다는 것이 이렇게 드러났습니다. 부끄럽습니다.😔😢

 

이런 상황을 맞이하고 나서야 'RecyclerView에 RxJava를 사용하는게 맞기는 한건가?'라는 질문이 들었어요. Activity나 Fragment가 destroyed 되는 상태의 콜백 함수에서 dispose()를 실행하는 코드들만 봤지, RecyclerView에서 사용하는 것은 사실 보지 못했다는 생각이 들더군요.

 

그리고 너무나 감사한 글을 찾았어요.

 

https://stackoverflow.com/questions/57609883/where-to-clear-compositedisposable-in-recyclerview-adapter

Where to clear CompositeDisposable in RecyclerView.Adapter

제가 도움을 받은 부분은 여기입니다.

 

It would be easier to answer if you could provide the code of the adapter. In general, Disposable should be disposed with respect to your business logic and the component containing lifecycle.

I would also say that it is better to not use Rx inside of RecyclerView adapter. Here are the benefits:
- Adapter logic remains simple and synchronous.
- You don't need to think about possible lifecycle or multithreading issues when developing the adapter.
- Rx streams always allocate a lot of memory, so (if we talk about RecyclerView) putting them into the wrong place can lead to performance issues.

So, I'd suggest moving Rx streams into Presenter/ViewModel or similar component.

 

발번역을 하자면

당신이 어댑터에 코드를 제공하는 게 더 쉬울 거에요. 보통, Disposable은 당신의 비지니스 로직과 컴포넌트의 라이프사이클을 따라 dispose되어야만해요.

또 RecyclerView 어댑터 안에는 Rx를 쓰지 않는게 낫다는 말을 하고 싶네요.

-어댑터 로직은 간단하고 동기적이죠
-어댑터를 개발할 때에 가능한 라이프사이클 또는 멀티쓰레드 이슈들은 생각할 필요가 없어요.
-Rx 스트림은 항상 많은 메모리를 할당해서, 만약 (우리가 리싸이클러뷰에 대해서 말한다면) 잘못된 곳에 가져다 쓰면 퍼포먼스 이슈를 만들 수 있어요.

그래서 Rx 스트림을 Presenter/ViewModel 또는 비슷한 컴포넌트로 옮기는 것을 권해요.

저렇게 친절하게 글을 작성해주신 Dimsky님께 감사를 드리면서, 코드 수정했습니다.

 

해결되서 다행이에요. 끝😀