Cuando se implementa un BroadcastReceiver, es importante saber que tenemos que registrar y desregistrar el receiver., pero NO es buena practica cancelar el receiver en onStop() y esta es la razón, la cual puedes ver en la documentación:
Cuando tu implementación requiere el registro en onResume()
existe una consideración importante.
Si está registrando un receiver en
Activity.onResume()
, debes anular el registro enActivity.onPause()
. (No recibirá intents cuando este en pausa, y esto puede reducir innecesaria sobrecarga del sistema)
###En cuanto a tu error,:
MainActivity has leaked IntentReceiver
la documentación misma lo señala :
No olvides cancelar el registro de receiver registrado en forma dinámica mediante el método
Context.unregisterReceiver()
. Si se olvida de esto, el sistema Android reporta un Leaked error. Por ejemplo, si registraste un receiver enonResume()
de su actividad, se debe cancelar el registro en el métodoonPause()
.
Por lo tanto :
En el método onResume() generalmente se realiza el registro del receiver.
En el método onPause() generamente se realiza el desregistro del receiver.
NO es buena practica cancelar el receiver en onStop() y esta es la razón, la cual puedes ver en la documentación:
Cuando tu implementación requiere el registro en onResume()
existe una consideración importante.
Si está registrando un receiver en
Activity.onResume()
, debes anular el registro enActivity.onPause()
. (No recibirá intents cuando este en pausa, y esto puede reducir innecesaria sobrecarga del sistema)
Agrego un ejemplo:
public class MyActivity extends Activity
{
private final BroadcastReceiver mybroadcast = new BroadcastReceiver ();
//REGISTRA!
public void onResume()
{
IntentFilter filter = new IntentFilter();
filter.addAction("android.provider.Telephony.SMS_RECEIVED");
registerReceiver(mybroadcast, filter);
}
//CANCELA!
public void onPause()
{
unregisterReceiver(mybroadcast);
}
}