ProgressDialog не отображается, когда AsyncTask.get () называется [дубликат]

17
  

Возможный дубликат:
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 секунды)

В чем проблема?     

задан Ogulcan Orhan 26.01.2012 в 15:04
источник

4 ответа

23

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

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

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

    
ответ дан rds 26.01.2012 в 15:24
источник
14

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

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

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

    
ответ дан Jake Wharton 26.01.2012 в 15:23
источник
4

Если я правильно понял ваш вопрос, вам нужно обновить ход вашей 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();
  }
}
    
ответ дан Marvin Pinto 26.01.2012 в 15:16
источник
-2

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

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

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

    
ответ дан James andresakis 26.01.2012 в 15:12
источник