2

我遇到了一个奇怪的问题,我不确定我是不是做错了什么。出于某种原因,我的 AsyncTask 不会调用 doInBackground,但会调用 onPostExecute()。调用此 AsyncTask 至关重要,因为它初始化了我在应用程序的其余部分中将需要的变量。

我的主要活动中有一个嵌套类,它扩展了 AsyncTask。此类负责从用户的 Dropbox 帐户下载文件名:

protected class FilesLister extends AsyncTask<String, Integer, Long>
{
    @Override
    protected Long doInBackground(String... params)
    {
        Log.i("Entries", "We believe Entries has some values now.");
        ArrayList<Entry> theFiles = new ArrayList<Entry>();
        try
        {
            Entry entries = mDBApi.metadata("/", 20000, null, true, null);
            for (Entry e : entries.contents)
            {
                Log.i("Hm", e.fileName());
                if (!e.isDeleted)
                {
                    if(!e.isDir)
                    {
                        theFiles.add(e);
                    }
                }
            }
            theEntries = theFiles.toArray(new Entry[theFiles.size()]);
        } catch (DropboxException e1)
        {
            // TODO Auto-generated catch block
            e1.printStackTrace();
        }
        return null;
    }

    protected void onPostExecute(Long result)
    {
        super.onPostExecute(result);
        Log.i("Now", "Calling refresh.");
        refresh();
    }
}

如您所见,onPostExecute 调用了一个名为 refresh() 的方法。refresh() 实现如下:

public void refresh()
{
    //Sort all this files by last modified date.
    Arrays.sort(theEntries, new Comparator<Entry>()
    {
        @Override
        public int compare(Entry firstFile, Entry secondFile)
        {
            //"EEE, dd MMM yyyy kk:mm:ss ZZZZZ"
            SimpleDateFormat formater = new SimpleDateFormat("EEE, dd MMM yyyy kk:mm:ss ZZZZZ");
            try
            {
                Date date1 = (Date)formater.parse(firstFile.modified);
                Date date2 = (Date)formater.parse(secondFile.modified);
                return date1.compareTo(date2);
            } catch (ParseException e1)
            {
                e1.printStackTrace();
            }
            return 0;
        }
    });
    //Now we create a String[] array to hold the names of all the fetched files...
    ArrayList<String>txtFilesNames = new ArrayList<String>();
    for(int i = theEntries.length - 1; i >= 0; i--)
    {
        //Loops goes from top to bottom so the latest file appears at the top of the ListView.
        txtFilesNames.add(theEntries[i].fileName());
    }
    actualFileNames = txtFilesNames.toArray(new String[txtFilesNames.size()]);
    ArrayAdapter<String> ad = new ArrayAdapter<String>(this, android.R.layout.simple_list_item_1, actualFileNames);
    lv.setAdapter(ad);
}

我知道调用 refresh() 有两个原因: LogCat 记录“调用刷新”。然后应用程序由于 NullPointerException 而崩溃。抛出空指针异常是因为 theEntries 确实为空,除非调用 doInBackground(),否则它将为空。而且我知道 doInBackground 永远不会因为空指针而被调用,因为我放在它的主体上的 Log 永远不会被调用。那么是什么导致我的 doInBackground() 方法没有被调用呢?如果重要,我会在 onResume 方法中执行 AsyncTask,并且不会在 onCreate 或 onStart 中执行任何其他 AsyncTask。

4

3 回答 3

1

你确定 doInBackground 没有被调用吗?还有很多其他方法可能导致 theEntries 为空。例如,这里有一个例外:

        Entry entries = mDBApi.metadata("/", 20000, null, true, null);
        for (Entry e : entries.contents)
        {
            Log.i("Hm", e.fileName());
            if (!e.isDeleted)
            {
                if(!e.isDir)
                {
                    theFiles.add(e);
                }
            }
        }

可能导致 theEntries 为空。另外,我认为您可以以更好的方式做到这一点。与其在 doInBackground 中设置 theEntries(与创建对象的线程不同),不如让该函数返回将它们传递给 onPostExecute 的条目。我相信代码如下所示:

protected class FilesLister extends AsyncTask<String, Integer, Entry[]>
{
    @Override
    protected Entry[] doInBackground(String... params)
    {
        Log.i("Entries", "We believe Entries has some values now.");
        Entry[] entriesReturn = null;
        ArrayList<Entry> theFiles = new ArrayList<Entry>();
        try
        {
            Entry entries = mDBApi.metadata("/", 20000, null, true, null);
            for (Entry e : entries.contents)
            {
                Log.i("Hm", e.fileName());
                if (!e.isDeleted)
                {
                    if(!e.isDir)
                    {
                        theFiles.add(e);
                    }
                }
            }
            entriesReturn = theFiles.toArray(new Entry[theFiles.size()]);
        } catch (DropboxException e1)
        {
            // TODO Auto-generated catch block
            e1.printStackTrace();
        }
        return entriesReturn;
    }

    protected void onPostExecute(Entry[] result)
    {
        super.onPostExecute(result);
        Log.i("Now", "Calling refresh.");
        refresh(result);
    }
}

然后在您的刷新方法中进行分配。您还应该在刷新方法中检查 null 。

于 2012-06-15T19:56:08.330 回答
0

此问题已由其他人发布(相关问题请参见侧边栏)。到目前为止,这个问题仍然没有解决方案。我自己也遇到过这个问题,经过几天的努力,我接受了这个错误。你可以做的是使用Activity.runOnUiThread(). 它将解决您遇到的这个问题,但不会解决AsyncTasks. 这就是我解决这个问题的方法。

于 2012-06-15T19:52:05.433 回答
0

您可以通过这种方式检查在 onPostExecute() 中是否调用了 doInBackground()。

 protected Long doInBackground(String... params)
    {
   /////your code...
        return 200;
    }
  protected void onPostExecute(Long result)
    {
       if(result==200)
{
        super.onPostExecute(result);
        Log.i("Now", "Calling refresh.");
        refresh();
}
    }

我不确定您的条目是否给出空值。通过这种方式,您可以知道 doInBackground() 是否执行。

我认为它提供了一些帮助。

谢谢。

于 2012-06-15T19:54:16.147 回答