Question

我正在使用DownloadManager从服务器下载文件，我是从适配器执行的，并且我使用runOnUiThread显示片段中的下载进度。它工作得很好，但是当我在下载时按下后退按钮时，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之前尝试检查您的活动：

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

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

“你不应该通过代码散布回报吗？（...）”
“清洁”是它变得含糊不清的地方。我们不是在编写汇编程序，所以我不知道这是如何相关的。苹果和橘子。编写方法时，尽可能单一返回可提高可读性和维护性。如果您的代码变得复杂或者需要遵循这种编码风格需要多个return语句，那么该方法可能需要进一步分析和细分。 - giulio

@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

Answer 2

管理此类事情的最佳方式是Service。原因是服务可以更好地管理后台线程，即使活动没有运行，而AsyncTask之类的东西将在活动完成后停止工作。请参阅this site了解如何设置下载服务。

按“返回”按钮时Android runOnUiThread崩溃

2 个答案: