¿Debo verificar si ProgressBar está mostrando antes de descartarlo?

Question

Im despedir diálogo de progreso cuando se termina AsyncTask. ¿Debo verificar isShowing antes de descartarlo?

He intentado eliminar este cheque y funciona normalmente, pero puede haber hay trampas ocultas?

if (progressDialog.isShowing()) { 
    progressDialog.dismiss(); 
} 

Answer 1

Is parece que se comprueba el interior de aplicación:

public void dismiss() { 
    if (Thread.currentThread() != mUiThread) { 
     mHandler.post(mDismissAction); 
    } else { 
     mDismissAction.run(); 
    } 
} 

private void dismissDialog() { 
    if (mDecor == null || !mShowing) { 
     return; 
    } 

    try { 
     mWindowManager.removeView(mDecor); 
    } finally { 
     mDecor = null; 
     mWindow.closeAllPanels(); 
     onStop(); 
     mShowing = false; 

     sendDismissMessage(); 
    } 
} 

Answer 2

No debe tener que comprobar si está mostrándose como que lo desecha. Si no marca isShowing, simplemente ignorará el descarte() que la barra de progreso no muestra.

Pero la comprobación de isShowing es una buena práctica. Entonces no va a doler seguir comprobando.

Answer 3

Si sólo tiene un lugar en su código que, cuando dismiss() se llama, entonces no. Debería saber si el diálogo se muestra o no en ese momento.

Sin embargo, he tenido algunos problemas con ProgressDialogs y cambio de orientación, lo que ha provocado la necesidad de tener más de un lugar donde el diálogo podría ser descartado, lo que hace necesaria la verificación isShowing().

Answer 4

verificación de la source code, si se maneja de código que no tiene que preocuparse. Si no se realiza ningún manejo, es su elección crear un entorno más riesgoso pero rápido o lento pero seguro.