Фрагмент MyFragment, не привязанный к Activity


393

Я создал небольшое тестовое приложение, которое представляет мою проблему. Я использую ActionBarSherlock для реализации вкладок с (Шерлок) фрагментами.

Мой код: TestActivity.java

public class TestActivity extends SherlockFragmentActivity {
    private ActionBar actionBar;

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

    private void setupTabs(Bundle savedInstanceState) {
        actionBar = getSupportActionBar();
        actionBar.setNavigationMode(ActionBar.NAVIGATION_MODE_TABS);

        addTab1();
        addTab2();
    }

    private void addTab1() {
        Tab tab1 = actionBar.newTab();
        tab1.setTag("1");
        String tabText = "1";
        tab1.setText(tabText);
        tab1.setTabListener(new TabListener<MyFragment>(TestActivity.this, "1", MyFragment.class));

        actionBar.addTab(tab1);
    }

    private void addTab2() {
        Tab tab1 = actionBar.newTab();
        tab1.setTag("2");
        String tabText = "2";
        tab1.setText(tabText);
        tab1.setTabListener(new TabListener<MyFragment>(TestActivity.this, "2", MyFragment.class));

        actionBar.addTab(tab1);
    }
}

TabListener.java

public class TabListener<T extends SherlockFragment> implements com.actionbarsherlock.app.ActionBar.TabListener {
    private final SherlockFragmentActivity mActivity;
    private final String mTag;
    private final Class<T> mClass;

    public TabListener(SherlockFragmentActivity activity, String tag, Class<T> clz) {
        mActivity = activity;
        mTag = tag;
        mClass = clz;
    }

    /* The following are each of the ActionBar.TabListener callbacks */

    public void onTabSelected(Tab tab, FragmentTransaction ft) {
        SherlockFragment preInitializedFragment = (SherlockFragment) mActivity.getSupportFragmentManager().findFragmentByTag(mTag);

        // Check if the fragment is already initialized
        if (preInitializedFragment == null) {
            // If not, instantiate and add it to the activity
            SherlockFragment mFragment = (SherlockFragment) SherlockFragment.instantiate(mActivity, mClass.getName());
            ft.add(android.R.id.content, mFragment, mTag);
        } else {
            ft.attach(preInitializedFragment);
        }
    }

    public void onTabUnselected(Tab tab, FragmentTransaction ft) {
        SherlockFragment preInitializedFragment = (SherlockFragment) mActivity.getSupportFragmentManager().findFragmentByTag(mTag);

        if (preInitializedFragment != null) {
            // Detach the fragment, because another one is being attached
            ft.detach(preInitializedFragment);
        }
    }

    public void onTabReselected(Tab tab, FragmentTransaction ft) {
        // User selected the already selected tab. Usually do nothing.
    }
}

MyFragment.java

public class MyFragment extends SherlockFragment {

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

        new AsyncTask<Void, Void, Void>() {

            @Override
            protected Void doInBackground(Void... params) {
                try {
                    Thread.sleep(2000);
                } catch (InterruptedException ex) {
                }
                return null;
            }

            @Override
            protected void onPostExecute(Void result){
                getResources().getString(R.string.app_name);
            }

        }.execute();
    }
}

Я добавил Thread.sleepчасть для имитации загрузки данных. Код в onPostExecuteдля имитации использования Fragment.

Когда я очень быстро поворачиваю экран между пейзажем и портретом, я получаю исключение в onPostExecuteкоде:

java.lang.IllegalStateException: фрагмент MyFragment {410f6060} не присоединен к операции

Я думаю, это потому MyFragment, что тем временем был создан новый объект , который был прикреплен к действию до его AsyncTaskзавершения. Код в onPostExecuteпризыве к неприсоединению MyFragment.

Но как я могу это исправить?


1
Вы должны использовать вид из фрагмента inflater. mView = inflater.inflate(R.layout.my_layout, container, false) И теперь использовать эту точку зрения , когда вы хотите , чтобы получить ресурсы: mView.getResources().***. Это поможет мне исправить эту ошибку.
Фоксис

@foxis Это утечка, Contextкоторая прикреплена к вашему `mView`.
nhaarman

Может быть, я еще не проверял это. Чтобы избежать утечки, как насчет получения нуля mViewв onDestroy?
foxis

Ответы:


774

Я нашел очень простой ответ isAdded():

Возврат, trueесли фрагмент в настоящее время добавлен к его активности.

@Override
protected void onPostExecute(Void result){
    if(isAdded()){
        getResources().getString(R.string.app_name);
    }
}

Для того, чтобы избежать onPostExecuteот вызова , когда Fragmentне привязан к Activityэто отменить , AsyncTaskкогда паузы или остановки Fragment. Тогда isAdded()не было бы необходимости больше. Тем не менее, желательно сохранить эту проверку на месте.


В моем случае, когда я запускаю другое приложение Intent from ... тогда я получаю ту же ошибку ... любое предложение?
Код

1
developer.android.com/reference/android/app/… ... также есть isDetached(), что было добавлено на уровне API 13
Lucas Jota

5
Когда на API <11, вы используете developer.android.com/reference/android/support/v4/app/… где это будет работать.
nhaarman

Я столкнулся с этой проблемой, когда использовал DialogFragment. После отклонения dialogFragment я попытался начать другое действие. Тогда эта ошибка произошла. Я избежал этой ошибки, вызывая dismiss () после startActivity. Проблема заключалась в том, что фрагмент был уже отсоединен от Activity.
Атару

28

Проблема в том, что вы пытаетесь получить доступ к ресурсам (в данном случае к строкам) с помощью getResources (). GetString (), который попытается получить ресурсы из Activity. Посмотрите этот исходный код класса Fragment:

 /**
  * Return <code>getActivity().getResources()</code>.
  */
 final public Resources getResources() {
     if (mHost == null) {
         throw new IllegalStateException("Fragment " + this + " not attached to Activity");
     }
     return mHost.getContext().getResources();
 }

mHost это объект, который содержит вашу активность.

Поскольку действие не может быть присоединено, ваш вызов getResources () вызовет исключение.

Принятое решение ИМХО - это не тот путь, которым вы просто скрываете проблему. Правильный способ - просто получить ресурсы из другого места, которое всегда гарантированно существует, например, из контекста приложения:

youApplicationObject.getResources().getString(...)

Я использовал это решение, потому что мне нужно было выполнить, getString()когда мой фрагмент был приостановлен. Спасибо
Geekarist

24

Здесь я столкнулся с двумя разными сценариями:

1) Когда я хочу в любом случае завершить асинхронную задачу: представьте, что мой onPostExecute сохраняет полученные данные, а затем вызывает слушателя для обновления представлений, поэтому для большей эффективности я хочу, чтобы задача все равно завершилась, чтобы у меня были данные готовы, когда пользователь придет обратно. В этом случае я обычно делаю это:

@Override
protected void onPostExecute(void result) {
    // do whatever you do to save data
    if (this.getView() != null) {
        // update views
    }
}

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

@Override
protected void onStop() {
    // notice here that I keep a reference to the task being executed as a class member:
    if (this.myTask != null && this.myTask.getStatus() == Status.RUNNING) this.myTask.cancel(true);
    super.onStop();
}

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

Жаль, что это помогает кому-то! :)


18

Проблема с вашим кодом заключается в том, как вы используете AsyncTask, потому что когда вы поворачиваете экран во время спящего потока:

Thread.sleep(2000) 

AsyncTask все еще работает, это потому, что вы не отменили экземпляр AsyncTask должным образом в onDestroy () до перестройки фрагмента (при вращении), и когда этот же экземпляр AsyncTask (после вращения) запускается onPostExecute (), он пытается найти ресурсы с getResources () со старым экземпляром фрагмента (недопустимый экземпляр):

getResources().getString(R.string.app_name)

что эквивалентно:

MyFragment.this.getResources().getString(R.string.app_name)

Таким образом, окончательное решение заключается в управлении экземпляром AsyncTask (чтобы отменить, если он все еще работает), прежде чем фрагмент перестраивается при повороте экрана, и, если он отменен во время перехода, перезапустите AsyncTask после реконструкции с помощью логического флага:

public class MyFragment extends SherlockFragment {

    private MyAsyncTask myAsyncTask = null;
    private boolean myAsyncTaskIsRunning = true;

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        if(savedInstanceState!=null) {
            myAsyncTaskIsRunning = savedInstanceState.getBoolean("myAsyncTaskIsRunning");
        }
        if(myAsyncTaskIsRunning) {
            myAsyncTask = new MyAsyncTask();
            myAsyncTask.execute();
        }
    }

    @Override
    public void onSaveInstanceState(Bundle outState) {
        super.onSaveInstanceState(outState);
        outState.putBoolean("myAsyncTaskIsRunning",myAsyncTaskIsRunning);
    }

    @Override
    public void onDestroy() {
        super.onDestroy();
        if(myAsyncTask!=null) myAsyncTask.cancel(true);
        myAsyncTask = null;

    }

    public class MyAsyncTask extends AsyncTask<Void, Void, Void>() {

        public MyAsyncTask(){}

        @Override
        protected void onPreExecute() {
            super.onPreExecute();
            myAsyncTaskIsRunning = true;
        }
        @Override
        protected Void doInBackground(Void... params) {
            try {
                Thread.sleep(2000);
            } catch (InterruptedException ex) {}
            return null;
        }

        @Override
        protected void onPostExecute(Void result){
            getResources().getString(R.string.app_name);
            myAsyncTaskIsRunning = false;
            myAsyncTask = null;
        }

    }
}

вместо этого, если getResources().***использование Fragments.this.getResource().***помогло
Prabs

17

Это довольно хитрое решение для этого и утечка фрагмента из активности.

Таким образом, в случае getResource или чего-либо, что зависит от доступа к контексту активности из фрагмента, всегда проверяется статус активности и статус фрагментов следующим образом.

 Activity activity = getActivity(); 
    if(activity != null && isAdded())

         getResources().getString(R.string.no_internet_error_msg);
//Or any other depends on activity context to be live like dailog


        }
    }

7
isAdded () достаточно, потому что: final public boolean isAdded () {return mHost! = null && mAdded; }
NguyenDat

В моем случае, эти проверки не достаточно, все еще получаю сбои, несмотря на то, что я добавил их.
Дэвид

@ Давид, isAddedдостаточно. Я никогда не видел ситуацию, когда getString()разбился, если isAdded == true. Вы уверены, что активность была показана, а фрагмент был прикреплен?
CoolMind


10

Я столкнулся с той же проблемой, я просто добавил экземпляр синглета, чтобы получить ресурс, как указано Эриком

MainFragmentActivity.defaultInstance().getResources().getString(R.string.app_name);

Вы также можете использовать

getActivity().getResources().getString(R.string.app_name);

Я надеюсь, это поможет.


2

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

При отладке это выглядело так, как метод onCreate () в PreferencesFragment вызывался дважды, когда содержимое дисплея вращалось. Это было уже достаточно странно. Затем я добавил проверку isAdded () вне блока, где он будет указывать на сбой, и это решило проблему.

Вот код слушателя, который обновляет сводку настроек, чтобы показать новую запись. Он находится в методе onCreate () моего класса Preferences, который расширяет класс PreferenceFragment:

public static class Preferences extends PreferenceFragment {
    SharedPreferences.OnSharedPreferenceChangeListener listener;

    @Override
    public void onCreate(Bundle savedInstanceState) {
        // ...
        listener = new SharedPreferences.OnSharedPreferenceChangeListener() {
            @Override
            public void onSharedPreferenceChanged(SharedPreferences sharedPreferences, String key) {
                // check if the fragment has been added to the activity yet (necessary to avoid crashes)
                if (isAdded()) {
                    // for the preferences of type "list" set the summary to be the entry of the selected item
                    if (key.equals(getString(R.string.pref_fileviewer_textsize))) {
                        ListPreference listPref = (ListPreference) findPreference(key);
                        listPref.setSummary("Display file content with a text size of " + listPref.getEntry());
                    } else if (key.equals(getString(R.string.pref_fileviewer_segmentsize))) {
                        ListPreference listPref = (ListPreference) findPreference(key);
                        listPref.setSummary("Show " + listPref.getEntry() + " bytes of a file at once");
                    }
                }
            }
        };
        // ...
    }

Я надеюсь, что это поможет другим!


0

Если вы расширяете Applicationкласс и поддерживаете статический объект «глобального» контекста, как показано ниже, вы можете использовать его вместо действия для загрузки ресурса String.

public class MyApplication extends Application {
    public static Context GLOBAL_APP_CONTEXT;

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

Если вы используете это, вы можете сойти с рук Toastи загрузки ресурсов, не беспокоясь о жизненных циклах.


5
Меня опускают, но никто не объяснил почему. Статические контексты обычно плохие, но я думал, что это не утечка памяти, если у вас есть статическая ссылка на приложение.
Энтони Чуинард

Ваш ответ опущен, потому что это просто взломать не правильное решение. Проверьте решение, предоставленное @nhaarman
Вивек Кумар Шривастава

0

В моем случае методы фрагмента были вызваны после

getActivity().onBackPressed();

0

Старый пост, но я был удивлен самым голосуемым ответом.

Правильное решение для этого должно состоять в том, чтобы отменить asynctask в onStop (или где это уместно в вашем фрагменте). Таким образом, вы не вносите утечку памяти (асинхронную задачу, сохраняющую ссылку на ваш уничтоженный фрагмент), и вы лучше контролируете то, что происходит в вашем фрагменте.

@Override
public void onStop() {
    super.onStop();
    mYourAsyncTask.cancel(true);
}

1
Наиболее одобренный ответ включает это. Кроме того, cancelможет не помешать onPostExecuteбыть вызванным.
nhaarman

Вызов отмены гарантирует, что onPostExecute никогда не будет вызываться, оба вызова выполняются в одном и том же потоке, поэтому вы гарантированно не будете вызываться после вызова отмены
Raz
Используя наш сайт, вы подтверждаете, что прочитали и поняли нашу Политику в отношении файлов cookie и Политику конфиденциальности.
Licensed under cc by-sa 3.0 with attribution required.