视图未附加到窗口管理器崩溃

Question

我正在使用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>

我需要做什么才能阻止这次崩溃发生？

- Howli

---

你解决了这个问题吗？我也遇到了同样的问题。似乎无法解决。 - tomjung

很遗憾，我现在不行。我可能会稍后发起一个赏金。你应该查看一些其他处理此问题的线程，以防它们能帮到你。 - Howli

你的 AsyncTask 是在 Activity 或 Fragment 中声明的吗？ - erakitin

请发布函数“doMoreStuff()”和“something()”。 - berserk

问题可能是主线程的工作过多，所以尝试使用处理程序来显示进度对话框，如果 (pDialog!=null) 这一行不需要，因为 isShowing 本身检查对话框是否正在进行中。 - Madhu

Answer 1

如何重现此错误:

1. 在您的设备上启用此选项：设置 -> 开发人员选项 -> 不保留活动。
2. 在AsyncTask正在执行且ProgressDialog正在显示时，按下主页按钮。

Android操作系统将在活动被隐藏后立即销毁该活动。当调用onPostExecute时，该Activity将处于完成状态，并且该ProgressDialog不会附加到Activity上。

如何修复此问题：

1. 在您的onPostExecute方法中检查活动状态。
2. 在onDestroy方法中关闭ProgressDialog。否则，将抛出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);
        }
    }
}

- erakitin

---

好的！顺便提一句，isShowing() 不必要，因为如果 isShowing() == false，dismiss() 不会执行任何操作。源代码 - Peter Zhao

在这个特定的情况下，使用 isDestroyed() 相比于 isFinishing() 对所有 API 的好处是什么？ - Alexander Abakumov

据我所了解，如果活动被系统销毁，请参阅有关活动的文档，isFinishing()不能保证为 true。(@AlexanderAbakumov) - Markus Penguin

@MarkusPenguin：没错。但是，在这种情况下，如果尝试使用作者评论中的建议“// or call isFinishing() if min sdk version < 17”，则会遇到完全相同的异常。因此，对于在API < 17上运行的应用程序，我们需要与此答案不同的解决方案。 - Alexander Abakumov

@AlexanderAbakumov 我也遇到了同样的问题，但 isFinishing 对我没用。在 AsyncCallback 中保存一个对我的 Activity 的 WeakReference 并使用以下代码解决了问题：myActivityWeakReference.get() != null && !myActivityWeakReference.get().isFinishing()。 - rusito23

Answer 2

问题可能是Activity已经被完成或正在完成的过程中。

添加一个检查isFinishing，只有在此为false时才关闭对话框。

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

isFinishing：检查此活动是否正在完成过程中，即您调用finish或其他人已请求其完成。

- Libin

---

通过检查，异常仍然被抛出。 - Marcos Vasconcelos

我的ProgressDialog在一个非Activity类中，因此我无法使用isFinishing检查 @Libin - Maniraj

对我来说，isFinishing 没有起作用。这个条件可以工作（Kotlin）：if(![e-mail])。 - Mikhail

Answer 3

对于在Fragment中创建的Dialog，我使用以下代码：

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

我使用这个模式来处理Fragment可能从Activity中分离的情况。

- Teng-pao Yu

---

我相信这仅适用于处理声音的活动，对吗？ - Chisko

Answer 4

在这里查看代码的工作方式：

调用异步任务后，异步任务在后台运行。这是期望的。现在，这个异步任务有一个进度对话框，它附加到 Activity 上，如果你想知道如何查看代码：

pDialog = new ProgressDialog(CLASS.this);

你正在将Class.this作为上下文传递给参数。因此，进度对话框仍然附加到活动中。

现在考虑以下场景： 如果我们在异步任务进行中尝试使用finish()方法完成活动，则在活动不再存在时，即在访问附加到活动的资源（即进度条）的点处会出现问题。

因此你会得到：

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

解决方法：

1）确保在活动结束之前，对话框已被关闭或取消。

2）仅在对话框被关闭，即异步任务完成后，才结束活动。

- Kailas

---

关于#2：你无法完全控制何时完成该“Activity”；Android可能随时结束它。因此，#2没有意义。 - Alexander Abakumov

Answer 5

参考@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
        }
    }
}

- Kapé

Answer 6

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

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

请参考这个链接。

- Meghna

Answer 7

重写 Activity 的 onDestroy 方法并关闭您的对话框，并将其设置为 null。

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

- surya

Answer 8

覆盖onConfigurationChanged并解除进度对话框。 如果进度对话框在纵向创建并在横向解除，则会抛出“View not attached to window manager”错误。

此外，还要在onPause()、onBackPressed和onDestroy方法中停止进度条和异步任务。

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

    asyncTaskObj.cancel(true);

}

- Pratapi Hemant Patel

Answer 9

覆盖dismiss()方法，像这样：

最初的回答

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

为了重现该问题，只需在关闭对话框之前完成活动。最初的回答。

- thundertrick

Answer 10

首先，崩溃的原因是 decorView 的索引为 -1，在 Android 源代码中可以得知，以下是代码片段：

类名：android.view.WindowManagerGlobal

文件名：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，如果是，则继续执行；否则，返回并放弃dismiss。代码如下：

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

- aolphn

---

这是一种危险的黑客行为，依赖于许多内部实现细节。 - hqzxzwb

这是一种保护措施，而非黑客行为。 - aolphn

看到有人否认这是黑客攻击，实际上相当令人震惊。 - hqzxzwb

四年后，我找到了答案，感谢自己，^_^!!! - aolphn