Convert RxJava Observables To Live Data With Kotlin Extension Functions

前端 未结 3 464
攒了一身酷
攒了一身酷 2021-01-30 14:26

I\'ve been using alot of RxJava Observables converted to LiveData in my code using LiveDataReactiveStreams.fromPublisher() library. So I t

3条回答
  •  北恋
    北恋 (楼主)
    2021-01-30 14:49

    I think this is a pretty good idea. An example benefit of LiveData is the ability of using it directly in your data-binding layouts. Let's say in your view-model you have:

    val user: LiveData
    
    data class User(val firstName: String, val lastName: String)
    

    in your layout you can bind the properties of the User directly:

    android:text="${viewModel.user.firstName}"
    

    You can't use reactive streams in data-binding like this. If user was Flowable, referencing ${viewModel.user.firstName} wouldn't work.

    Furthermore, data-binding will handle the lifecycle for you (observing changes only in active state and updating the view when changes happen) if your activity or fragment calls ViewDataBinding.setLifecycleOwner(LifecycleOwner):

    binding.setLifecycleOwner(this)
    

    The one for converting Completable to LiveData doesn't really make sense to me, because it will never notify the observer about anything, so I'd just get rid of it.

    There are some considerations when converting from reactive streams to live data (like I had when I wanted to resume a countdown after rotation), but I don't think they are related to the extension functions you presented, those seem to do their job. The issue to keep in mind here is that when the lifecycle owner moves from active to inactive state, PublisherLiveData cancels the subscription to the stream, and when the state changes to active, it will create a new subscription, which means restarting the stream in many cases (I guess that is if the stream is "cold"), while you probably want to resume the stream from where it was after rotation or other configuration changes. If the stream was "hot" on the other hand, emissions get ignored during the inactive state. I think this problem has to be addressed even if you used reactive streams directly and handled life cycle manually. But the thing is that simply converting reactive streams to LiveData isn't enough to solve this problem.

    It's good to document those methods as not handling the error state, which has to be handled upstream. Alternatively, that could be one of the improvements for these functions - transforming the stream first to handle errors (as a lambda parameter with a default for example). Another possibility would be to utilize Result (experimental at the moment), or something similar to encapsulate the success or error.


    As an afterthought, regarding this part that I wrote above:

    There are some considerations when converting from reactive streams to live data, but I don't think they are related to the extension functions you presented.

    I still think it holds true in general, however I'm not sure if you actually want to use Single.toLiveData() and Maybe.toLiveData() in practice most of the time. Since Maybe and Single are modeling one-time operations, then it may be preferable not to cancel it when there are no active observers and have to re-start it once there are new active observers. Instead, posting to some MutableLiveData and disposing the Single/Maybe in onCleared might be useful (I'm not sure that can be encapsulated in an extension function). They still may have some use that I simply don't see at the moment.

    By the way, your Flowable.toLiveData() is already in the androidx.lifecycle:lifecycle-reactivestreams-ktx artifact.

    This leaves the Observable.toLiveData(), which I think should be just as useful as Flowable one.

提交回复
热议问题