419

私の問題を表す小さなテストアプリを作成しました。(Sherlock)Fragments でタブを実装するために 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

しかし、どうすればこれを修正できますか?

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() を使用してリソース (この場合は文字列) にアクセスしようとしていることです。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();
 }

mHostActivity を保持するオブジェクトです。

アクティビティがアタッチされていない可能性があるため、getResources() 呼び出しで例外がスローされます。

問題を隠しているだけなので、受け入れられた解決策IMHOは進むべき道ではありません。正しい方法は、アプリケーション コンテキストのように、常に存在することが保証されている別の場所からリソースを取得することです。

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

ここで 2 つの異なるシナリオに直面しました。

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 に答える
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 に答える
10

私は同じ問題に直面しました。シングルトーンインスタンスを追加して、エリックが参照するリソースを取得するだけです

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

あなたも使うことができます

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

これが役立つことを願っています。

于 2014-08-27T15:03:47.240 に答える
2

ロードされた設定を含むアプリケーション設定アクティビティが表示されたときに、同様の問題に直面しました。設定の 1 つを変更し、表示コンテンツを回転させて設定を再度変更すると、フラグメント (設定クラス) がアクティビティに関連付けられていないというメッセージが表示されてクラッシュします。

デバッグ時に、表示コンテンツが回転したときに、PreferencesFragment の onCreate() メソッドが 2 回呼び出されているように見えました。それはもう十分に奇妙でした。次に、クラッシュを示すブロックの外側に isAdded() チェックを追加し、問題を解決しました。

以下は、設定の概要を更新して新しいエントリを表示するリスナーのコードです。これは、PreferenceFragment クラスを拡張する私の Preferences クラスの onCreate() メソッドにあります。

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

シンプルなソリューションと作業 100%

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