반응형
리뷰 내용 :
subscribeOn과 observeOn을 모두 메인스레드에 하고 있는데 둘 다 메인스레드로 설정할 필요는 없어 보입니다.
제 생각에는 sendPhotoObservable와 sendSignObservable를 병렬로 처리할 수 있도록
subscribeOn은 IO스레드나 새로운 스레드(Schedulers.newThread())에서 하고
observeOn을 지금처럼 메인스레드에서 처리하여 zip한 결과를 UI 스레드에서 처리하는게 좋지 않을까 싶네요
반응형
'코딩 > 코드리뷰' 카테고리의 다른 글
자바 함수 및 exception 발생 여부 (0) | 2023.04.11 |
---|---|
Observable, Single 비교 (0) | 2022.01.27 |
TakeIf로 값 넣기 (0) | 2021.12.20 |
변수 3개 선언하기 (0) | 2021.11.23 |
뷰홀더 LayoutInflater Activity에 의존하지 않기 (0) | 2021.11.18 |