按下后退按钮时 Android runOnUiThread 崩溃

Question

我正在使用 DownloadManager 从服务器下载文件，我正在从适配器执行此操作，并且我正在使用 runOnUiThread 在 fragment 中显示下载进度。它运行良好，但是当我在下载时按下后退按钮时，runOnUiThread 抛出 NullPointerException。我尝试使用 if 语句在 onPause 中放置一个变量来停止该方法，但它也不起作用。

您建议如何解决此崩溃问题？

这是我的代码:

new Thread(new Runnable() {
                        @Override
                        public void run() {

                            downloading = true;

                            while (downloading) {

                                DownloadManager.Query q = new DownloadManager.Query();
                                q.setFilterById(myDownloadReference);

                                Cursor cursor = downloadManager.query(q);
                                cursor.moveToFirst();
                                int bytes_downloaded = cursor.getInt(cursor
                                        .getColumnIndex(DownloadManager.COLUMN_BYTES_DOWNLOADED_SO_FAR));
                                int bytes_total = cursor.getInt(cursor.getColumnIndex(DownloadManager.COLUMN_TOTAL_SIZE_BYTES));

                                if (cursor.getInt(cursor.getColumnIndex(DownloadManager.COLUMN_STATUS)) == DownloadManager.STATUS_SUCCESSFUL) {
                                    downloading = false;
                                    downloadingDocs.remove(doc);

                                    //remove the progressBar when download is complete
                                    fragment.getActivity().runOnUiThread(new Runnable() {
                                        @Override
                                        public void run() {
                                            finalHolder2.downloadProgress.setVisibility(View.INVISIBLE);
                                        }
                                    });

                                    //check if it's the last download and open it.
                                    if (downloadingDocs.size() == 0) {
                                        actionIntentOpenDocs(doc, saveFile);
                                    }
                                }
                                //final double dl_progress = (bytes_downloaded / bytes_total) * 100;
                                final int dl_progress = (int) ((bytes_downloaded * 100l) / bytes_total);

                                //display the download progress
                                fragment.getActivity().runOnUiThread(new Runnable() {
                                    @Override
                                    public void run() {
                                        if(Constants.stopUiThreadWork == null) {
                                            finalHolder2.downloadProgress.setProgress((int) dl_progress);
                                        }
                                    }
                                });
                                cursor.close();
                            }
                        }
                    }).start();

它在这条线上崩溃了:

fragment.getActivity().runOnUiThread(new Runnable() {

崩溃堆栈跟踪:

FATAL EXCEPTION: Thread-4807
Process: com.appus.app, PID: 6553
java.lang.NullPointerException: Attempt to invoke virtual method 'void android.support.v4.app.FragmentActivity.runOnUiThread(java.lang.Runnable)' on a null object reference
        at com.availo.app.UI.adapters.MainFragmentDocumentAdapter$1$1.run(MainFragmentDocumentAdapter.java:265)
        at java.lang.Thread.run(Thread.java:818)

Answer 1

在像这样执行 runOnUiThread 之前尝试检查您的 Activity :

    /** check if activity still exist */
    if (getActivity() == null) {
        return;
    }

    getActivity().runOnUiThread(new Runnable() {
    ...
    });

“您不应该在您的代码中散布返回值吗？(...)”
“更清洁”是它变得模棱两可的地方。我们不是在编写汇编程序，所以我看不出这与这里有什么关系。苹果和橙子。在编写方法时，尽可能使用单个返回可以提高可读性和维护性。如果您的代码变得复杂或需要遵循此编码风格的多个 return 语句，则该方法可能需要进一步分析和分解。 – 朱利奥

@giulio - 引用资料、出版物、规范 - 我在你的评论中没有看到任何东西 - 但我可以给你一些阅读和获得一些知识。

你可以从这里开始让你大吃一惊:

• https://softwareengineering.stackexchange.com/questions/118703/where-did-the-notion-of-one-return-only-come-from

然后阅读我最喜欢的:

http://www.yegor256.com/2015/08/18/multiple-return-statements-in-oop.html