Visualização não anexada à falha do gerenciador de janelas

188

Estou usando o ACRA para relatar falhas no aplicativo. Eu estava recebendo uma View not attached to window managermensagem de erro e pensei que tinha corrigido, envolvendo a pDialog.dismiss();instrução if:

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

Reduziu a quantidade de View not attached to window managerfalhas que recebo, mas ainda estou recebendo algumas e não tenho certeza de como resolvê-la.

Mensagem de erro:

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)

Fragmento de código:

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

Manifesto:

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

O que estou perdendo para impedir que esse acidente aconteça?

Howli
fonte
1
Você já descobriu isso? Estou tendo o mesmo problema. Não consigo entender.
precisa saber é o seguinte
Infelizmente não. Eu posso começar uma recompensa daqui a pouco. Você deve verificar alguns dos outros tópicos que lidam com esse problema, caso eles o ajudem.
precisa saber é o seguinte
Seu AsyncTaské declarado dentro Activityou Fragment?
Erakitin # 10/14
Por favor, publique as funções "doMoreStuff ()" e "something ()".
berserk
O problema pode ser causado por muito trabalho do encadeamento principal, portanto, tente usar manipuladores para mostrar o diálogo de progresso e se (pDialog! = Null) essa linha não for necessária, porque isShowing se verifica se a caixa de diálogo está em andamento ou não.
Madhu 16/05

Respostas:

446

Como reproduzir o bug:

  1. Ative essa opção no seu dispositivo: Settings -> Developer Options -> Don't keep Activities.
  2. Pressione o botão Início enquanto AsyncTaskestiver em execução e em ProgressDialogexibição.

O sistema operacional Android destruirá uma atividade assim que estiver oculta. Quando onPostExecuteé chamado, o estado Activityestará em "acabamento" e ProgressDialognão será anexado Activity.

Como corrigi-lo:

  1. Verifique o estado da atividade no seu onPostExecutemétodo.
  2. Ignore o método ProgressDialogin onDestroy. Caso contrário, a android.view.WindowLeakedexceção será lançada. Essa exceção geralmente vem de diálogos que ainda estão ativos quando a atividade está terminando.

Experimente este código fixo:

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
fonte
8
Ótima resposta! BTW, isShowing () é desnecessário, porque unlock () não fará nada se isShowing () == false. Código-fonte
Peter Zhao
3
Qual é a vantagem de usar isDestroyed()ao longo isFinishing()de todas as APIs para esta finalidade específica?
Alexander Abakumov
@AlexanderAbakumov: Pelo que entendi, isFinishing()não é garantido trueque a atividade seja destruída pelo sistema, consulte a documentação em Atividades .
Markus Penguin
1
@MarkusPenguin: Certo. Mas, nesse caso, se alguém tentar usar um conselho do comentário do autor // or call isFinishing() if min sdk version < 17, ele se depara com a mesma exceção. Então, precisamos de uma solução diferente do que esta resposta para aplicativos que funcionam em API <17.
Alexander Abakumov
@AlexanderAbakumov Estou tendo o mesmo problema, mas isFinishing não está funcionando para mim, Ele trabalhou salvar um WeakReference à minha atividade dentro do AsyncCallback, e depois:myActivityWeakReference.get() != null && !myActivityWeakReference.get().isFinishing()
rusito23
34

Questão poderia ser que o Activitytenham sido finishedou em progress of finishing.

Adicione uma verificação isFinishinge descartar a caixa de diálogo apenas quando esta forfalse

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

isFinishing: verifique se esta atividade está em processo de conclusão, porque você a chamou finishou alguém solicitou a conclusão.

Libin
fonte
1
Com o cheque, a exceção ainda está sendo lançada
Marcos Vasconcelos
Meu ProgressDialog está em uma classe que não é uma atividade assim que eu sou incapaz de usar o cheque isFinishing @Libin
Maniraj
10

Para Dialogcriado em a Fragment, eu uso o seguinte código:

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

Eu uso esse padrão para lidar com o caso em que um Fragmentpode ser desanexado do Activity.

Teng-pao Yu
fonte
8

Veja como o Código está funcionando aqui:

Depois de chamar a tarefa assíncrona, a tarefa assíncrona é executada em segundo plano. isso é desejável. Agora, esta tarefa assíncrona tem uma caixa de diálogo de progresso anexada à atividade, se você perguntar como ver o código:

pDialog = new ProgressDialog(CLASS.this);

Você está passando o Class.thiscontexto como para o argumento. Portanto, a caixa de diálogo Progresso ainda está anexada à atividade.

Agora considere o cenário: Se tentarmos concluir a atividade usando o método finish (), enquanto a tarefa assíncrona estiver em andamento, é o ponto em que você está tentando acessar o Recurso anexado à atividade, ou seja, progress barquando a atividade não está mais há.

Daí você obtém:

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

Solução para isso:

1) Verifique se a caixa de diálogo foi descartada ou cancelada antes do término da atividade.

2) Conclua a atividade, somente depois que a caixa de diálogo for descartada, ou seja, a tarefa assíncrona terminou.

Kailas
fonte
1
Em relação ao item 2: você não tem controle total sobre quando terminar o Activity; O Android pode terminar a qualquer momento. Então, o nº 2 não faz sentido.
Alexander Abakumov
6

Com base na resposta @erakitin, mas também compatível com as versões do Android <nível API 17. O Sadly Activity.isDestroyed () só é suportado desde o nível 17 da API. Portanto, se você estiver direcionando para um nível API mais antigo como eu, será necessário verifique você mesmo. Não tenho a View not attached to window managerexceção depois disso.

Código de exemplo

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é
fonte
4
@Override
public void onPause() {
    super.onPause();

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

consulte isso .

Meghna
fonte
3

Substituir onConfigurationChanged e descartar a caixa de diálogo de progresso. Se a caixa de diálogo de progresso for criada em modo retrato e for descartada em paisagem, ocorrerá o erro Ver não anexado ao gerenciador de janelas.

Pare também a barra de progresso e a tarefa assíncrona nos métodos onPause (), onBackPressed e onDestroy.

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

    asyncTaskObj.cancel(true);

}
Pratapi Hemant Patel
fonte
3

Substituir na destruição da atividade e dispensar seu diálogo e torná-lo nulo

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

Em primeiro lugar, o motivo da falha é o índice do decorView é -1, podemos conhecê-lo a partir do código fonte do Android, existe um trecho de código:

classe: android.view.WindowManagerGlobal

arquivo: 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;
    }

para que possamos seguir a resolução, apenas julgue o índice do decorView, se mais de 0 continuar ou simplesmente retornar e desistir, descubra o código da seguinte maneira:

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
fonte
2

Substitua o dismiss()método como este:

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

Para reproduzir o problema, basta concluir a atividade antes de fechar a caixa de diálogo.

thundertrick
fonte
1

melhor solução. Verificar o primeiro contexto é o contexto da atividade ou o contexto do aplicativo se o contexto da atividade somente verificar a atividade concluída ou não chamar dialog.show()oudialog.dismiss();

Veja o código de exemplo abaixo ... espero que seja útil!

Caixa de diálogo Exibir

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

Ignorar caixa de diálogo

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

Se você deseja adicionar mais verificações, adicione dialog.isShowing()ou dialog !-nulluse a &&condição.

Yogesh Rathi
fonte
0

Esse problema ocorre porque sua atividade é concluída antes que a função de dispensa seja chamada. Manipule a exceção e verifique seu log do ADB pelo motivo exato.

/**
     * 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);
}
Rajiv Manivannan
fonte
0

Eu tenho uma maneira de reproduzir essa exceção.

Eu uso 2 AsyncTask. Um faz tarefas longas e outro faz tarefas curtas. Após a conclusão da tarefa curta, ligue finish(). Quando uma tarefa longa é concluída e é chamada Dialog.dismiss(), ela falha.

Aqui está o meu código de exemplo:

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

Você pode tentar isso e descobrir qual é a melhor maneira de corrigir esse problema. No meu estudo, há pelo menos quatro maneiras de corrigi-lo:

  1. @ resposta de erakitin: ligue isFinishing()para verificar o estado da atividade
  2. @ Resposta de Kapé: defina o sinalizador para verificar o estado da atividade
  3. Use try / catch para lidar com isso.
  4. Chamada AsyncTask.cancel(false)no onDestroy(). Isso impedirá que o asynctask seja executado, onPostExecute()mas é executado onCancelled().
    Nota: onPostExecute()ainda será executado mesmo que você ligue AsyncTask.cancel(false)no sistema operacional Android mais antigo, como o Android 2.XX

Você pode escolher o melhor para você.

Yueh-Ming Chien
fonte
0

Pode ser que você inicialize o pDialog globalmente, remova-o e inicialize sua exibição ou caixa de diálogo localmente. Eu tenho o mesmo problema, eu fiz isso e meu problema foi resolvido. Espero que funcione para você.

Shashwat Gupta
fonte
0

também descartamos nosso diálogo sobre onPausemétodo ou onDestroymétodo

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

@Override
protected void onDestroy() {
    super.onDestroy();
    dialog.dismiss();
}
tarun bhola
fonte