ウィンドウマネージャーのクラッシュにアタッチされていないビュー


188

ACRAを使用してアプリのクラッシュを報告しています。私はなっていたView not attached to window managerエラーメッセージを、私はラップすることにより、それを固定していたと思ったpDialog.dismiss();if文で:

if (pDialog!=null) 
{
    if (pDialog.isShowing()) 
    {
        pDialog.dismiss();   
    }
}

受けたView not attached to window managerクラッシュの量は減りましたが、それでもいくつかの問題が発生しており、解決方法がわかりません。

エラーメッセージ:

java.lang.IllegalArgumentException: View not attached to window manager
at android.view.WindowManagerGlobal.findViewLocked(WindowManagerGlobal.java:425)
at android.view.WindowManagerGlobal.removeView(WindowManagerGlobal.java:327)
at android.view.WindowManagerImpl.removeView(WindowManagerImpl.java:83)
at android.app.Dialog.dismissDialog(Dialog.java:330)
at android.app.Dialog.dismiss(Dialog.java:312)
at com.package.class$LoadAllProducts.onPostExecute(class.java:624)
at com.package.class$LoadAllProducts.onPostExecute(class.java:1)
at android.os.AsyncTask.finish(AsyncTask.java:631)
at android.os.AsyncTask.access$600(AsyncTask.java:177)
at android.os.AsyncTask$InternalHandler.handleMessage(AsyncTask.java:644)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:176)
at android.app.ActivityThread.main(ActivityThread.java:5419)
at java.lang.reflect.Method.invokeNative(Native Method)
at java.lang.reflect.Method.invoke(Method.java:525)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1046)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:862)
at dalvik.system.NativeStart.main(Native Method)

コードスニペット:

class LoadAllProducts extends AsyncTask<String, String, String> 
{

    /**
     * Before starting background thread Show Progress Dialog
     * */
    @Override
    protected void onPreExecute() 
    {
        super.onPreExecute();
        pDialog = new ProgressDialog(CLASS.this);
        pDialog.setMessage("Loading. Please wait...");
        pDialog.setIndeterminate(false);
        pDialog.setCancelable(false);
        pDialog.show();
    }

    /**
     * getting All products from url
     * */
    protected String doInBackground(String... args) 
    {
        // Building Parameters
        doMoreStuff("internet");
        return null;
    }


    /**
     * After completing background task Dismiss the progress dialog
     * **/
    protected void onPostExecute(String file_url) 
    {
         // dismiss the dialog after getting all products
         if (pDialog!=null) 
         {
                if (pDialog.isShowing()) 
                {
                    pDialog.dismiss();   //This is line 624!    
                }
         }
         something(note);
    }
}

マニフェスト:

    <activity
        android:name="pagename.CLASS" 
        android:configChanges="keyboard|keyboardHidden|orientation|screenSize|screenLayout"            
        android:label="@string/name" >
    </activity>

このクラッシュの発生を防ぐには何が欠けていますか?


1
これを理解したことはありますか?私も同じ問題を抱えています。理解できないようです。
トムジュン

残念だけど違う。少しで賞金がかかるかもしれません。それらがあなたを助ける場合に備えて、このように問題を処理している他のスレッドのいくつかをチェックする必要があります。
ハウリ14

あなたがAsyncTask内部で宣言されていますActivityFragment
エラキチン2014年

関数「doMoreStuff()」と「something()」を投稿してください。
凶暴2014年

この問題は、メインスレッドの処理が多すぎるために発生する可能性があるため、ハンドラーを使用して進捗ダイアログを表示し、(pDialog!= null)の場合、この行は不要です。isShowing自体がダイアログが進行中かどうかを確認するためです。
マドゥ

回答:


446

バグを再現する方法:

  1. デバイスでこのオプションを有効にしますSettings -> Developer Options -> Don't keep Activities
  2. AsyncTaskが実行され、が表示されている間に、ホームボタンを押しますProgressDialog

Android OSは、非表示になるとすぐにアクティビティを破棄します。ときにonPostExecute呼ばれているActivityになります「仕上げ」の状態とProgressDialogに接続されていないことになりますActivity

修正方法:

  1. onPostExecuteメソッドでアクティビティの状態を確認してください。
  2. ProgressDialogin onDestroyメソッドを閉じます。それ以外の場合は、android.view.WindowLeaked例外がスローされます。この例外は通常、アクティビティの終了時にまだアクティブなダイアログから発生します。

この固定コードを試してください:

public class YourActivity extends Activity {

    private void showProgressDialog() {
        if (pDialog == null) {
            pDialog = new ProgressDialog(StartActivity.this);
            pDialog.setMessage("Loading. Please wait...");
            pDialog.setIndeterminate(false);
            pDialog.setCancelable(false);
        }
        pDialog.show();
    }

    private void dismissProgressDialog() {
        if (pDialog != null && pDialog.isShowing()) {
            pDialog.dismiss();
        }
    }

    @Override
    protected void onDestroy() {
        dismissProgressDialog();
        super.onDestroy();
    }

    class LoadAllProducts extends AsyncTask<String, String, String> {

        // Before starting background thread Show Progress Dialog
        @Override
        protected void onPreExecute() {
            showProgressDialog();
        }

        //getting All products from url
        protected String doInBackground(String... args) {
            doMoreStuff("internet");
            return null;
        }

        // After completing background task Dismiss the progress dialog
        protected void onPostExecute(String file_url) {
            if (YourActivity.this.isDestroyed()) { // or call isFinishing() if min sdk version < 17
                return;
            }
            dismissProgressDialog();
            something(note);
        }
    }
}

8
正解です。ところで、isShowing()== falseの場合、dismiss()は何もしないため、isShowing()は不要です。ソースコード
Peter Zhao

3
使用することの利点は何であるisDestroyed()以上isFinishing()、この特定の目的のためのすべてのAPIには?
アレクサンダーアバクモフ2017

@AlexanderAbakumov:私が理解してisFinishing()いることからtrue、アクティビティがシステムによって破棄された場合の保証はありません。アクティビティに関するドキュメントを参照してください。
マーカスペンギン

1
@MarkusPenguin:そうです。しかし、この場合、作者のコメントからのアドバイスを使おうとすると// or call isFinishing() if min sdk version < 17、まったく同じ例外が発生します。そこで、我々は、17 <API上で動作しているアプリケーションのために、この答えは異なるソリューションが必要
アレクサンダーAbakumov

@AlexanderAbakumov私は同じ問題を抱えていますが、isFinishingは機能していません。AsyncCallback内の私のアクティビティにWeakReferenceを保存し、それから:myActivityWeakReference.get() != null && !myActivityWeakReference.get().isFinishing()
rusito23

34

問題がある可能性がActivityありますfinishedまたはされていprogress of finishingます。

チェックを追加し、isFinishingこれが場合にのみダイアログを閉じますfalse

if (!YourActivity.this.isFinishing() && pDialog != null) {
    pDialog.dismiss();
}

isFinishing: このアクティビティが呼び出し中finishであるか、他の誰かが終了を要求したために、このアクティビティが終了中であるかどうかを確認します。


1
チェックしても例外はスローされます
Marcos Vasconcelos

私のProgressDialogは、アクティビティではないクラスにあるため、isFinishingチェック@Libinを使用できません
Maniraj

10

Dialog作成したFragment場合は、次のコードを使用します。

ProgressDialog myDialog = new ProgressDialog(getActivity());
myDialog.setOwnerActivity(getActivity());
...
Activity activity = myDialog.getOwnerActivity();
if( activity!=null && !activity.isFinishing()) {
    myDialog.dismiss();
}

このパターンを使用して、Fragmentがから分離される場合に対処しActivityます。


8

ここでコードがどのように機能するかを見てください:

非同期タスクを呼び出した後、非同期タスクはバックグラウンドで実行されます。それが望ましい。これで、この非同期タスクには、コードの表示方法を尋ねると、アクティビティに添付された進行状況ダイアログが表示されます。

pDialog = new ProgressDialog(CLASS.this);

あなたはClass.this引数としてコンテキストを渡しています。そのため、Progressダイアログはまだアクティビティにアタッチされています。

ここでシナリオを考えてみましょう。finish()メソッドを使用してアクティビティを終了しようとしたときに、非同期タスクの進行中に、アクティビティにアタッチされたリソースにアクセスしようとしているポイント、つまりprogress barアクティビティがなくなったときですそこ。

したがって、次のようになります。

java.lang.IllegalArgumentException: View not attached to the window manager

これに対する解決策:

1)アクティビティが完了する前に、ダイアログボックスが閉じられているかキャンセルされていることを確認してください。

2)ダイアログボックスが閉じられた後、つまり非同期タスクが終了した後にのみ、アクティビティを終了します。


1
#2について:を終了するタイミングを完全に制御することはできませんActivity。Androidはいつでも終了する可能性があります。したがって、#2は意味がありません。
アレクサンダーアバクモフ2018

6

@erakitinの回答に基づいていますが、Androidバージョン<APIレベル17にも対応しています。悲しいことに、Activity.isDestroyed()はAPIレベル17以降でのみサポートされているため、私のように古いAPIレベルをターゲットにしている場合は、自分で確認してください。View not attached to window managerその後も例外はありません。

コード例

public class MainActivity extends Activity {
    private TestAsyncTask mAsyncTask;
    private ProgressDialog mProgressDialog;
    private boolean mIsDestroyed;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        if (condition) {
            mAsyncTask = new TestAsyncTask();
            mAsyncTask.execute();
        }
    }

    @Override
    protected void onResume() {
        super.onResume();

        if (mAsyncTask != null && mAsyncTask.getStatus() != AsyncTask.Status.FINISHED) {
            Toast.makeText(this, "Still loading", Toast.LENGTH_LONG).show();
            return;
        }
    }

    @Override
    protected void onDestroy() {
        super.onDestroy();
        mIsDestroyed = true;

        if (mProgressDialog != null && mProgressDialog.isShowing()) {
            mProgressDialog.dismiss();
        }
    }

    public class TestAsyncTask extends AsyncTask<Void, Void, AsyncResult> {    
        @Override
        protected void onPreExecute() {
            super.onPreExecute();
            mProgressDialog = ProgressDialog.show(MainActivity.this, "Please wait", "doing stuff..");
        }

        @Override
        protected AsyncResult doInBackground(Void... arg0) {
            // Do long running background stuff
            return null;
        }

        @Override
        protected void onPostExecute(AsyncResult result) {
            // Use MainActivity.this.isDestroyed() when targeting API level 17 or higher
            if (mIsDestroyed)// Activity not there anymore
                return;

            mProgressDialog.dismiss();
            // Handle rest onPostExecute
        }
    }
}

4
@Override
public void onPause() {
    super.onPause();

    if(pDialog != null)
        pDialog .dismiss();
    pDialog = null;
}

これを参照してください。


3

onConfigurationChangedをオーバーライドして、進行状況ダイアログを閉じます。進行状況ダイアログが縦向きで作成され、横向きで閉じると、ウィンドウマネージャーにアタッチされていないビューエラーがスローされます。

また、進行状況バーを停止し、onPause()、onBackPressed、およびonDestroyメソッドで非同期タスクを停止します。

if(asyncTaskObj !=null && asyncTaskObj.getStatus().equals(AsyncTask.Status.RUNNING)){

    asyncTaskObj.cancel(true);

}

3

アクティビティのonDestroyをオーバーライドしてダイアログを閉じ、nullにします

protected void onDestroy ()
    {
        if(mProgressDialog != null)
            if(mProgressDialog.isShowing())
                mProgressDialog.dismiss();
        mProgressDialog= null;
    }

3

まず、クラッシュの理由はdecorViewのインデックスが-1であることです。Androidのソースコードからそれを知ることができます。コードスニペットがあります。

class:android.view.WindowManagerGlobal

file:WindowManagerGlobal.java

private int findViewLocked(View view, boolean required) {
        final int index = mViews.indexOf(view);
//here, view is decorView,comment by OF
        if (required && index < 0) {
            throw new IllegalArgumentException("View=" + view + " not attached to window manager");
        }
        return index;
    }

したがって、フォロー解決を取得し、decorViewのインデックスを判断します。それが0より大きい場合は、続行するか、単に戻って却下を中止し、次のようにコード化します。

try {
            Class<?> windowMgrGloable = Class.forName("android.view.WindowManagerGlobal");
            try {
                Method mtdGetIntance = windowMgrGloable.getDeclaredMethod("getInstance");
                mtdGetIntance.setAccessible(true);
                try {
                    Object windownGlobal = mtdGetIntance.invoke(null,null);
                    try {
                        Field mViewField = windowMgrGloable.getDeclaredField("mViews");
                        mViewField.setAccessible(true);
                        ArrayList<View> mViews = (ArrayList<View>) mViewField.get(windownGlobal);
                        int decorViewIndex = mViews.indexOf(pd.getWindow().getDecorView());
                        Log.i(TAG,"check index:"+decorViewIndex);
                        if (decorViewIndex < 0) {
                            return;
                        }
                    } catch (NoSuchFieldException e) {
                        e.printStackTrace();
                    }
                } catch (IllegalAccessException e) {
                    e.printStackTrace();
                } catch (InvocationTargetException e) {
                    e.printStackTrace();
                }
            } catch (NoSuchMethodException e) {
                e.printStackTrace();
            }
        } catch (ClassNotFoundException e) {
            e.printStackTrace();
        }
        if (pd.isShowing()) {
            pd.dismiss();
        }

2

次のdismiss()ようにメソッドをオーバーライドします。

@Override
public void dismiss() {
    Window window = getWindow();
    if (window == null) {
        return;
    }
    View decor = window.getDecorView();
    if (decor != null && decor.getParent() != null) {
        super.dismiss();
    }
}

問題を再現するには、ダイアログを閉じる前にアクティビティを終了してください。


1

最善の解決策。最初のコンテキストをチェックし、活動状況が続いチェック活動が終了しただけか、次に呼び出す場合、活動状況やアプリケーションのコンテキストですdialog.show()dialog.dismiss();

以下のサンプルコードを参照してください ...参考になれば幸いです。

ダイアログを表示

if (context instanceof Activity) {
   if (!((Activity) context).isFinishing())
     dialog.show();
}

ダイアログを閉じる

if (context instanceof Activity) {
       if (!((Activity) context).isFinishing())
         dialog.dismiss();
    }

さらにチェックを追加するdialog.isShowing()場合はdialog !-null&&条件を追加または使用します。


0

この問題は、dismiss関数が呼び出される前にアクティビティが完了するためです。例外を処理し、正確な理由についてADBログを確認してください。

/**
     * After completing background task Dismiss the progress dialog
     * **/
    protected void onPostExecute(String file_url) {
    try {
         if (pDialog!=null) {
            pDialog.dismiss();   //This is line 624!    
         }
    } catch (Exception e) {
        // do nothing
    }
     something(note);
}

0

この例外を再現する方法を得ました。

私は2を使用しますAsyncTask。1つは長いタスクを実行し、もう1つは短いタスクを実行します。短いタスクが完了したら、を呼び出しますfinish()。長いタスクが完了してを呼び出すとDialog.dismiss()、クラッシュします。

これが私のサンプルコードです:

public class MainActivity extends Activity {
    private static final String TAG = "MainActivity";
    private ProgressDialog mProgressDialog;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        Log.d(TAG, "onCreate");

        new AsyncTask<Void, Void, Void>(){
            @Override
            protected void onPreExecute() {
                mProgressDialog = ProgressDialog.show(MainActivity.this, "", "plz wait...", true);
            }

            @Override
            protected Void doInBackground(Void... nothing) {
                try {
                    Log.d(TAG, "long thread doInBackground");
                    Thread.sleep(20000);
                } catch (InterruptedException e) {
                    e.printStackTrace();
                }

                return null;
            }

            @Override
            protected void onPostExecute(Void result) {
                Log.d(TAG, "long thread onPostExecute");
                if (mProgressDialog != null && mProgressDialog.isShowing()) {
                    mProgressDialog.dismiss();
                    mProgressDialog = null;
                }
                Log.d(TAG, "long thread onPostExecute call dismiss");
            }
        }.executeOnExecutor(AsyncTask.THREAD_POOL_EXECUTOR);

        new AsyncTask<Void, Void, Void>(){
            @Override
            protected Void doInBackground(Void... params) {
                try {
                    Log.d(TAG, "short thread doInBackground");
                    Thread.sleep(5000);
                } catch (InterruptedException e) {
                    e.printStackTrace();
                }
                return null;
            }

            @Override
            protected void onPostExecute(Void aVoid) {
                super.onPostExecute(aVoid);
                Log.d(TAG, "short thread onPostExecute");
                finish();
                Log.d(TAG, "short thread onPostExecute call finish");
            }
        }.executeOnExecutor(AsyncTask.THREAD_POOL_EXECUTOR);
    }

    @Override
    protected void onDestroy() {
        super.onDestroy();
        Log.d(TAG, "onDestroy");
    }
}

これを試して、この問題を解決するための最良の方法を見つけることができます。私の研究から、それを修正するには少なくとも4つの方法があります:

  1. @erakitinの回答:isFinishing()アクティビティの状態を確認するための呼び出し
  2. @Kapéの答え:アクティビティの状態をチェックするフラグを設定します
  3. それを処理するには、try / catchを使用します。
  4. に電話AsyncTask.cancel(false)してくださいonDestroy()。asynctaskが実行されonPostExecute()ないようonCancelled()にしますが、代わりに実行します。
    注:Android 2.XXなどの古いAndroid OS onPostExecute()を呼び出しAsyncTask.cancel(false)ても実行されます

最適なものをお選びいただけます。


0

pDialogをグローバルに初期化してから、pDialogを削除して、ローカルでビューまたはダイアログを初期化することができます。同じ問題が発生しました。これで問題は解決しました。それがあなたのために働くことを願っています。


0

onPauseメソッドまたはonDestroyメソッドに関するダイアログも閉じます

@Override
protected void onPause() {
    super.onPause();
    dialog.dismiss();
}

@Override
protected void onDestroy() {
    super.onDestroy();
    dialog.dismiss();
}
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.