ProgressDialog не отображается при вызове AsyncTask.get()

Возможный дубликат:
AsyncTask блокирует пользовательский интерфейс и показывает панель прогресса с задержкой

Я хочу показать progressDialog при извлечении JSON с любого сервера. Поэтому я использовал AsyncTask в качестве решения (не уверен в другом выходе).

Все в порядке, ProgressDialog работает правильно, пока я не вызову метод .get(), используя экземпляр AsyncTask. Полагаю, что это как-то блокирует пользовательский интерфейс. Вот мой AsyncTask:

public class myAsync extends AsyncTask<String, String, List> {

    String message; // for dialog message
    ProgressDialog progress; 
    Intent myIntent;
    Context ctx;

    public myAsync(String message, Context ctx) {
        this.message = message;
        this.ctx = ctx;
        progress = new ProgressDialog(ctx);
    }

    @Override
    protected void onPreExecute() { 
        progress.setMessage(message);
        progress.setIndeterminate(true);
        progress.setCancelable(false);
        progress.show();    
    }

    @Override
    protected List doInBackground(String... params) {
        //returns any list after the task
        return anyList; 
    }

    @Override
    protected void onPostExecute(List result) {
        if(progress.isShowing())
            progress.dismiss();
    }
}

И вот myActivity, который вызывает AsyncTask:

myAsync asyncTask = new myAsync("Loading...", this);
asyncTask.execute("Any string", "Other string");
asyncTask.get(); // If I comment out this line, ProgressDialog works

После выполнения, когда я попытался записать результат из doInBackground и onPostExecute, проблема не возникает. Но если я хочу получить с .get() результат, ProgressDialog не отображается или показывается так мало времени (может быть, 0,2 секунды)

Какая проблема?

Ответ 1

Да, get() ждет, если это необходимо для завершения вычисления, а затем извлекает его результат. Это означает, что вы блокируете свой поток пользовательского интерфейса, ожидая результата.

Решение: не вызывайте get

Обычно вы вызываете функцию (обратный вызов) в postExecute.

Ответ 2

Вызов .get() изменяет ваш AsyncTask на эффективный "SyncTask", поскольку он вызывает текущий поток (который будет UI), чтобы дождаться завершения AsyncTask обработки. Поскольку теперь вы блокируете поток пользовательского интерфейса, вызов метода ProgressDialog .show() никогда не дает возможности разрешить диалоговому экрану сам экран.

Удаление вызова позволит ему правильно работать в фоновом режиме.

Если вам нужно выполнить обработку после завершения задачи, я предлагаю вам либо поместить ее внутри самого метода onPostExecute, либо использовать обратный вызов для Activity из onPostExecute.

Ответ 3

Если я правильно понял ваш вопрос, вам нужно обновить ход вашей AsyncTask в ProgressDialog, и это не работает в настоящее время. Итак, несколько замечаний: я не уверен, чего вы пытаетесь достичь с помощью .get(), но я предполагаю, что вы хотите отобразить прогресс.

Я изменил вашу программу ниже, чтобы обновить поток пользовательского интерфейса с вашим прогрессом AsyncTask. Каждый раз, когда вам нужно обновить ход выполнения, обновите переменную prog в методе doInBackground.

public class myAsync extends AsyncTask<String, Integer, List> {

  String message; // for dialog message
  ProgressDialog progress; 
  Intent myIntent;
  Context ctx;

  public myAsync(String message, Context ctx) {
    this.message = message;
    this.ctx = ctx;
    progress = new ProgressDialog(ctx);
  }

  @Override
  protected void onPreExecute() { 
    // Runs on the UI thread
    progress.setMessage(message);
    progress.setIndeterminate(true);
    progress.setCancelable(false);
    progress.show();    
  }

  @Override
  protected List doInBackground(String... params) {
    // Runs in the background thread
    // publish your progress here!!
    int prog = 5; // This number will represent your "progress"
    publishProgress(prog);
    return anyList; 
  }


  protected void onProgressUpdate(Integer... progress) {
    // Runs in the UI thread
    // This method will fire (on the UI thread) EVERYTIME publishProgress
    // is called.
    Log.d(TAG, "Progress is: " +progress);
  }

  @Override
  protected void onPostExecute(List result) {
    // Runs in the UI thread

    for (int i=0; i<result.size(); i++) {
      Log.d(TAG, "List item: " + result.get(i));
    }

    if(progress.isShowing())
      progress.dismiss();
  }
}

Ответ 4

Попробуйте использовать runOnUiThread следующим образом:

         runOnUiThread(new Runnable(){
            public void run() {
                dialog.show();
                }});

Запуск чего-то в AsyncTask означает, что он убегает от UIthread, поэтому обычно вы не можете запускать операции ui изнутри методов Async без обработчиков и т.д., от которых я обычно не веду себя. Я также обрабатываю такое решение, создавая progressDialog как переменную в моем классе выше моего oncreate, чтобы он был видимым для всего класса. Затем я вызываю progressdialog прямо перед моей асинхромой, а затем, поскольку его видимый для всего класса я вызываю .dissmiss() в onPostExecute