RxJava2 onSubscribe and onComplete

The name of the pictureThe name of the pictureThe name of the pictureClash Royale CLAN TAG#URR8PPP



RxJava2 onSubscribe and onComplete



I'm learning RxJava2, and I have a question about proper usage to prevent memory leaks.



Do I need to ensure that onComplete() gets called for every time onSubscribe() gets called?



I have a stream that's set up like this:


Observables.combineLatest(
siteDataController
.GetCurrentSelectionObservable()
.distinctUntilChanged site -> site.GetStid() ,
uiController
.GetDisplayFieldsObservable()
.observeOn(AndroidSchedulers.mainThread())
) site, data -> Pair(site, data)
.takeWhilecontinueUpdates.get()
.subscribeOn(Schedulers.computation())
.observeOn(AndroidSchedulers.mainThread())
.subscribe(object : Observer<Pair<SiteDataController.ProcessedSite, UIController.DisplayFields>>
override fun onComplete()
Log.e("Lifespan", "Site / UI onComplete()")

override fun onSubscribe(d: Disposable)
Log.e("Lifespan", "Site / UI onSubscribe()")
disposable = d


override fun onNext(t: Pair<SiteDataController.ProcessedSite, UIController.DisplayFields>)
// use result to build out ui


override fun onError(e: Throwable)
e.printStackTrace()

)



This gets set up in the fragment's onResume(). Also, "continueUpdates" gets set to true in onResume() and false in onPause(). The roots of both Observables that make up the combineLatest() stream are BehaviorSubjects. But despite the takeWhile condition being switched to false, and even despite calling onComplete() on both BehaviorSubjects, the onComplete() on the observer never gets called. Am I doing something wrong?





You seem to be recreating this flow over and over. You should be disposing the disposable in onPause to avoid any memory leak. Also the predicate of takeWhile gets evaluated only when an item passes through so setting that reference to false outside of such invocation has no "completion" effect.
– akarnokd
Aug 12 at 18:39


disposable


takeWhile





Yes, forgot to mention, I am calling dispose on the disposable, I only removed it so I could ensure onComplete being called. But will simply calling dispose() suffice? Does the observable even have any knowledge of what observers it has, and will it know to clear itself up when it has no more observers?
– KairisCharm
Aug 12 at 18:41






Yes, the chain cleans up after itself upon termination or disposing it. However, you should also "forget" any references to it, including the disposable.
– akarnokd
Aug 12 at 18:45





Ok, by "forget," you mean: disposable?.dispose() disposable = null
– KairisCharm
Aug 12 at 18:46



disposable?.dispose()


disposable = null





Yes, if you track them individually, null the references out (and ?. is advised) or if you collect them into a ComposideDisposable, clear the composite.
– akarnokd
Aug 12 at 18:48


ComposideDisposable









By clicking "Post Your Answer", you acknowledge that you have read our updated terms of service, privacy policy and cookie policy, and that your continued use of the website is subject to these policies.

-

Popular posts from this blog

Firebase Auth - with Email and Password - Check user already registered

Image
Clash Royale CLAN TAG #URR8PPP Firebase Auth - with Email and Password - Check user already registered I want to check when a user attempts to signup with createUserWithEmailAndPassword() in Firebase user Authentication method, this user is already registered with my app. createUserWithEmailAndPassword() 8 Answers 8 To detect whether a user with that email address already exists, you can detect when the call to createUserWithEmailAndPassword () fails with auth/email-already-in-use . I see that @Srinivasan just posted an answer for this. createUserWithEmailAndPassword () auth/email-already-in-use Alternatively, you can detect that an email address is already used by calling fetchSignInMethodsForEmail() (previous called fetchProvidersForEmail() ). fetchSignInMethodsForEmail() fetchProvidersForEmail() When the user trying to create an user with same email address, the task response will be "Response: The email address is already in use by another account." mFirebas
Read more

Dynamically update html content plain JS

Image
Clash Royale CLAN TAG #URR8PPP Dynamically update html content plain JS So I have a script bound in my index.html, which literally has to fetch an array of jsons from an api, then dynamically update my page when get response. But as a result I'm getting an as the only update of my page. here is my script fetch(url) .then((resp) => resp.json()) .then((resp) => resp.results.forEach(item => fetch(item.url) .then((response)=> response.json()) .then((response) => pokeArr.push(response)) .catch(console.log) ) ).catch(console.log) const filler = () => if(!pokeArr) return 0 pokeArr.map((i,j)=> return `<tr>$i.name</tr>` ) const pokeindex = () => document.getElementById('a').appendChild(document.createElement(filler())) pokeindex() When I'm consoling it, I can see in the console all the responses I get, so I'm at least doing the fetching part right. What's parberakan ? What's pokeArr ? – T.J. Crowder Aug 8 at 12
Read more

How to determine optimal route across keyboard

Image
Clash Royale CLAN TAG #URR8PPP How to determine optimal route across keyboard I'm trying to automate the typing process in a video game. Players are given an on-screen QWERTY keyboard layout that they can navigate with the control stick (up, down, left, right) in order to select a letter to type. I was wondering if there was a module for Python (or some other resource I could explore) that would help my program find a path across the keyboard as it types each letter in a given message. In other words, if the first letter typed was "A" and the next letter was "B", the module would know that the program should move 1 space down on the keyboard and 4 spaces to the right. Currently, I have the program resetting to the "G" key after each letter is typed (allowing me to use fixed-directions as the program parses each character in the desired message). As you can imagine, this is quite inefficient and defeats the purpose of automating the typing in the fir
Read more