Вещательный приемник IntentReceiverLeaked

Question

Я пытаюсь загрузить файл и после загрузки хочу обновить его в базе данных, но в моем намерении приемник службы вещания создает эту проблему:

android.app.IntentReceiverLeaked: Service com.dev.newtermain.services.intent_service.DownloadService has leaked IntentReceiver com.dev.newtermain.services.intent_service.DownloadService$DownloadCompleteReceiver@d4fdde9 that was originally registered here. Are you missing a call to unregisterReceiver()?
    at android.app.LoadedApk$ReceiverDispatcher.<init>(LoadedApk.java:969)
    at android.app.LoadedApk.getReceiverDispatcher(LoadedApk.java:731)
    at android.app.ContextImpl.registerReceiverInternal(ContextImpl.java:1179)
    at android.app.ContextImpl.registerReceiver(ContextImpl.java:1159)
    at android.app.ContextImpl.registerReceiver(ContextImpl.java:1153)
    at android.content.ContextWrapper.registerReceiver(ContextWrapper.java:554)
    at com.dev.newtermain.services.intent_service.DownloadService.onStartCommand(DownloadService.java:70)
    at android.app.ActivityThread.handleServiceArgs(ActivityThread.java:3228)
    at android.app.ActivityThread.-wrap17(ActivityThread.java)
    at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1591)
    at android.os.Handler.dispatchMessage(Handler.java:111)
    at android.os.Looper.loop(Looper.java:207)
    at android.app.ActivityThread.main(ActivityThread.java:5769)
    at java.lang.reflect.Method.invoke(Native Method)
    at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:789)
    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:679)

Вот мой сервисный код:

public class DownloadService extends IntentService {
 public DownloadService() {
    super("Download Service");
}

Long downloadId;
private DownloadManager downloadManager;

@Override public void onCreate() {
    super.onCreate();
    downloadManager = (DownloadManager) getSystemService(Context.DOWNLOAD_SERVICE);
    completeReceiver = new DownloadCompleteReceiver();
}

@Override public int onStartCommand(@Nullable Intent intent, int flags, int startId) {
    onHandleIntent(intent);
    this.intent = intent;
    return START_STICKY;
}

@Override
protected void onHandleIntent(Intent intent) {

    String key = intent.getStringExtra(DOWNLOAD_KEY);
    messageId = intent.getStringExtra(MESSAGE_ID);
    mReactionId = intent.getStringExtra(REACTION_ID);
    int messageType = intent.getIntExtra(MESSAGE_TYPE, 3);
    initDownload(key, messageType);
}

private void initDownload(String key, int messageType) {
    DownloadManager.Request request = new DownloadManager.Request(Uri.parse(url));
    request.addRequestHeader("Authorization", SharedPref.User.getToken());

    // In order for this if to run, you must use the Android 3.2 to
    // compile your app
    request.allowScanningByMediaScanner();
    request.setAllowedOverMetered(true);
    request.setNotificationVisibility(DownloadManager.Request.VISIBILITY_VISIBLE_NOTIFY_COMPLETED);
    request.setDestinationInExternalPublicDir(path, fileName);

    // Get download service and enqueue file
    downloadId = downloadManager.enqueue(request);

    Logs.e("file download", "downloading begin");
    IntentFilter intentFilter = new IntentFilter(DownloadManager.ACTION_DOWNLOAD_COMPLETE);
    this.registerReceiver(completeReceiver, intentFilter);
}

public class DownloadCompleteReceiver extends BroadcastReceiver {
    public DownloadCompleteReceiver() {
    }

    @Override public void onReceive(Context context, Intent intent) {
        Logs.e("file download", "broadcast received");
        DownloadManager.Query query = new DownloadManager.Query().setFilterById(downloadId);
        Cursor cursor = downloadManager.query(query);
        if (cursor.moveToFirst()) {
            int columnIndex = cursor.getColumnIndex(DownloadManager.COLUMN_STATUS);
            int status = cursor.getInt(columnIndex);
            cursor.close();

            switch (status) {
                case DownloadManager.STATUS_SUCCESSFUL:
                    Logs.e("file download", "download successful");
                    updateDb(fileName, messageId, Id);
                    break;
                case DownloadManager.STATUS_FAILED:
                    Logs.e("file download", "download Fail");
                    break;
                case DownloadManager.STATUS_PENDING:
                    Logs.e("file download", "STATUS_PENDING");
                    break;
                case DownloadManager.STATUS_RUNNING:
                    Logs.e("file download", "STATUS_RUNNING");
                    break;
            }
        }
    }
}

@Override public void onDestroy() {
    Logs.e("file download", "stopping service");
    this.unregisterReceiver(completeReceiver);
    downloadManager.remove(downloadId);
    super.onDestroy();
}

Как видно из документации приемника вещания, я отменяю регистрацию приемника вещания, но не понимаю, почему у меня возникает эта проблема.

EDIT

Я применил все предлагаемые ответы на новые вопросы. Проблема: если я добавлю широковещательный приемник, он даст успешный ответ, но не сохранит файл, но когда удалит файл широковещательного приемника, загрузка успешно

Answer 1

Просто замените свой код, изменив положение функции отмены регистрации вашего широковещания

@Override public void onDestroy() {
    downloadManager.remove(downloadId);
    this.unregisterReceiver(completeReceiver);
    super.onDestroy();
}

Answer 2

Проблема:

Причина, по которой вы столкнулись с этой проблемой, заключается в том, что IntentService запускается до завершения до завершения загрузки, а ваш DownloadCompleteReceiver все еще зарегистрирован.

downloadId = downloadManager.enqueue(request); не загружается синхронно. В результате IntentService выполняется до завершения.

Подход к решению

Если вам необходимо получить статус, вы можете запустить ForegroundService вместо IntentService и затем зарегистрировать получателя. В onDestory() вашей Службы отмените регистрацию получателя.