Tôi có nên kiểm tra xem ProgressBar có đang phát trước khi loại bỏ nó không?

Question

Im loại bỏ hộp thoại tiến trình khi AsyncTask kết thúc. Tôi có nên kiểm tra isShowing trước khi loại bỏ nó?

Tôi đã thử xóa kiểm tra này và hoạt động bình thường, nhưng có thể có các bẫy ẩn?

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

Answer 1

Có vẻ như đã được kiểm tra bên trong việc triển khai:

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

Bạn không cần phải kiểm tra làHiển thị loại bỏ nó. Nếu bạn không kiểm tra cho isShowing nó sẽ chỉ bỏ qua việc bỏ qua() là thanh tiến trình không hiển thị.

Nhưng việc kiểm tra làHiển thị là một thực hành tốt. Vì vậy, nó sẽ không bị tổn thương để tiếp tục kiểm tra nó.

Answer 3

Nếu bạn chỉ có một chỗ trong mã của mình, trong đó dismiss() được gọi, thì không. Bạn nên biết nếu hộp thoại đang hiển thị hay không tại thời điểm đó.

Tuy nhiên, tôi đã gặp phải một số vấn đề với ProgressDialog và thay đổi định hướng, khiến nhu cầu có nhiều hơn một nơi mà hộp thoại có khả năng bị loại bỏ, do đó cần thực hiện kiểm tra isShowing().

Answer 4

kiểm tra source code, nếu mã được xử lý bằng mã bạn không phải lo lắng. Nếu không có xử lý được thực hiện nó là sự lựa chọn của bạn để tạo ra môi trường nhiều rủi ro hơn nhưng nhanh hoặc chậm nhưng an toàn.