419

我创建了一个代表我的问题的小型测试应用程序。我正在使用 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.

但是我该如何解决这个问题?

4

13 回答 13

820

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

true如果片段当前已添加到其活动中,则返回。

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

为了避免在未附加到onPostExecute时被调用,是在暂停或停止时取消。那么就没有必要了。但是,建议保留此检查。FragmentActivityAsyncTaskFragmentisAdded()

于 2012-06-07T23:51:23.167 回答
35

问题是您正在尝试使用 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是持有你的活动的对象。

因为 Activity 可能未附加,所以您的 getResources() 调用将引发异常。

接受的解决方案恕我直言,因为您只是在隐藏问题,因此不是可行的方法。正确的方法是从其他始终保证存在的地方获取资源,例如应用程序上下文:

youApplicationObject.getResources().getString(...)
于 2016-06-30T21:00:55.903 回答
26

我在这里遇到了两种不同的情况:

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();
}

我发现这没有问题,尽管我也使用了(也许)更复杂的方式,包括从活动而不是片段启动任务。

希望这对某人有帮助!:)

于 2013-04-24T10:13:35.340 回答
22

他们是非常巧妙的解决方案,并且会从活动中泄漏片段。

因此,在 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


        }
    }
于 2016-09-02T09:44:26.933 回答
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;
        }

    }
}
于 2014-05-29T23:24:59.957 回答
15
if (getActivity() == null) return;

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

于 2017-07-04T16:17:16.903 回答
10

我遇到了同样的问题,我只是添加了单例实例来获取 Erick 提到的资源

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

你也可以使用

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

我希望这将有所帮助。

于 2014-08-27T15:03:47.240 回答
2

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

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

这是更新首选项摘要以显示新条目的侦听器的代码。它位于我的 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");
                    }
                }
            }
        };
        // ...
    }

我希望这对其他人有帮助!

于 2015-07-10T21:37:15.383 回答
1

如果您扩展Application该类并维护一个静态的“全局”上下文对象,如下所示,那么您可以使用它而不是活动来加载字符串资源。

public class MyApplication extends Application {
    public static Context GLOBAL_APP_CONTEXT;

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

如果您使用它,您可以摆脱Toast资源加载而不必担心生命周期。

于 2015-06-08T22:39:41.653 回答
0

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

getActivity().onBackPressed();
于 2016-10-18T12:33:09.670 回答
0

一个旧帖子,但我对投票最多的答案感到惊讶。

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

@Override
public void onStop() {
    super.onStop();
    mYourAsyncTask.cancel(true);
}
于 2017-01-05T11:46:52.577 回答
0

我在 Xamarine Android 中有类似的错误消息“Fragment MyFragment 未附加到上下文” 。

由于此资源调用而出现此错误消息

button.Text = Resources.GetString(Resource.String.please_wait)

我确实通过在 Xamarine Android 中使用来解决这个问题。

if (Context != null && IsAdded){ 
    button.Text = Resources.GetString(Resource.String.please_wait);
}
于 2021-01-28T01:41:17.563 回答
-1

简单的解决方案和工作 100%

if (getActivity() == null || !isAdded()) return;
于 2020-10-25T09:58:48.767 回答