シナリオ開発中、一度は経験したことがあるでしょう。手動テストでは完璧に動作していた機能が、せっかちなユーザーがボタンを連打した瞬間に壊れてしまうことがあります。もし誰かが「削除」や「送信」ボタンを200ミリ秒以内に2回タップすると、アプリが突然終了してしまうかもしれません。これは典型的なレースコンディション（競合状態）です。最初のタップでダイアログのトランザクションが開始されますが、それが完了する前に2回目のタップが実行されてしまいます。その結果、恐ろしい IllegalStateException が発生します。
スタックトレース```
java.lang.IllegalStateException: Fragment already added: MyDialogFragment{6e9f1a2 #0 MyDialogFragmentTag} at androidx.fragment.app.FragmentStore.addFragment(FragmentStore.java:67) at androidx.fragment.app.FragmentManager.addFragment(FragmentManager.java:1555) at androidx.fragment.app.BackStackRecord.executeOps(BackStackRecord.java:405) at androidx.fragment.app.FragmentManager.executeOpsTogether(FragmentManager.java:1890) at androidx.fragment.app.FragmentManager.removeRedundantOperationsAndExecute(FragmentManager.java:1845) at androidx.fragment.app.FragmentManager.execPendingActions(FragmentManager.java:1747)
## 根本原因内部的には、`DialogFragment.show()` は `FragmentTransaction` をトリガーします。`FragmentManager` はすべてのアクティブなフラグメントを厳密に管理しています。すでに `isAdded == true` とマークされているインスタンスをさらに追加しようとすると、システムはパニックを起こします。UIの状態が破損するのを防ぐために、例外をスローするのです。
これは通常、以下の2つのケースで発生します：
- **ボタンの連打:** 最初の処理が完了する前に、2つの `show()` 呼び出しが実行される。- **インスタンスの再利用:** ViewModelやActivity内で単一のダイアログ参照を保持し、現在の状態を確認せずに `show()` を複数回呼び出す。## 解決策1：findFragmentByTagで存在を確認する闇雲に `show()` を呼び出すのはやめましょう。代わりに、ダイアログがすでに存在するかどうかを `FragmentManager` に問い合わせます。これは、ダイアログの重複を防ぐための最も堅牢な防御策です。
private void showMyDialog() { FragmentManager fm = getSupportFragmentManager(); // ユニークなタグを使用してフラグメントを探す MyDialogFragment dialog = (MyDialogFragment) fm.findFragmentByTag("MyDialogTag");
if (dialog == null) {
// 既存のフラグメントが見つからないため、新しく表示しても安全
dialog = new MyDialogFragment();
dialog.show(fm, "MyDialogTag");
} else if (!dialog.isAdded()) {
// インスタンスは存在するが、現在は表示またはアタッチされていない
dialog.show(fm, "MyDialogTag");
}
}
## 解決策2：isAdded() でガードする存続期間の長いActivityなどでダイアログのローカル参照を保持している場合は、必ず最初に `isAdded()` プロパティを確認してください。この単純なチェックがゲートキーパーとして機能します。
private MyDialogFragment myDialog;
private void triggerDialog() { if (myDialog == null) { myDialog = new MyDialogFragment(); }
if (!myDialog.isAdded()) {
myDialog.show(getSupportFragmentManager(), "unique_tag");
}
}
## 解決策3：UIクリックのデバウンス（多重押し防止）クラッシュを発生源で防ぐことは、ユーザーエクスペリエンスにとっても多くの場合最善です。単純なタイムスタンプのチェックを使用して、500ミリ秒以内などの短時間に行われた複数のクリックをブロックできます。
private long lastClickTime = 0;
public void onButtonClick(View view) { // 前回のクリックから500ミリ秒以内に発生したクリックをブロックする if (SystemClock.elapsedRealtime() - lastClickTime < 500) { return; } lastClickTime = SystemClock.elapsedRealtime();
showMyDialog();
}
## 解決策4：コールバック中の状態喪失（State Loss）の処理非同期コールバックは厄介です。ユーザーが他のアプリを見ている間にネットワークリクエストが完了した場合、ダイアログを表示しようとすると「Can not perform this action after onSaveInstanceState」というクラッシュが発生します。このような稀で重要度の低いシナリオでは、手動でトランザクションをコミットできます。
public void showSafely(FragmentManager manager, String tag) { if (manager.isStateSaved()) return;
FragmentTransaction ft = manager.beginTransaction();
ft.add(this, tag);
ft.commitAllowingStateLoss();
}
**警告:** `commitAllowingStateLoss()` の使用は控えめにしてください。アプリがバックグラウンドにいる間にシステムがプロセスを終了させた場合、ダイアログが消えてしまう可能性があります。
## 予防のためのベストプラクティス- **定数タグを使用する:** 複数の場所に文字列をハードコードしないでください。すべてのダイアログに対して `private static final String TAG` を定義しましょう。- **ライフサイクルの意識:** LiveDataを使用してダイアログを表示する場合は、必ず `viewLifecycleOwner` を使用してください。これにより、古いオブザーバーが実行されるのを防げます。- **静的参照を避ける:** フラグメントを static 変数に保持すると、メモリリークの原因になります。永続化は `FragmentManager` や `ViewModel` に任せましょう。## 検証手順以下の3つのテストで修正を確認してください。
- **ストレステスト:** トリガーボタンを5秒間できるだけ速くタップします。アプリが終了せず、ダイアログが1つだけ表示されれば成功です。- **回転テスト:** ダイアログを開いた状態で、端末を横向きにします。ダイアログが表示されたままであり、再生成されたActivityがクラッシュしないことを確認してください。- **ADB Monkey:** 自動ストレステストを素早く実行します。このコマンドは500個のランダムなイベントを生成し、エッジケースを見つけ出します。 ```
adb shell monkey -p your.package.name -v 500