wake-up-neo.com

Android Fragmente. Beibehalten einer AsyncTask während der Bildschirmdrehung oder Konfigurationsänderung

Ich arbeite an einer Smartphone-/Tablet-App, benutze nur einen APK und lade je nach Bildschirmgröße Ressourcen. Die beste Design-Wahl schien Fragmente über die ACL zu verwenden.

Diese App hat bisher gut funktioniert, da sie nur auf Aktivitäten basiert. Dies ist eine Mock-Klasse, wie ich mit AsyncTasks und ProgressDialogs in den Aktivitäten umgehe, damit sie auch dann funktionieren, wenn der Bildschirm gedreht wird oder eine Konfigurationsänderung während der Kommunikation erfolgt.

Ich werde das Manifest nicht ändern, um die Neuerstellung der Aktivität zu vermeiden. Es gibt viele Gründe, warum ich es nicht tun möchte, aber vor allem, weil die offiziellen Dokumente sagen, es sei nicht zu empfehlen, und ich habe es bisher ohne geschafft, also empfehlen Sie es bitte nicht Route.

public class Login extends Activity {

    static ProgressDialog pd;
    AsyncTask<String, Void, Boolean> asyncLoginThread;

    @Override
    public void onCreate(Bundle icicle) {
        super.onCreate(icicle);
        setContentView(R.layout.login);
        //SETUP UI OBJECTS
        restoreAsyncTask();
    }

    @Override
    public Object onRetainNonConfigurationInstance() {
        if (pd != null) pd.dismiss();
        if (asyncLoginThread != null) return (asyncLoginThread);
        return super.onRetainNonConfigurationInstance();
    }

    private void restoreAsyncTask();() {
        pd = new ProgressDialog(Login.this);
        if (getLastNonConfigurationInstance() != null) {
            asyncLoginThread = (AsyncTask<String, Void, Boolean>) getLastNonConfigurationInstance();
            if (asyncLoginThread != null) {
                if (!(asyncLoginThread.getStatus()
                        .equals(AsyncTask.Status.FINISHED))) {
                    showProgressDialog();
                }
            }
        }
    }

    public class LoginThread extends AsyncTask<String, Void, Boolean> {
        @Override
        protected Boolean doInBackground(String... args) {
            try {
                //Connect to WS, recieve a JSON/XML Response
                //Place it somewhere I can use it.
            } catch (Exception e) {
                return true;
            }
            return true;
        }

        protected void onPostExecute(Boolean result) {
            if (result) {
                pd.dismiss();
                //Handle the response. Either deny entry or launch new Login Succesful Activity
            }
        }
    }
}

Dieser Code funktioniert einwandfrei. Ich habe ungefähr 10.000 Benutzer ohne Beschwerde. Daher erschien es logisch, diese Logik einfach in das neue Fragment Based Design zu kopieren, aber es funktioniert natürlich nicht.

Hier ist das LoginFragment:

public class LoginFragment extends Fragment {

    FragmentActivity parentActivity;
    static ProgressDialog pd;
    AsyncTask<String, Void, Boolean> asyncLoginThread;

    public interface OnLoginSuccessfulListener {
        public void onLoginSuccessful(GlobalContainer globalContainer);
    }

    public void onSaveInstanceState(Bundle outState){
        super.onSaveInstanceState(outState);
        //Save some stuff for the UI State
    }

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        //setRetainInstance(true);
        //If I setRetainInstance(true), savedInstanceState is always null. Besides that, when loading UI State, a NPE is thrown when looking for UI Objects.
        parentActivity = getActivity();
    }

    @Override
    public void onAttach(Activity activity) {
        super.onAttach(activity);
        try {
            loginSuccessfulListener = (OnLoginSuccessfulListener) activity;
        } catch (ClassCastException e) {
            throw new ClassCastException(activity.toString() + " must implement OnLoginSuccessfulListener");
        }
    }

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container,
            Bundle savedInstanceState) {
        RelativeLayout loginLayout = (RelativeLayout) inflater.inflate(R.layout.login, container, false);
        return loginLayout;
    }

    @Override
    public void onActivityCreated(Bundle savedInstanceState) {
        super.onActivityCreated(savedInstanceState);
        //SETUP UI OBJECTS
        if(savedInstanceState != null){
            //Reload UI state. Im doing this properly, keeping the content of the UI objects, not the object it self to avoid memory leaks.
        }
    }

    public class LoginThread extends AsyncTask<String, Void, Boolean> {
            @Override
            protected Boolean doInBackground(String... args) {
                try {
                    //Connect to WS, recieve a JSON/XML Response
                    //Place it somewhere I can use it.
                } catch (Exception e) {
                    return true;
                }
                return true;
            }

            protected void onPostExecute(Boolean result) {
                if (result) {
                    pd.dismiss();
                    //Handle the response. Either deny entry or launch new Login Succesful Activity
                }
            }
        }
    }
}

Ich kann onRetainNonConfigurationInstance() nicht verwenden, da es aus der Aktivität und nicht aus dem Fragment aufgerufen werden muss. Das gleiche gilt für getLastNonConfigurationInstance(). Ich habe hier einige ähnliche Fragen ohne Antwort gelesen.

Ich verstehe, dass es einige Arbeit erfordert, um dieses Material in Fragmenten ordentlich zu ordnen. Ich möchte jedoch die gleiche grundlegende Designlogik beibehalten.

Was wäre der richtige Weg, um die AsyncTask während einer Konfigurationsänderung beizubehalten, und wenn sie noch ausgeführt wird, zeigen Sie einen progressDialog an. Berücksichtigen Sie, dass die AsyncTask eine innere Klasse des Fragments ist und das Fragment selbst die AsyncTask aufruft .ausführen()?

83
blindstuff

Fragmente können das tatsächlich viel einfacher machen. Verwenden Sie einfach die Methode Fragment.setRetainInstance (boolean) , um Ihre Fragmentinstanz bei Konfigurationsänderungen beizubehalten. Beachten Sie, dass dies der empfohlene Ersatz für Activity.onRetainnonConfigurationInstance () in den Dokumenten ist.

Wenn Sie aus irgendeinem Grund wirklich kein beibehaltenes Fragment verwenden möchten, gibt es andere Ansätze, die Sie verwenden können. Beachten Sie, dass jedes Fragment einen eindeutigen Bezeichner hat, der von Fragment.getId () zurückgegeben wird. Sie können auch herausfinden, ob ein Fragment für eine Konfigurationsänderung über Fragment.getActivity (). IsChangingConfigurations () abgebrochen wird. An dem Punkt, an dem Sie Ihre AsyncTask (in onStop () oder onDestroy () höchstwahrscheinlich) anhalten möchten, können Sie beispielsweise prüfen, ob sich die Konfiguration ändert, und ob sie in einem statischen SparseArray unter der Kennung des Fragments gespeichert wird. und dann in Ihrem onCreate () oder onStart () nachsehen, ob Sie eine AsyncTask im spärlichen Array zur Verfügung haben.

75
hackbod

Ich denke, Sie werden mein sehr ausführliches und ausführliches Arbeitsbeispiel genießen, das unten beschrieben wird.

  1. Die Rotation funktioniert und der Dialog bleibt erhalten.
  2. Sie können die Aufgabe und den Dialog abbrechen, indem Sie die Zurück-Taste drücken (wenn Sie dieses Verhalten wünschen).
  3. Es werden Fragmente verwendet.
  4. Das Layout des Fragments unter der Aktivität ändert sich ordnungsgemäß, wenn sich das Gerät dreht.
  5. Es gibt einen vollständigen Quellcode-Download und eine vorkompilierte APK , damit Sie sehen können, ob das Verhalten Ihren Wünschen entspricht.

Bearbeiten

Wie von Brad Larson gefordert, habe ich den Großteil der unten verlinkten Lösung reproduziert. Seit ich es gepostet habe, wurde ich auf AsyncTaskLoader hingewiesen. Ich bin mir nicht sicher, ob es für dieselben Probleme gilt, aber Sie sollten es trotzdem prüfen.

Verwenden von AsyncTask mit Fortschrittsdialogen und Gerätedrehung.

Eine funktionierende Lösung!

Ich habe endlich alles zum Laufen bekommen. Mein Code hat die folgenden Funktionen:

  1. Ein Fragment, dessen Layout sich mit der Ausrichtung ändert.
  2. Ein AsyncTask, in dem Sie etwas arbeiten können.
  3. Ein DialogFragment, das den Fortschritt der Aufgabe in einem Fortschrittsbalken anzeigt (nicht nur ein unbestimmter Spinner).
  4. Die Rotation funktioniert, ohne die Aufgabe zu unterbrechen oder den Dialog zu schließen.
  5. Die Zurück-Schaltfläche schließt den Dialog und bricht die Aufgabe ab (Sie können dieses Verhalten jedoch ziemlich leicht ändern).

Ich glaube nicht, dass diese Kombination von Arbeitsfähigkeit irgendwo anders zu finden ist.

Die Grundidee ist wie folgt. Es gibt eine MainActivity-Klasse, die ein einzelnes Fragment enthält - MainFragment. MainFragment hat unterschiedliche Layouts für die horizontale und vertikale Ausrichtung, und setRetainInstance() ist falsch, damit sich das Layout ändern kann. Dies bedeutet, dass MainActivity und MainFragment bei einer Änderung der Geräteorientierung vollständig gelöscht und neu erstellt werden.

Getrennt davon haben wir MyTask (erweitert von AsyncTask), das die ganze Arbeit erledigt. Wir können es nicht in MainFragment speichern, da dies zerstört wird und Google mit etwas wie setRetainNonInstanceConfiguration() veraltet ist. Das ist sowieso nicht immer verfügbar und ist bestenfalls ein hässlicher Hack. Stattdessen speichern wir MyTask in einem anderen Fragment, einem DialogFragment namens TaskFragment. Für dieses Fragment wirdsetRetainInstance() auf true gesetzt. Wenn sich das Gerät dreht, wird dieses Fragment nicht zerstört und MyTask wird beibehalten.

Schließlich müssen wir dem TaskFragment mitteilen, an wen er sich informieren soll, wenn er fertig ist, und dies tun wir mit setTargetFragment(<the MainFragment>), wenn wir ihn erstellen. Wenn das Gerät gedreht wird und MainFragment zerstört wird und eine neue Instanz erstellt wird, verwenden wir FragmentManager, um den Dialog zu finden (basierend auf seinem Tag) und tun setTargetFragment(<the new MainFragment>). Das wars so ziemlich.

Es gab zwei weitere Dinge, die ich tun musste: Zuerst die Aufgabe abbrechen, wenn das Dialogfeld geschlossen wurde, und zweitens die Ablehnungsnachricht auf null setzen, andernfalls wird das Dialogfeld beim Drehen des Geräts seltsam abgebrochen.

Der Code

Ich werde die Layouts nicht auflisten, sie sind ziemlich offensichtlich und Sie können sie im Projekt-Download unten finden.

Hauptaktivität

Das ist ziemlich einfach. Ich habe einen Rückruf zu dieser Aktivität hinzugefügt, damit er weiß, wann die Aufgabe abgeschlossen ist, aber Sie brauchen das vielleicht nicht. Hauptsächlich wollte ich nur den Callback-Mechanismus für Fragmentaktivität zeigen, weil er ziemlich ordentlich ist und Sie ihn vielleicht noch nicht gesehen haben.

public class MainActivity extends Activity implements MainFragment.Callbacks
{
    @Override
    public void onCreate(Bundle savedInstanceState)
    {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
    }
    @Override
    public void onTaskFinished()
    {
        // Hooray. A toast to our success.
        Toast.makeText(this, "Task finished!", Toast.LENGTH_LONG).show();
        // NB: I'm going to blow your mind again: the "int duration" parameter of makeText *isn't*
        // the duration in milliseconds. Android Y U NO ENUM? 
    }
}

MainFragment

Es ist lang, aber es lohnt sich!

public class MainFragment extends Fragment implements OnClickListener
{
    // This code up to onDetach() is all to get easy callbacks to the Activity. 
    private Callbacks mCallbacks = sDummyCallbacks;

    public interface Callbacks
    {
        public void onTaskFinished();
    }
    private static Callbacks sDummyCallbacks = new Callbacks()
    {
        public void onTaskFinished() { }
    };

    @Override
    public void onAttach(Activity activity)
    {
        super.onAttach(activity);
        if (!(activity instanceof Callbacks))
        {
            throw new IllegalStateException("Activity must implement fragment's callbacks.");
        }
        mCallbacks = (Callbacks) activity;
    }

    @Override
    public void onDetach()
    {
        super.onDetach();
        mCallbacks = sDummyCallbacks;
    }

    // Save a reference to the fragment manager. This is initialised in onCreate().
    private FragmentManager mFM;

    // Code to identify the fragment that is calling onActivityResult(). We don't really need
    // this since we only have one fragment to deal with.
    static final int TASK_FRAGMENT = 0;

    // Tag so we can find the task fragment again, in another instance of this fragment after rotation.
    static final String TASK_FRAGMENT_TAG = "task";

    @Override
    public void onCreate(Bundle savedInstanceState)
    {
        super.onCreate(savedInstanceState);

        // At this point the fragment may have been recreated due to a rotation,
        // and there may be a TaskFragment lying around. So see if we can find it.
        mFM = getFragmentManager();
        // Check to see if we have retained the worker fragment.
        TaskFragment taskFragment = (TaskFragment)mFM.findFragmentByTag(TASK_FRAGMENT_TAG);

        if (taskFragment != null)
        {
            // Update the target fragment so it goes to this fragment instead of the old one.
            // This will also allow the GC to reclaim the old MainFragment, which the TaskFragment
            // keeps a reference to. Note that I looked in the code and setTargetFragment() doesn't
            // use weak references. To be sure you aren't leaking, you may wish to make your own
            // setTargetFragment() which does.
            taskFragment.setTargetFragment(this, TASK_FRAGMENT);
        }
    }

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container,
            Bundle savedInstanceState)
    {
        return inflater.inflate(R.layout.fragment_main, container, false);
    }

    @Override
    public void onViewCreated(View view, Bundle savedInstanceState)
    {
        super.onViewCreated(view, savedInstanceState);

        // Callback for the "start task" button. I originally used the XML onClick()
        // but it goes to the Activity instead.
        view.findViewById(R.id.taskButton).setOnClickListener(this);
    }

    @Override
    public void onClick(View v)
    {
        // We only have one click listener so we know it is the "Start Task" button.

        // We will create a new TaskFragment.
        TaskFragment taskFragment = new TaskFragment();
        // And create a task for it to monitor. In this implementation the taskFragment
        // executes the task, but you could change it so that it is started here.
        taskFragment.setTask(new MyTask());
        // And tell it to call onActivityResult() on this fragment.
        taskFragment.setTargetFragment(this, TASK_FRAGMENT);

        // Show the fragment.
        // I'm not sure which of the following two lines is best to use but this one works well.
        taskFragment.show(mFM, TASK_FRAGMENT_TAG);
//      mFM.beginTransaction().add(taskFragment, TASK_FRAGMENT_TAG).commit();
    }

    @Override
    public void onActivityResult(int requestCode, int resultCode, Intent data)
    {
        if (requestCode == TASK_FRAGMENT && resultCode == Activity.RESULT_OK)
        {
            // Inform the activity. 
            mCallbacks.onTaskFinished();
        }
    }

TaskFragment

    // This and the other inner class can be in separate files if you like.
    // There's no reason they need to be inner classes other than keeping everything together.
    public static class TaskFragment extends DialogFragment
    {
        // The task we are running.
        MyTask mTask;
        ProgressBar mProgressBar;

        public void setTask(MyTask task)
        {
            mTask = task;

            // Tell the AsyncTask to call updateProgress() and taskFinished() on this fragment.
            mTask.setFragment(this);
        }

        @Override
        public void onCreate(Bundle savedInstanceState)
        {
            super.onCreate(savedInstanceState);

            // Retain this instance so it isn't destroyed when MainActivity and
            // MainFragment change configuration.
            setRetainInstance(true);

            // Start the task! You could move this outside this activity if you want.
            if (mTask != null)
                mTask.execute();
        }

        @Override
        public View onCreateView(LayoutInflater inflater, ViewGroup container,
                Bundle savedInstanceState)
        {
            View view = inflater.inflate(R.layout.fragment_task, container);
            mProgressBar = (ProgressBar)view.findViewById(R.id.progressBar);

            getDialog().setTitle("Progress Dialog");

            // If you're doing a long task, you probably don't want people to cancel
            // it just by tapping the screen!
            getDialog().setCanceledOnTouchOutside(false);

            return view;
        }

        // This is to work around what is apparently a bug. If you don't have it
        // here the dialog will be dismissed on rotation, so tell it not to dismiss.
        @Override
        public void onDestroyView()
        {
            if (getDialog() != null && getRetainInstance())
                getDialog().setDismissMessage(null);
            super.onDestroyView();
        }

        // Also when we are dismissed we need to cancel the task.
        @Override
        public void onDismiss(DialogInterface dialog)
        {
            super.onDismiss(dialog);
            // If true, the thread is interrupted immediately, which may do bad things.
            // If false, it guarantees a result is never returned (onPostExecute() isn't called)
            // but you have to repeatedly call isCancelled() in your doInBackground()
            // function to check if it should exit. For some tasks that might not be feasible.
            if (mTask != null) {
                mTask.cancel(false);
            }

            // You don't really need this if you don't want.
            if (getTargetFragment() != null)
                getTargetFragment().onActivityResult(TASK_FRAGMENT, Activity.RESULT_CANCELED, null);
        }

        @Override
        public void onResume()
        {
            super.onResume();
            // This is a little hacky, but we will see if the task has finished while we weren't
            // in this activity, and then we can dismiss ourselves.
            if (mTask == null)
                dismiss();
        }

        // This is called by the AsyncTask.
        public void updateProgress(int percent)
        {
            mProgressBar.setProgress(percent);
        }

        // This is also called by the AsyncTask.
        public void taskFinished()
        {
            // Make sure we check if it is resumed because we will crash if trying to dismiss the dialog
            // after the user has switched to another app.
            if (isResumed())
                dismiss();

            // If we aren't resumed, setting the task to null will allow us to dimiss ourselves in
            // onResume().
            mTask = null;

            // Tell the fragment that we are done.
            if (getTargetFragment() != null)
                getTargetFragment().onActivityResult(TASK_FRAGMENT, Activity.RESULT_OK, null);
        }
    }

Meine Aufgabe

    // This is a fairly standard AsyncTask that does some dummy work.
    public static class MyTask extends AsyncTask<Void, Void, Void>
    {
        TaskFragment mFragment;
        int mProgress = 0;

        void setFragment(TaskFragment fragment)
        {
            mFragment = fragment;
        }

        @Override
        protected Void doInBackground(Void... params)
        {
            // Do some longish task. This should be a task that we don't really
            // care about continuing
            // if the user exits the app.
            // Examples of these things:
            // * Logging in to an app.
            // * Downloading something for the user to view.
            // * Calculating something for the user to view.
            // Examples of where you should probably use a service instead:
            // * Downloading files for the user to save (like the browser does).
            // * Sending messages to people.
            // * Uploading data to a server.
            for (int i = 0; i < 10; i++)
            {
                // Check if this has been cancelled, e.g. when the dialog is dismissed.
                if (isCancelled())
                    return null;

                SystemClock.sleep(500);
                mProgress = i * 10;
                publishProgress();
            }
            return null;
        }

        @Override
        protected void onProgressUpdate(Void... unused)
        {
            if (mFragment == null)
                return;
            mFragment.updateProgress(mProgress);
        }

        @Override
        protected void onPostExecute(Void unused)
        {
            if (mFragment == null)
                return;
            mFragment.taskFinished();
        }
    }
}

Laden Sie das Beispielprojekt herunter

Hier ist der Quellcode und der APK . Entschuldigung, das ADT bestand darauf, die Unterstützungsbibliothek hinzuzufügen, bevor ich ein Projekt erstellen konnte. Ich bin sicher, Sie können es entfernen.

66
Timmmm

Ich habe kürzlich einen Artikel veröffentlicht, der beschreibt, wie Konfigurationsänderungen mit beibehaltenen Fragments behandelt werden. Es löst das Problem, ein AsyncTask über einen Drehungswechsel gut zu behalten.

Das TL; DR ist zu verwenden. Hosten Sie Ihr AsyncTask in einem Fragment, rufen Sie setRetainInstance(true) auf dem Fragment an und melden Sie den Fortschritt/die Ergebnisse des AsyncTask an das Activity (oder dessen Ziel Fragment), wenn Sie den voreingestellten Durchlauf verwenden möchten die zurückbehaltene Fragment.

16
Alex Lockwood

Mein erster Vorschlag ist, innere AsyncTasks zu vermeiden. Sie können eine Frage lesen, die ich dazu und die Antworten dazu gestellt habe: Android: AsyncTask-Empfehlungen: Privatklasse oder öffentliche Klasse?

Danach begann ich mit der Verwendung von nicht-innerem und ... jetzt sehe ich VIELE Vorteile.

Die zweite ist, halten Sie einen Verweis auf Ihre laufende AsyncTask in der Application-Klasse - http://developer.Android.com/reference/Android/app/Application.html .

Jedes Mal, wenn Sie eine AsyncTask starten, setzen Sie sie in der Anwendung und am Ende auf Null.

Wenn ein Fragment/eine Aktivität gestartet wird, können Sie überprüfen, ob eine AsyncTask ausgeführt wird (indem Sie überprüfen, ob sie in der Anwendung null ist oder nicht) und dann den Verweis auf das setzen, was Sie möchten (Aktivität, Fragment usw.), um Callbacks auszuführen.

Damit lösen Sie Ihr Problem: Wenn nur 1 AsyncTask zu einem bestimmten Zeitpunkt ausgeführt wird, können Sie eine einfache Referenz hinzufügen:

AsyncTask<?,?,?> asyncTask = null;

Ansonsten haben Sie in der Anwendung eine HashMap mit Verweisen darauf.

Der Fortschrittsdialog kann dem gleichen Prinzip folgen.

13
neteinstein

Ich habe mir eine Methode für die Verwendung von AsyncTaskLoaders ausgedacht. Es ist ziemlich einfach zu bedienen und erfordert weniger Overhead-IMO.

Grundsätzlich erstellen Sie einen AsyncTaskLoader wie folgt:

public class MyAsyncTaskLoader extends AsyncTaskLoader {
    Result mResult;
    public HttpAsyncTaskLoader(Context context) {
        super(context);
    }

    protected void onStartLoading() {
        super.onStartLoading();
        if (mResult != null) {
            deliverResult(mResult);
        }
        if (takeContentChanged() ||  mResult == null) {
            forceLoad();
        }
    }

    @Override
    public Result loadInBackground() {
        SystemClock.sleep(500);
        mResult = new Result();
        return mResult;
    }
}

Dann in Ihrer Aktivität, die den obigen AsyncTaskLoader verwendet, wenn auf eine Schaltfläche geklickt wird:

public class MyActivityWithBackgroundWork extends FragmentActivity implements LoaderManager.LoaderCallbacks<Result> {

    private String username,password;       
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        // TODO Auto-generated method stub
        super.onCreate(savedInstanceState);
        setContentView(R.layout.mylayout);
        //this is only used to reconnect to the loader if it already started
        //before the orientation changed
        Loader loader = getSupportLoaderManager().getLoader(0);
        if (loader != null) {
            getSupportLoaderManager().initLoader(0, null, this);
        }
    }

    public void doBackgroundWorkOnClick(View button) {
        //might want to disable the button while you are doing work
        //to prevent user from pressing it again.

        //Call resetLoader because calling initLoader will return
        //the previous result if there was one and we may want to do new work
        //each time
        getSupportLoaderManager().resetLoader(0, null, this);
    }   


    @Override
    public Loader<Result> onCreateLoader(int i, Bundle bundle) {
        //might want to start a progress bar
        return new MyAsyncTaskLoader(this);
    }


    @Override
    public void onLoadFinished(Loader<LoginResponse> loginLoader,
                               LoginResponse loginResponse)
    {
        //handle result
    }

    @Override
    public void onLoaderReset(Loader<LoginResponse> responseAndJsonHolderLoader)
    {
        //remove references to previous loader resources

    }
}

Dies scheint Orientierungsänderungen gut zu behandeln, und Ihre Hintergrundaufgabe wird während der Drehung fortgesetzt.

Ein paar Dinge zu beachten:

  1. Wenn Sie in onCreate erneut zum asynctaskloader wechseln, werden Sie in onLoadFinished () mit dem vorherigen Ergebnis zurückgerufen (auch wenn Ihnen bereits mitgeteilt wurde, dass die Anforderung abgeschlossen war). Das ist eigentlich meistens ein gutes Verhalten, aber manchmal kann es schwierig sein, damit umzugehen. Während ich mir vorstellen kann, dass es viele Möglichkeiten gibt, um mit dem umzugehen, was ich getan habe, war ich loader.abandon () in onLoadFinished. Dann fügte ich Check in onCreate hinzu, um den Loader nur wieder anzuschließen, wenn er nicht bereits aufgegeben wurde. Wenn Sie die resultierenden Daten erneut benötigen, möchten Sie dies nicht tun. In den meisten Fällen möchten Sie die Daten.

Ich habe weitere Informationen zur Verwendung für http-Anrufe hier

4
Matt Wolfe

Ich habe eine sehr kleine Open Source-Aufgabenbibliothek erstellt, die stark auf Marshmallow AsyncTask basiert, jedoch mit zusätzlichen Funktionen wie:

  1. Automatisches Speichern von Aufgaben bei Konfigurationsänderungen;
  2. UI-Callback (Listener);
  3. Startet oder storniert den Task nicht, wenn sich das Gerät dreht (wie es Loader tun würden);

Die Bibliothek verwendet intern eine Fragment ohne Benutzeroberfläche, die bei Konfigurationsänderungen (setRetainInstance(true)) beibehalten wird.

Sie finden es auf GitHub: https://github.com/NeoTech-Software/Android-Retainable-Tasks

Das einfachste Beispiel (Version 0.2.0):

In diesem Beispiel wird die Task vollständig beibehalten, wobei sehr wenig Code verwendet wird.

Aufgabe:

private class ExampleTask extends Task<Integer, String> {

    public ExampleTask(String tag){
        super(tag);
    }

    protected String doInBackground() {
        for(int i = 0; i < 100; i++) {
            if(isCancelled()){
                break;
            }
            SystemClock.sleep(50);
            publishProgress(i);
        }
        return "Result";
    }
}

Aktivität:

public class Main extends TaskActivityCompat implements Task.Callback {

    @Override
    public void onClick(View view){
        ExampleTask task = new ExampleTask("activity-unique-tag");
        getTaskManager().execute(task, this);
    }

    @Override
    public Task.Callback onPreAttach(Task<?, ?> task) {
        //Restore the user-interface based on the tasks state
        return this; //This Activity implements Task.Callback
    }

    @Override
    public void onPreExecute(Task<?, ?> task) {
        //Task started
    }

    @Override
    public void onPostExecute(Task<?, ?> task) {
        //Task finished
        Toast.makeText(this, "Task finished", Toast.LENGTH_SHORT).show();
    }
}
3
Rolf ツ

Mein Ansatz ist die Verwendung eines Delegierungsentwurfsmusters. Im Allgemeinen können wir die tatsächliche Geschäftslogik (Lesen von Daten aus dem Internet oder aus Datenbanken oder was auch immer) von AsyncTask (dem Delegator) zu BusinessDAO (dem Delegaten) in Ihrer AysncTask.doInBackground () - Methode isolieren delegieren Sie die eigentliche Task an BusinessDAO, implementieren Sie anschließend einen Singleton-Prozessmechanismus in BusinessDAO, sodass ein mehrfacher Aufruf von BusinessDAO.doSomething () nur jeweils eine tatsächliche Task auslöst und auf das Taskergebnis wartet. Die Idee ist, den Delegaten (d. H. BusinessDAO) während der Konfigurationsänderung anstelle des Delegators (d. H. AsyncTask) beizubehalten. 

  1. Erstellen/Implementieren Sie unsere eigene Anwendung. Der Zweck besteht darin, BusinessDAO hier zu erstellen/initialisieren, sodass der Lebenszyklus unseres BusinessDAOs auf Anwendungsbereiche und nicht auf Aktivitäten ausgerichtet ist. Beachten Sie, dass Sie AndroidManifest.xml ändern müssen, um MyApplication verwenden zu können: 

    public class MyApplication extends Android.app.Application {
      private BusinessDAO businessDAO;
    
      @Override
      public void onCreate() {
        super.onCreate();
        businessDAO = new BusinessDAO();
      }
    
      pubilc BusinessDAO getBusinessDAO() {
        return businessDAO;
      }
    
    }
    
  2. Unsere bestehenden Activity/Fragement sind weitgehend unverändert, implementieren AsyncTask immer noch als innere Klasse und beinhalten AsyncTask.execute () von Activity/Fragement. Der Unterschied ist nun, dass AsyncTask die eigentliche Task an BusinessDAO delegiert, also während der Konfigurationsänderung eine zweite AsyncTask wird initialisiert und ausgeführt und ruft BusinessDAO.doSomething () zum zweiten Mal auf. Der zweite Aufruf von BusinessDAO.doSomething () löst jedoch keine neue laufende Task aus, sondern wartet, bis die laufende Task beendet ist:

    public class LoginFragment extends Fragment {
      ... ...
    
      public class LoginAsyncTask extends AsyncTask<String, Void, Boolean> {
        // get a reference of BusinessDAO from application scope.
        BusinessDAO businessDAO = ((MyApplication) getApplication()).getBusinessDAO();
    
        @Override
        protected Boolean doInBackground(String... args) {
            businessDAO.doSomething();
            return true;
        }
    
        protected void onPostExecute(Boolean result) {
          //Handle task result and update UI stuff.
        }
      }
    
      ... ...
    }
    
  3. Implementieren Sie in BusinessDAO den Singleton-Prozessmechanismus, zum Beispiel:

    public class BusinessDAO {
      ExecutorCompletionService<MyTask> completionExecutor = new ExecutorCompletionService<MyTask(Executors.newFixedThreadPool(1));
      Future<MyTask> myFutureTask = null;
    
      public void doSomething() {
        if (myFutureTask == null) {
          // nothing running at the moment, submit a new callable task to run.
          MyTask myTask = new MyTask();
          myFutureTask = completionExecutor.submit(myTask);
        }
        // Task already submitted and running, waiting for the running task to finish.
        myFutureTask.get();
      }
    
      // If you've never used this before, Callable is similar with Runnable, with ability to return result and throw exception.
      private class MyTask extends Callable<MyTask> {
        public MyAsyncTask call() {
          // do your job here.
          return this;
        }
      }
    
    }
    

Ich bin nicht zu 100% sicher, ob dies funktioniert. Außerdem sollte das Beispielcode-Snippet als Pseudocode betrachtet werden. Ich versuche nur, Ihnen einen Hinweis vom Designniveau zu geben. Rückmeldungen oder Anregungen sind willkommen und willkommen.

1
yorkw

Wenn jemand seinen Weg zu diesem Thread findet, fand ich einen sauberen Ansatz, die Async-Task von einem app.Service (mit START_STICKY gestartet) auszuführen und dann die laufenden Dienste erneut zu durchlaufen, um herauszufinden, ob der Service (und damit die Async-Task) ist funktioniert immer noch;

    public boolean isServiceRunning(String serviceClassName) {
    final ActivityManager activityManager = (ActivityManager) Application.getContext().getSystemService(Context.ACTIVITY_SERVICE);
    final List<RunningServiceInfo> services = activityManager.getRunningServices(Integer.MAX_VALUE);

    for (RunningServiceInfo runningServiceInfo : services) {
        if (runningServiceInfo.service.getClassName().equals(serviceClassName)){
            return true;
        }
    }
    return false;
 }

Ist dies der Fall, fügen Sie die Variable DialogFragment (oder was auch immer) erneut hinzu. Wenn dies nicht der Fall ist, stellen Sie sicher, dass der Dialog geschlossen wurde.

Dies ist besonders relevant, wenn Sie die v4.support.*-Bibliotheken verwenden, da sie (zum Zeitpunkt des Schreibens) Probleme mit der setRetainInstance-Methode und dem Ansichts-Paging haben. Wenn Sie die Instanz nicht beibehalten, können Sie Ihre Aktivität mit einem anderen Satz von Ressourcen (d. H. Einem anderen Ansichtslayout für die neue Ausrichtung) neu erstellen.

1
OceanLife

Sehen Sie sich unten ein Beispiel an, wie Sie ein beibehaltenes Fragment verwenden, um die Hintergrundaufgabe beizubehalten:

public class NetworkRequestFragment extends Fragment {

    // Declare some sort of interface that your AsyncTask will use to communicate with the Activity
    public interface NetworkRequestListener {
        void onRequestStarted();
        void onRequestProgressUpdate(int progress);
        void onRequestFinished(SomeObject result);
    }

    private NetworkTask mTask;
    private NetworkRequestListener mListener;

    private SomeObject mResult;

    @Override
    public void onAttach(Activity activity) {
        super.onAttach(activity);

        // Try to use the Activity as a listener
        if (activity instanceof NetworkRequestListener) {
            mListener = (NetworkRequestListener) activity;
        } else {
            // You can decide if you want to mandate that the Activity implements your callback interface
            // in which case you should throw an exception if it doesn't:
            throw new IllegalStateException("Parent activity must implement NetworkRequestListener");
            // or you could just swallow it and allow a state where nobody is listening
        }
    }

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

        // Retain this Fragment so that it will not be destroyed when an orientation
        // change happens and we can keep our AsyncTask running
        setRetainInstance(true);
    }

    /**
     * The Activity can call this when it wants to start the task
     */
    public void startTask(String url) {
        mTask = new NetworkTask(url);
        mTask.execute();
    }

    @Override
    public void onActivityCreated(Bundle savedInstanceState) {
        super.onActivityCreated(savedInstanceState);
        // If the AsyncTask finished when we didn't have a listener we can
        // deliver the result here
        if ((mResult != null) && (mListener != null)) {
            mListener.onRequestFinished(mResult);
            mResult = null;
        }
    }

    @Override
    public void onDestroy() {
        super.onDestroy();

        // We still have to cancel the task in onDestroy because if the user exits the app or
        // finishes the Activity, we don't want the task to keep running
        // Since we are retaining the Fragment, onDestroy won't be called for an orientation change
        // so this won't affect our ability to keep the task running when the user rotates the device
        if ((mTask != null) && (mTask.getStatus == AsyncTask.Status.RUNNING)) {
            mTask.cancel(true);
        }
    }

    @Override
    public void onDetach() {
        super.onDetach();

        // This is VERY important to avoid a memory leak (because mListener is really a reference to an Activity)
        // When the orientation change occurs, onDetach will be called and since the Activity is being destroyed
        // we don't want to keep any references to it
        // When the Activity is being re-created, onAttach will be called and we will get our listener back
        mListener = null;
    }

    private class NetworkTask extends AsyncTask<String, Integer, SomeObject> {

        @Override
        protected void onPreExecute() {
            if (mListener != null) {
                mListener.onRequestStarted();
            }
        }

        @Override
        protected SomeObject doInBackground(String... urls) {
           // Make the network request
           ...
           // Whenever we want to update our progress:
           publishProgress(progress);
           ...
           return result;
        }

        @Override
        protected void onProgressUpdate(Integer... progress) {
            if (mListener != null) {
                mListener.onRequestProgressUpdate(progress[0]);
            }
        }

        @Override
        protected void onPostExecute(SomeObject result) {
            if (mListener != null) {
                mListener.onRequestFinished(result);
            } else {
                // If the task finishes while the orientation change is happening and while
                // the Fragment is not attached to an Activity, our mListener might be null
                // If you need to make sure that the result eventually gets to the Activity
                // you could save the result here, then in onActivityCreated you can pass it back
                // to the Activity
                mResult = result;
            }
        }

    }
}
0
DeepakPanwar

Ich schreibe den gleichen Code, um dieses Problem zu lösen

Erster Schritt ist die Klasse make application:

public class TheApp extends Application {

private static TheApp sTheApp;
private HashMap<String, AsyncTask<?,?,?>> tasks = new HashMap<String, AsyncTask<?,?,?>>();

@Override
public void onCreate() {
    super.onCreate();
    sTheApp = this;
}

public static TheApp get() {
    return sTheApp;
}

public void registerTask(String tag, AsyncTask<?,?,?> task) {
    tasks.put(tag, task);
}

public void unregisterTask(String tag) {
    tasks.remove(tag);
}

public AsyncTask<?,?,?> getTask(String tag) {
    return tasks.get(tag);
}
}

In AndroidManifest.xml

<application
        Android:allowBackup="true"
        Android:icon="@drawable/ic_launcher"
        Android:label="@string/app_name"
        Android:theme="@style/AppTheme"
        Android:name="com.example.tasktest.TheApp">

Code in der Aktivität:

public class MainActivity extends Activity {

private Task1 mTask1;

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    mTask1 = (Task1)TheApp.get().getTask("task1");

}

/*
 * start task is not running jet
 */
public void handletask1(View v) {
    if (mTask1 == null) {
        mTask1 = new Task1();
        TheApp.get().registerTask("task1", mTask1);
        mTask1.execute();
    } else
        Toast.makeText(this, "Task is running...", Toast.LENGTH_SHORT).show();

}

/*
 * cancel task if is not finished
 */
public void handelCancel(View v) {
    if (mTask1 != null)
        mTask1.cancel(false);
}

public class Task1 extends AsyncTask<Void, Void, Void>{

    @Override
    protected Void doInBackground(Void... params) {
        try {
            for(int i=0; i<120; i++) {
                Thread.sleep(1000);
                Log.i("tests", "loop=" + i);
                if (this.isCancelled()) {
                    Log.e("tests", "tssk cancelled");
                    break;
                }
            }
        } catch (InterruptedException e) {
            e.printStackTrace();
        }
        return null;
    }

    @Override
    protected void onCancelled(Void result) {
        TheApp.get().unregisterTask("task1");
        mTask1 = null;
    }

    @Override
    protected void onPostExecute(Void result) {
        TheApp.get().unregisterTask("task1");
        mTask1 = null;
    }
}

}

Wenn sich die Aktivitätsorientierung ändert, wird die Variable mTask aus dem App-Kontext heraus eingefügt. Wenn die Aufgabe abgeschlossen ist, wird die Variable auf null gesetzt und aus dem Speicher entfernt.

Für mich ist es genug.

0
Kenumir

Sie könnten die AsyncTask zu einem statischen Feld machen. Wenn Sie einen Kontext benötigen, sollten Sie Ihren Anwendungskontext versenden. Dadurch werden Speicherlecks vermieden, ansonsten behalten Sie einen Hinweis auf Ihre gesamte Aktivität.

0
Boude