Handling onNewIntent in Fragment

Question

I am writing an application that uses NFC to read some data stored on it. My application uses Fragments and Fragment don\'t come with onNewIntent() method. Since, the data I

Answer 1

There is at least one alternative: From Activity.onNewIntent documentation:

An activity will always be paused before receiving a new intent, so you can count on onResume() being called after this method.

Note that getIntent() still returns the original Intent. You can use setIntent(Intent) to update it to this new Intent.

FragmentActivity.onNewIntent documentation is different but I don't think it contradicts the above statements. I also make the assumption that Fragment.onResume will be called after FragmentActivity.onResume, even though the documentation seems a little fussy to me, though my tests confirm this assumption. Based on this I updated the Intent in the activity like so (examples in Kotlin)

override fun onNewIntent(intent: Intent?) {
    setIntent(intent)
    super.onNewIntent(intent)
}

And in Fragment.onResume I could handle the new intent like so

override fun onResume() {
    super.onResume()
    doStuff(activity.intent)
}

This way the activity don't need to know about what fragments it holds.

Answer 2

This is an old question, but let me answer it in case anybody bumps into it.

First of all you have a bug in your code:

You can't register Fragments as listeners inside Activity the way you do it. The reason is that Activity and Fragments can be destroyed by the system and re-created later from saved state (see documentation on Recreating an Activity). When this happens, new instances of both the Activity and the Fragment will be created, but the code that sets the Fragment as a listener will not run, therefore onNewBalanceRead() will never be called. This is very common bug in Android applications.

In order to communicate events from Activity to Fragment I see at least two possible approaches:

Interface based:

There is an officially recommended approach for communication between Fragments. This approach is similar to what you do now in that it uses callback interfaces implemented by either Fragment or Activity, but its drawback is a tight coupling and lots of ugly code.

Event bus based:

The better approach (IMHO) is to make use of event bus - "master component" (Activity in your case) posts "update" events to event bus, whereas "slave component" (Fragment in your case) registers itself to event bus in onStart() (unregisters in onStop()) in order to receive these events. This is a cleaner approach which doesn't add any coupling between communicating components.

All my projects use Green Robot's EventBus, and I can't recommend it highly enough.

Answer 3

No, there is no better way. Fragments can live longer than Activities and are not necessarily tied to them at all so providing new intents would not make sense.

Btw, you have a few bugs in your code :)

if (actionBar.getSelectedNavigationIndex() != 1) {

Magic numbers are bad! use a constant.

    if (myBalanceBundle == null)
        myBalanceBundle = new Bundle();

    myBalanceBundle.putString(Keys.BALANCE.toString(), balance);
    actionBar.setSelectedNavigationItem(1);

we already know that the navigationitem is set to 1

} else {
    newBlanceListener.onNewBalanceRead(balance);

Add a null check. The user might have never selected a navigation item.