フラグメントMyFragmentがActivityに添付されていません
私は自分の問題を表す小さなテストアプリを作成しました。私は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();
}
}
データのダウンロードをシミュレートするためにThread.sleep部分を追加しました。 onPostExecuteのコードはFragmentの使用をシミュレートすることです。
横長と縦長の間で画面をすばやく回転させると、onPostExecuteコードで例外が発生します。
Java.lang.IllegalStateException：Fragment MyFragment {410f6060}がActivityに添付されていません
その間に新しいMyFragmentが作成され、AsyncTaskが終了する前にActivityにアタッチされたためだと思います。 onPostExecuteのコードは、アタッチされていないMyFragmentを呼び出します。
しかし、どうすればこれを修正できますか？
私は非常に単純な答えを見つけました： isAdded() ：
フラグメントが現在そのアクティビティに追加されている場合は
trueを返します。
@Override
protected void onPostExecute(Void result){
if(isAdded()){
getResources().getString(R.string.app_name);
}
}
onPostExecuteがFragmentにアタッチされていないときにActivityが呼び出されないようにするには、AsyncTaskを一時停止または停止するときにFragmentを取り消します。それでisAdded()はもう必要ではないでしょう。ただし、このチェックを適切に行っておくことをお勧めします。
私はここで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();
}
フラグメントの代わりにアクティビティからタスクを起動することを含む（おそらく）より複雑な方法も使用しますが、これについては問題ありません。
これが誰かに役立つことを願います！ :)
問題は、アクティビティからリソースを取得しようとする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();
}
mHostはあなたのアクティビティを保持するオブジェクトです。
Activityが添付されていない可能性があるので、getResources（）呼び出しはExceptionをスローします。
承認された解決方法私見はあなたがただ問題を隠しているので行く道ではありません。正しい方法は、アプリケーションコンテキストのように、常に存在することが保証されている他の場所からリソースを取得することです。
youApplicationObject.getResources().getString(...)
あなたのコードの問題はあなたがAsyncTaskを使っている方法です。なぜならあなたがあなたのスリープスレッドの間にスクリーンを回転させるとき：
Thread.sleep(2000)
asyncTaskがまだ機能しているのは、フラグメントが再構築される前（回転したとき）と、同じAsyncTaskインスタンス（回転後）がonPostExecute（）を実行したときに、onDestroy（）でAsyncTaskインスタンスを正しくキャンセルできなかったためです。古いフラグメントインスタンス（無効なインスタンス）を持つ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;
}
}
}
彼らはこのための非常にトリックな解決策であり、アクティビティからのフラグメントのリークです。
したがって、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
}
}
私は同じ問題に直面した私はただErickによって参照されるようにリソースを得るためにsingletoneインスタンスを追加する
MainFragmentActivity.defaultInstance().getResources().getString(R.string.app_name);
あなたも使うことができます
getActivity().getResources().getString(R.string.app_name);
これが役立つことを願っています。
if (getActivity() == null) return;
いくつかのケースでも動作します。そこからコードの実行を中断し、アプリがクラッシュしないようにするだけです。
ロードされた設定を使用したアプリケーション設定アクティビティが表示されているときにも、同様の問題に直面しました。設定の1つを変更してから表示内容を回転させて設定を再度変更すると、フラグメント（私のPreferencesクラス）がアクティビティに添付されていないというメッセージが表示されてクラッシュします。
デバッグ時には、表示コンテンツが回転したときに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");
}
}
}
};
// ...
}
これが他の人に役立つことを願っています！
古い投稿ですが、私は最も賛成票のある回答に驚きました。
これに対する適切な解決策は、onStopでasynctaskをキャンセルすることです（またはあなたのフラグメントで適切なところはどこでも）。これにより、メモリリーク（破壊されたフラグメントへの参照を保持しているasynctask）を発生させず、フラグメント内で何が起こっているのかをより適切に制御できます。
@Override
public void onStop() {
super.onStop();
mYourAsyncTask.cancel(true);
}
次のように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とリソースのロードを回避できます。
私の場合はフラグメントメソッドが呼び出されました
getActivity().onBackPressed();