Вид не привязан к сбою оконного менеджера

190

Я использую 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);
    }
}

Manifest:

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

Чего мне не хватает, чтобы остановить этот сбой?

Howli
источник
1
Вы когда-нибудь понимали это? У меня та же проблема. Не могу понять это.
Tomjung
К сожалению нет. Я мог бы начать щедрость через немного. Вам следует проверить некоторые другие темы, связанные с этой проблемой, на случай, если они вам помогут.
Howli
Ваш AsyncTaskобъявлен внутри Activityили Fragment?
эракитин
Пожалуйста, отправьте функции "doMoreStuff ()" и "что-то ()".
берсерк
Проблема может быть из-за слишком большой работы основного потока, поэтому попробуйте использовать обработчики, чтобы отобразить диалоги выполнения, и если (pDialog! = Null) эта строка не нужна, так как isShowing сам проверяет, выполняется ли диалог или нет.
Мадху,

Ответы:

446

Как воспроизвести ошибку:

  1. Включите эту опцию на устройстве: Settings -> Developer Options -> Don't keep Activities.
  2. Нажмите кнопку «Домой» во время AsyncTaskвыполнения и ProgressDialogотображения.

ОС Android уничтожит активность, как только она будет скрыта. Когда onPostExecuteвызывается, Activityон будет в «завершающем» состоянии и ProgressDialogне будет привязан к Activity.

Как это исправить:

  1. Проверьте состояние активности в вашем onPostExecuteметоде.
  2. Оставить ProgressDialogв 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);
        }
    }
}
erakitin
источник
10
Отличный ответ! Кстати, isShowing () не нужен, потому что dismiss () ничего не сделает, если isShowing () == false. Исходный код
Питер Чжао
3
В чем преимущество использования isDestroyed()over isFinishing()на всех API для этой конкретной цели?
Александр Абакумов
@AlexanderAbakumov: Насколько я понимаю, isFinishing()гарантированно не будет, trueесли действие уничтожено системой, см. Документацию по действиям .
Маркус Пингвин
1
@MarkusPenguin: Верно. Но в этом случае, если кто-то попытается воспользоваться советом из комментария автора // or call isFinishing() if min sdk version < 17, он столкнется с тем же исключением. Таким образом, нам нужно другое решение, чем этот ответ для приложений, работающих на API <17.
Александр Абакумов
@AlexanderAbakumov У меня та же проблема, но у меня не работает отделка, сработало сохранение слабой ссылки на мою активность в AsyncCallback, а затем:myActivityWeakReference.get() != null && !myActivityWeakReference.get().isFinishing()
rusito23
36

Проблема может заключаться в том, что Activityбыл finishedили в progress of finishing.

Добавить проверку isFinishingи закрыть диалог только тогда, когда этоfalse

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

isFinishing: Проверьте, находится ли это действие в процессе завершения, потому что вы вызвали finishего или кто-то еще запросил его завершение.

Либин
источник
1
С проверкой исключение все еще выбрасывается
Маркос Васконселос
Мой ProgressDialog находится в классе, который не является деятельностью, поэтому я не могу использовать проверку isFinishing @Libin
Манираж
12

Для Dialogсозданного в Fragmentя использую следующий код:

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

Я использую этот шаблон, чтобы иметь дело со случаем, когда a Fragmentможет быть отсоединен от Activity.

Тэн-пао ю
источник
8

Посмотрите, как работает Код здесь:

После вызова асинхронной задачи асинхронная задача запускается в фоновом режиме. это желательно. Теперь у этой асинхронной задачи есть диалоговое окно прогресса, которое прикреплено к Activity, если вы спросите, как просмотреть код:

pDialog = new ProgressDialog(CLASS.this);

Вы передаете в Class.thisкачестве контекста аргумент. Таким образом, диалог «Ход выполнения» все еще привязан к действию.

Теперь рассмотрим сценарий: если мы пытаемся завершить действие, используя метод finish (), когда выполняется асинхронная задача, это точка, где вы пытаетесь получить доступ к ресурсу, прикрепленному к действию, т.е. progress bar Когда действия больше нет там.

Отсюда вы получаете:

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

Решение этого:

1) Убедитесь, что диалоговое окно закрыто или отменено до завершения действия.

2) Завершите действие, только после закрытия диалогового окна, то есть асинхронная задача завершена.

Кайлас
источник
1
Что касается # 2: у вас нет полного контроля над тем, когда закончить Activity; Android может закончить это в любое время. Итак, № 2 не имеет смысла.
Александр Абакумов
7

Основано на ответе @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
        }
    }
}
Kape
источник
4
@Override
public void onPause() {
    super.onPause();

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

передайте это .

Meghna
источник
3

Переопределить onConfigurationChanged и закрыть диалоговое окно прогресса. Если диалоговое окно прогресса создается в книжной ориентации и отклоняется в альбомной, то при этом выдается ошибка «Вид не прикреплен к диспетчеру окон».

Также остановите индикатор выполнения и остановите асинхронную задачу в методах onPause (), onBackPressed и onDestroy.

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

    asyncTaskObj.cancel(true);

}
Пратапи Хемант Патель
источник
3

Переопределите onDestroy of the Activity и закройте диалог и сделайте его пустым

protected void onDestroy ()
    {
        if(mProgressDialog != null)
            if(mProgressDialog.isShowing())
                mProgressDialog.dismiss();
        mProgressDialog= null;
    }
сурйа
источник
3

Во-первых, причина сбоя в том, что индекс 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, затем продолжаем или просто возвращаем и прекращаем выполнение, кодируйте следующим образом:

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
источник
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();
    }
}

Чтобы воспроизвести проблему, просто завершите действие, прежде чем закрыть диалоговое окно.

thundertrick
источник
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. Один выполняет длинное задание, а другой - короткое задание. После выполнения короткого задания позвоните 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. @ ответ эракитина: позвоните, isFinishing()чтобы проверить состояние деятельности
  2. @ Капе ответ: установите флаг, чтобы проверить состояние деятельности
  3. Используйте try / catch, чтобы справиться с этим.
  4. Звоните AsyncTask.cancel(false)в onDestroy(). Это предотвратит выполнение асинхронной задачи, onPostExecute()но выполнит ее onCancelled().
    Примечание: onPostExecute()все равно будет выполняться даже при вызове AsyncTask.cancel(false)на более старой ОС Android, такой как Android 2.XX

Вы можете выбрать лучший для вас.

Юэ-Минг Чиен
источник
0

Может быть, вы инициализируете pDialog глобально, затем удалите его и инициализируйте ваш вид или диалог локально. У меня та же проблема, я сделал это, и моя проблема решена. Надеюсь, это сработает для тебя.

Шашват Гупта
источник
0

мы также отклонили наш диалог о onPauseметоде или onDestroyметоде

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

@Override
protected void onDestroy() {
    super.onDestroy();
    dialog.dismiss();
}
тарун бхола
источник