片段MyFragment未附加到活动


393

我创建了一个小测试应用程序来代表我的问题。我正在使用ActionBarSherlock来实现带有(Sherlock)Fragments的选项卡。

我的代码: 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}未附加到Activity

我认为这是因为MyFragment在此期间已创建了一个新对象,并在AsyncTask完成之前将其附加到了Activity上。中的代码onPostExecute调用独立的MyFragment

但是我该如何解决呢?


1
您应该使用片段充气器的视图。 mView = inflater.inflate(R.layout.my_layout, container, false) 现在,当您想获取资源时使用此视图:mView.getResources().***。它可以帮助我修复此错误。
foxis

@foxis泄漏了Context附加到您的mView中的那个。
nhaarman

可能是我还没有检查。为了避免泄漏,如何mView在onDestroy中获取null ?
foxis

Answers:


774

我找到了非常简单的答案isAdded()

返回true如果片段正在增加其活性。

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

要避免onPostExecuteFragment未附加时调用Activity,请AsyncTask在暂停或停止时取消Fragment。这样isAdded()就不再需要了。但是,建议保留此检查。


以我为例,当我从中启动另一个应用程序意图时...然后我遇到相同的错误...任何建议?
2013年

1
developer.android.com/reference/android/app / ... ...还有isDetached()API级别13中添加的
Lucas Jota 2014年

5
当使用API​​ <11时,您正在使用developer.android.com/reference/android/support/v4/app/…它将在其中工作。
nhaarman 2014年

当我使用DialogFragment时遇到了这个问题。解散dialogFragment之后,我尝试启动另一个活动。然后发生此错误。我通过在startActivity之后调用dismiss()来避免此错误。问题在于该片段已经与Activity分离。
阿塔鲁2014年

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's

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仍在工作,这是因为您没有在片段重建之前(旋转时)在onDestroy()中正确取消AsyncTask实例,并且当此相同的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或依赖于从Fragment访问活动上下文的任何内容,总是检查活动状态和碎片状态,如下所示

 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
可以使用iwego()就足够了,因为:final public boolean iFyreed()}
NguyenDat's

在我的情况下,这种检查不是很详尽,尽管我添加了这些检查,但仍然会崩溃。
大卫,

@David,isAdded足够了。getString()如果坠毁,我从未见过这种情况isAdded == true。您确定显示了活动并附加了片段吗?
CoolMind

14
if (getActivity() == null) return;

在某些情况下也有效。只是中断了代码执行,并确保应用不会崩溃


10

我遇到了同样的问题,我只添加了单调实例以获取Erick引用的资源

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

你也可以使用

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

我希望这将有所帮助。


2

当带有已加载的首选项的应用程序设置活动可见时,我遇到了类似的问题。如果我要更改首选项之一,然后旋转显示内容并再次更改首选项,则会崩溃,并显示一条消息(片段(我的Preferences类)未附加到活动)。

调试时,看起来好像旋转显示内容时,PreferencesFragment的onCreate()方法被调用了两次。那已经足够奇怪了。然后,我在该块之外添加了iFyreed()检查,该检查将指示崩溃并解决了该问题。

这是用于更新首选项摘要以显示新条目的侦听器的代码。它位于我的Preferences类的onCreate()方法中,该方法扩展了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类并维护一个静态的“全局” Context对象(如下所示),则可以使用该对象代替活动来加载String资源。

public class MyApplication extends Application {
    public static Context GLOBAL_APP_CONTEXT;

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

如果使用此方法,则可以避免Toast资源加载,而不必担心生命周期。


5
我被拒绝了,但没有人解释原因。静态上下文通常很糟糕,但是我的想法是,如果您有静态的Application引用,那不是内存泄漏。
安东尼·丘纳德

您的回答被否决了,因为这只是一个hack,不是正确的解决方案。@nhaarman共享的检查解决方案
Vivek Kumar Srivastava

0

在我的情况下,片段方法已被调用

getActivity().onBackPressed();

0

一篇旧文章,但我对最投票的答案感到惊讶。

正确的解决方案应该是取消onStop中的asynctask(或您的片段中的适当位置)。这样,您就不会造成内存泄漏(异步任务保留对已破坏片段的引用),并且您可以更好地控制片段中发生的事情。

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

1
最受欢迎的答案包括这一点。同样,cancel可能不会阻止onPostExecute调用。
nhaarman

调用cancel可以确保永远不会调用onPostExecute,这两个调用都在同一个线程上执行,因此您可以确保在调用cancel之后不会调用它
Raz
By using our site, you acknowledge that you have read and understand our Cookie Policy and Privacy Policy.
Licensed under cc by-sa 3.0 with attribution required.