Где отменить регистрацию BroadcastReceiver (в сервисе)?

Question

У меня есть BroadcastReceiver, который я динамически создаю в Сервисе.Его цель - определить, когда было отправлено SMS.Он работает, как и ожидалось.

Моя проблема в том, что я получаю сообщение об ошибке "Утечка получателя намерений".Я пропускаю звонок на unregisterReceiver()?

Я звоню на unregisterReceiver() в onDestroy().Я думаю, это должно быть неправильно.Где я должен отменить регистрацию получателя?

Код ...

public class MyService extends Service {
    BroadcastReceiver brSms;

    @Override
    public IBinder onBind(Intent intent) {
        return null;
    }

    @Override
    public void onCreate() {
    }

    @Override
    public void onStart(Intent intent, int startid) {
        //Define the receiver
        brSms = new BroadcastReceiver() {
            @Override
            public void onReceive(Context context, Intent intent) {
                //Do some  stuff
            };
        }

        registerReceiver(brSms, new IntentFilter(SENT_SMS_ACTION));
    }

    @Override
    public void onDestroy() {
        unregisterReceiver(brSms);
        Toast.makeText(this, "onDestroy has been called", Toast.LENGTH_SHORT).show();
    }
}//end of MyService

Answer 1

Попробуй так:

public class MyService extends Service {
    BroadcastReceiver brSms;

    @Override
    public IBinder onBind(Intent intent) {
        return null;
    }

    @Override
    public void onCreate() {
        brSms = new BroadcastReceiver() {
            @Override
            public void onReceive(Context context, Intent intent) {
                //Do some  stuff
            }
        };

        registerReceiver(brSms, new IntentFilter(SENT_SMS_ACTION));
    }

    @Override
    public void onStart(Intent intent, int startid) {
    }

    @Override
    public void onDestroy() {
        unregisterReceiver(brSms);
        Toast.makeText(this, "onDestroy has been called", Toast.LENGTH_SHORT).show();
    }
}

Answer 2

Я не знаю, что является причиной вашего сообщения об ошибке, но чтобы ответить на ваш вопрос, вы делаете это абсолютно правильно, чтобы отменить регистрацию в onDestroy(). По опыту я знаю, что это работает хорошо, и это также хорошее место, чтобы сделать это в соответствии с документацией для Service.onDestroy () :

... Служба должна очистить все ресурсы, которые она содержит (потоки, зарегистрированные получатели и т. Д.) На этом этапе. ...

Answer 3

Избегайте onStart() ... попробуйте onStartCommand(Intent intent, int flags, int startId)...

Answer 4

Вам нужно вызвать super.onDestroy ().

@Override
public void onDestroy() {
    unregisterReceiver(brSms);
    Toast.makeText(this, "onDestroy has been called", Toast.LENGTH_SHORT).show();

    super.onDestroy();
}

Надеюсь, это сработает для вас.