无法在Android中取消注册BroadcastReceiver

Question

I've already seen How to unregister BroadcastReceiver but it doesn't solve my problem.

On an Android Application, I register a BroadCastReceiver in the onResume() method, and I want to unregister it on the onPause() method

@Override
protected void onResume() {
    super.onResume();

    // TODO:
    // Register the BroadcastReceiver to receive a 
    // DATA_REFRESHED_ACTION broadcast
    log("RefreshReceiver registrao");
    registerReceiver(mRefreshReceiver, new   IntentFilter(DATA_REFRESHED_ACTION));  
}

@Override
protected void onPause() {
    // TODO:
    // Unregister the BroadcastReceiver
    unregisterReceiver(mRefreshReceiver); //fails here
    Log.i("QUE PASA PEñAAA","desregitro el receptor");
    super.onPause();
}

the logcat message says it can't unregister the method. Btw.:

public static final String DATA_REFRESHED_ACTION = "course.labs.notificationslab.DATA_REFRESHED";

Thanks for everything

Answer 1

Your code looks fine, but you should move super.onPause(); to top of onPause - I am not sure if this is the cause of your problems.

Assuming you are not unregistering receiver somewhere else, you could keep some flag whether your receiver was already registered or not, follow instructions from this SO: Correct pattern for registering a receiver?

Also you should swith to use LocalBroadcastManager which is better suited for local app broadcasts: http://www.vogella.com/tutorials/AndroidBroadcastReceiver/article.html#ownreceiver_localbroadcastmanager

---

[edit]

from your logcat:

java.lang.IllegalArgumentException: Receiver not registered: null

it looks like you have not initialized mRefreshReceiver and it is null, you should call:

mRefreshReceiver = new RefreshReceiver();

before calling register, or if you have such code then make sure your are not assigning null to mRefreshReceiver somewhere.