4

Tengo una clase extendida de BroadcastReceiver en MainActivity.java lanzo el listener

registerReceiver(
        new ConnectivityChangeReceiver(),
        new IntentFilter(
                ConnectivityManager.CONNECTIVITY_ACTION));

Me encuentro que cuando salgo de la app me muestra el siguiente error en el log:

E/ActivityThread: Activity com.webserveis.app.myconnection.MainActivity has leaked IntentReceiver com.webserveis.app.myconnection.ConnectivityChangeReceiver@32ad932e that was originally registered here. Are you missing a call to unregisterReceiver()? android.app.IntentReceiverLeaked: Activity com.webserveis.app.myconnection.MainActivity has leaked IntentReceiver com.webserveis.app.myconnection.ConnectivityChangeReceiver@32ad932e that was originally registered here. Are you missing a call to unregisterReceiver()?

that was originally registered here. Are you missing a call to unregisterReceiver()

En onStop() me gustaria desregistrar el listener, pero no ser como implementarlo

1
  • 1
    desregistrar el receiver en onStop() funciona pero en realidad no es buena practica ya que siempre estaras recibiendo intents =(, agregue una respuesta.
    – Jorgesys
    Commented el 29 jun. 2016 a las 15:07

2 respuestas 2

5

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 en Activity.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 en onResume() de su actividad, se debe cancelar el registro en el método onPause().

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.

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);
  }
}
3
  • Muy buenas Elena, si uso onResume y onPause, si se muestra otra app encima de la mia, se para el listener?
    – Codelaby
    Commented el 29 jun. 2016 a las 15:11
  • 1
    Buen día Webserveis. Se debe cancelar el receiver ya que esta en onPause() tu aplicación .
    – Jorgesys
    Commented el 29 jun. 2016 a las 15:15
  • 1
    Gracias, funciona perfectamente.
    – arxzel
    Commented el 30 nov. 2019 a las 16:29
2

Deberías de guardar tu BroadcastReceiver en una variable de tu MainActivity y en tu onStop() poner unregisterReceiver(myBroadcastReceiver);

Debes de tener en cuenta que si pasases por el onStop()sin haber registrado el BroadcastReceiver previamente, te saltará una excepción.

Un saludo

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.