我已经创建了一个小的测试应用程序来代表我的问题。 我使用ActionBarSherlock来实现标签与(Sherlock)片段。

我的代码: 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();
    }
}

我已经添加了线程。休眠部分模拟下载数据。onPostExecute中的代码是模拟Fragment的使用。

当我在横向和纵向之间快速旋转屏幕时,我在onPostExecute代码中得到一个异常:

java.lang.IllegalStateException: Fragment MyFragment{410f6060} not 附属于活动

我认为这是因为在此期间已经创建了一个新的MyFragment,并在AsyncTask完成之前附加到活动。onPostExecute中的代码调用一个未附加的MyFragment。

但我该怎么解决呢?


当前回答

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

if (getActivity() == null || !isAdded()) return;

其他回答

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

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

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

他们是相当技巧的解决方案,这和泄漏的片段从活动。

因此,在getResource或任何依赖于从Fragment访问的活动上下文的情况下,它总是检查活动状态和fragments状态,如下所示

 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


        }
    }

你的代码的问题是你使用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;
        }

    }
}

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

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

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

希望这能帮助到别人!:)

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

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

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

为了避免在Fragment没有附加到Activity时调用onPostExecute,需要在暂停或停止Fragment时取消AsyncTask。那么isAdded()就不再需要了。然而,保持这种检查是明智的。