Как обрабатывать AsyncTask onPostExecute при паузе, чтобы избежать IllegalStateException

Я ценю многочисленные сообщения об AsyncTask об изменении ротации. У меня есть следующая проблема при использовании библиотеки совместимости и пытается отклонить DialogFragment в onPostExecute,

У меня есть фрагмент, который запускает AsyncTask, который отображает прогресс DialogFragment, затем в onPostExecute закрывает диалог, а затем потенциально выкидывает другой DialogFragment,

Если, когда отображается диалоговое окно прогресса, я помещаю приложение в фоновый режим, я получаю следующее для своего фрагмента:

1) onPause

2) onSaveInstanceState

3) onPostExecute в котором я пытаюсь отклонить и вызвать диалог.

Я получаю IllegalStateException потому что я пытаюсь эффективно зафиксировать транзакцию, когда активность сохранила свое состояние, и я это понимаю.

На повороте я предположил (возможно, неправильно), что я не получу onPostExecute пока деятельность не была воссоздана. Однако, помещая приложение в фоновый режим, я предположил (определенно неправильно), что onPostExectute не будет вызван, пока фрагмент / активность приостановлен.

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

Заранее спасибо, Питер.

Редактировать 1

Нужно поддерживать 2.1 и выше

Редактировать 2

Я рассмотрел показ диалога, используя FragmentTransaction:add а также FragmentTransaction:commitAllowingStateLossОднако это не без проблем.

5 ответов

Решение

Если вам нужно синхронизировать свою задачу с жизненным циклом активности, я считаю, что загрузчики - это именно то, что вам нужно. В частности, вы должны использовать AsyncTaskLoader для выполнения этой работы. Так что теперь вместо запуска AsyncTask вы запускаете загрузчик, а затем ждете ответа в слушателе. Если действие приостановлено, вы не получите обратный вызов, эта часть будет управляться для вас.

Есть еще один способ справиться с этой задачей: использовать фрагмент, который сохраняет свой экземпляр. Общая идея заключается в том, что вы создаете фрагмент без интерфейса и вызываете setRetainInstance(true), У него есть задача, которая уведомляется о том, доступна ли операция или нет. Если нет, поток задачи приостанавливается, пока действие не станет доступным.

Еще один способ добиться того, что вам нужно, - реализовать класс PauseHandler, который я описал в этом посте.

Затем в вашем методе onPostExecute вызовите sendMessage() для отправки вашего сообщения в обработчик.

Когда ваше приложение возобновится, действие будет обработано.

Вместо того, чтобы использовать BroadcastReceiver, я предпочитаю использовать библиотеки шин, такие как guava, otto или eventbus. Их производительность намного лучше, чем у вещательного приемника.

Я нашел решение этой проблемы без какого-либо серьезного обходного пути: основная идея о том, как поддерживать диалоги выполнения и асинхронную задачу, описана в этом блогерстве (конечно, я использовал AsyncTaskComplex-Version). Все кредиты идут автору этого блогентства, я только добавил крошечную вещь:

Очевидно, я больше не использую showDialog (). Вместо этого я придерживаюсь DialogFragments.

Второй твик является важным и также решает проблему с IllegalStateException:

Вместо того, чтобы только сказать асинхронной задаче в onRetainCustomNonConfigurationInstance(), что больше нет активности, я также делаю это в onPause(). И вместо того, чтобы только сообщать асинхронной задаче в onCreate(), что есть новое действие, я также делаю это в onResume().

И все, ваш AsyncTask не будет пытаться проинформировать вашу активность о своем завершении, вызывая исключение IllegalStateException, когда активность не видна.

Если вы хотите увидеть больше кода вместо слов, оставьте комментарий.

/ edit: Исходный код, чтобы показать мое решение, которое я считаю довольно приличным:)

public class MyActivity extends Activity {

private MyTask mTask;

@Override
protected void onCreate(Bundle pSavedInstanceState) {
    super.onCreate(pSavedInstanceState);
    setContentView(R.layout.editaccount);

    Object retained = getLastCustomNonConfigurationInstance();
    if ( retained instanceof NewContactFolderIdTask ) {
        mTask = (MyTask) retained;
        mTask.setActivity(this);
    }

}
@Override
protected void onPause() {
    if(mTask != null) {
        mTask.setActivity(null);
    }
    super.onPause();
}

@Override
public Object onRetainCustomNonConfigurationInstance() {
    if(mTask != null) {
        mTask.setActivity(null);
        return mTask;
    }
    return null;
}

@Override
protected void onResume() {
    if(mTask != null) {
        mTask.setActivity(this);
    }
    loadValues(); // or refreshListView or whatever you need to do
    super.onResume();
}

public void onTaskCompleted() {
    loadValues();  // or refreshListView or whatever you need to do
    DialogFragment dialogFragment = (DialogFragment) getSupportFragmentManager().findFragmentByTag(PROGRESS_DIALOG_FRAGMENT);
    if(dialogFragment != null) {
        dialogFragment.dismiss();
    }
}

@Override
public boolean onCreateOptionsMenu(Menu menu) {
    MenuInflater menuInflater = getMenuInflater();
    menuInflater.inflate(R.menu.main, menu);
    return super.onCreateOptionsMenu(menu);
}
@Override
public boolean onOptionsItemSelected(MenuItem item) {
    switch (item.getItemId()) {
        case android.R.id.home:
            // app icon in Action Bar clicked; go home
            Intent intent = new Intent(this, OXClient.class);
            intent.addFlags(Intent.FLAG_ACTIVITY_CLEAR_TOP);
            startActivity(intent);
            return true;
        case R.id.menu_refresh:
            mTask = new MyTask(this);
            mTask.execute();
            break;
    }
    return super.onOptionsItemSelected(item);
}


private class NewContactFolderIdTask extends AsyncTask<Boolean, Integer, Bundle> {
    private MyActivity mActivity;
    private boolean mCompleted;

    private NewContactFolderIdTask(MyActivity pActivity) {
        this.mActivity = pActivity;
    }

    public void setActivity(MyActivity pActivity) {
        this.mActivity = pActivity;
        if(mCompleted) {
            notifiyActivityTaskCompleted();
        }
    }

    private void notifiyActivityTaskCompleted() {
        if(mActivity != null) {
            mActivity.onTaskCompleted();
        }
    }

    @Override
    protected Bundle doInBackground(Boolean... pBoolean) {
        // Do your stuff, return result
    }

    @Override
    protected void onPreExecute() {
        DialogFragment newFragment = ProgressDialogFragment.newInstance();
        newFragment.show(getSupportFragmentManager(), PROGRESS_DIALOG_FRAGMENT);
    }

    @Override
    protected void onPostExecute(Bundle pResult) {
        mCompleted = true;
        notifiyActivityTaskCompleted();
    }
}

}

В разделе Как обрабатывать сообщения обработчика, когда действие / фрагмент приостановлен, я предлагаю другой подход с использованием BroadcastReceiver.

Я считаю, что он чище и элегантнее, и он предлагает преимущества, которые вы можете вызывать из базового фрагмента кода из любой точки вашего приложения, и с помощью липких трансляций ваш вызов можно "запомнить" и выполнить после возобновления фрагмента.

Другие вопросы по тегам