从OnPostExecute更新UI(异步)

时间:2016-02-23 18:06:39

标签: android android-asynctask android-4.2-jelly-bean

我有一个代表迷宫的数组。在UI上,迷宫表示为按钮的行和列。在异步任务的doInBackground方法中,我搜索了一个路径,并使用指向目标的路径初始化解决方案数组。我要做的是更新这些按钮的按钮文本以显示通向目标的路径。我在OnPostExecute中做这个。但是,它不起作用。它甚至不执行启用解决方案按钮的最后一行。我在哪里做什么?

private void updateUI() {
    Button cell;
    TableRow row;

    do {
        row = (TableRow) (State.maze.getChildAt(solution.row));
        cell = (Button) (row.getChildAt(solution.col));
        cell.setText(State.pathCell);
        try {
            Thread.sleep(100);
        } catch (InterruptedException e) {
            e.printStackTrace();
        }
        solution = solution.next;
    } while (solution.next != null);
}

@Override
protected void onPostExecute(String result) {
    super.onPostExecute(result);
    updateUI();
    //Enable solution button
    State.solveResetButton.setEnabled(true);

}

更新: 我检查确保解决方案变量包含一个解决方案,它确实包含有效数据。我也尝试删除睡眠,但无济于事。

更新: Logcat输出(红色的那些)

 02-23 13:09:15.471 4640-4640/? E/Zygote: MountEmulatedStorage()
 02-23 13:09:15.471 4640-4640/? E/Zygote: v2
 02-23 13:09:15.471 4640-4640/? E/Zygote: accessInfo : 0
 02-23 13:09:15.471 4640-4640/? E/SELinux: [DEBUG] get_category: variable seinfo: default sensitivity: NULL, cateogry: NULL

1 个答案:

答案 0 :(得分:1)

循环时,solution.next的值不会更新。这导致无限循环,因此updateUI()方法永远不会停止执行。

此外,onPostExecute作为AsyncTask的一部分始终在主线程上调用。您不希望在主线程上使用Thread.sleep。这是获得ANR的好方法。

关于如何实现这一目标,有许多方法可行。使用Handler作为主线程来考虑这样的事情:

private void updateUI() {
    Button cell;
    TableRow row;

    if (solution != null){
        row = (TableRow) (State.maze.getChildAt(solution.row));
        cell = (Button) (row.getChildAt(solution.col));
        cell.setText(State.pathCell);
        solution = solution.next;
        new Handler(Looper.getMainLooper()).postDelayed(new Runnable() {
            @Override
            public void run() {
                updateUI(); 
            }
        }, 100);
    }
}

您也可以使用publishProgress使用新的AsyncTask来回调UI线程

private void updateUI(){
    new AsyncTask<Solution, Solution, Void>(){
        @Override
        protected Void doInBackground(Solution... solutions) {
            Solution solution = solutions[0];
            do {
                publishProgress(solution);
                try {
                    Thread.sleep(100);
                } catch (InterruptedException e) {
                    e.printStackTrace();
                }
                solution = solution.next;
            }
            while(solution.next != null);
            return null;
        }

        @Override
        protected void onProgressUpdate(Solution... values) {
            super.onProgressUpdate(values);
            Solution solution = values[0];
            row = (TableRow) (State.maze.getChildAt(solution.row));
            cell = (Button) (row.getChildAt(solution.col));
            cell.setText(State.pathCell);
        }

        @Override
        protected void onPostExecute(Void aVoid) {
            super.onPostExecute(aVoid);
            //Enable solution button
            State.solveResetButton.setEnabled(true);
        }
    }.execute(solution);
}

@Override
protected void onPostExecute(String result) {
    super.onPostExecute(result);
    updateUI();
}