Android中使用Volley时Progress Dialog偶现无法消失问题排查

嘿，我来帮你排查这个Volley请求中Progress Dialog偶尔卡住的问题～ 结合你的代码来看，主要有几个可能的原因，我会逐一说明并给出修复方案：

一、核心问题分析 #

你的代码逻辑看起来没问题，但偶尔出现Dialog无法消失，大概率是以下几种情况：

• Volley请求未被及时取消，导致Activity销毁后回调仍在执行：如果用户在请求过程中切换页面、旋转屏幕或者把App切到后台，Activity可能被销毁，但Volley请求还在后台运行。当回调触发时，ProgressDialog的上下文已经失效，dismiss()无法正常执行，导致Dialog残留。
• ProgressDialog的强引用导致内存泄漏：如果你的progressDialog是Activity的成员变量，Volley的回调（持有Activity的引用）会导致Activity无法被GC回收，Dialog也一直停留在界面上。
• dismiss()调用时抛出异常，导致代码中断：比如Dialog已经被系统强制销毁，但你的代码还在尝试调用dismiss()，抛出的异常会让后续代码无法执行，Dialog也就没被关闭。

二、具体修复方案 #

1. 在Activity生命周期销毁时取消Volley请求 #

给你的请求设置一个唯一Tag，然后在Activity的onDestroy()方法中取消该请求，避免回调在Activity销毁后执行：

// 定义请求Tag（可以放在Activity类中）
private static final String GET_SERVER_COUNTS_TAG = "GetServerCountsRequest";

private void getServerCountsForLocation(JSONObject mDataPacket) {
    String url = REST_API_END_POINT + "GetServerCounts/";
    if (myApp.isNetworkAvailable()) {
        ProgressDialog dialog = ProgressDialog.show(this, "", "Getting total counts", false);
        progressDialog = dialog; // 假设progressDialog是成员变量
        
        JsonObjectRequest jsObjRequest = new JsonObjectRequest(Request.Method.POST, url, mDataPacket,
                response -> {
                    dismissProgressDialog();
                    parseResponseForServerCounts(response);
                }, error -> {
                    dismissProgressDialog();
                    renderView();
                });
        
        // 设置请求Tag
        jsObjRequest.setTag(GET_SERVER_COUNTS_TAG);
        jsObjRequest.setRetryPolicy(new DefaultRetryPolicy(3000, 2, DefaultRetryPolicy.DEFAULT_BACKOFF_MULT));
        VolleySingleton.getInstance(this.getApplicationContext()).getRequestQueue().add(jsObjRequest);
    } else {
        Log.print("Network connection not available in DeviceCountActivity for total counts query");
        Toast.makeText(DeviceCountActivity.this, "Network connection unavailable. Please try again after connecting with WiFi or SIM.", Toast.LENGTH_LONG).show();
    }
}

// 统一封装Dialog关闭方法
private void dismissProgressDialog() {
    if (progressDialog != null && progressDialog.isShowing()) {
        try {
            progressDialog.dismiss();
        } catch (Exception e) {
            // 捕获异常，避免代码中断
            e.printStackTrace();
        } finally {
            progressDialog = null; // 清空引用
        }
    }
}

// 在Activity的onDestroy中取消请求并关闭Dialog
@Override
protected void onDestroy() {
    super.onDestroy();
    // 取消对应Tag的Volley请求
    VolleySingleton.getInstance(this).getRequestQueue().cancelAll(GET_SERVER_COUNTS_TAG);
    // 确保Dialog被关闭
    dismissProgressDialog();
}

2. 使用弱引用持有ProgressDialog（可选，进一步避免内存泄漏） #

把progressDialog的强引用改成弱引用，这样即使Activity被销毁，GC也能回收Dialog对象：

// 将成员变量改为弱引用
private WeakReference<ProgressDialog> progressDialogRef;

// 显示Dialog时
ProgressDialog dialog = ProgressDialog.show(this, "", "Getting total counts", false);
progressDialogRef = new WeakReference<>(dialog);

// 统一关闭方法修改为：
private void dismissProgressDialog() {
    ProgressDialog dialog = progressDialogRef != null ? progressDialogRef.get() : null;
    if (dialog != null && dialog.isShowing()) {
        try {
            dialog.dismiss();
        } catch (Exception e) {
            e.printStackTrace();
        } finally {
            progressDialogRef.clear();
        }
    }
}

3. 优化无网络分支的Dialog处理 #

你的代码中，无网络分支里调用了progressDialog.dismiss()，但此时progressDialog还没被初始化（只有有网络时才会show），虽然不会报错，但可以优化一下，避免不必要的判断：

// 无网络分支修改为：
else {
    Log.print("Network connection not available in DeviceCountActivity for total counts query");
    Toast.makeText(DeviceCountActivity.this, "Network connection unavailable. Please try again after connecting with WiFi or SIM.", Toast.LENGTH_LONG).show();
    // 这里不需要调用dismiss，因为Dialog还没被创建
}

三、额外建议 #

• 尽量避免使用ProgressDialog，它已经被标记为过时了，推荐使用ProgressBar或者自定义Dialog，更灵活也更稳定。
• 可以考虑封装Volley的请求工具类，统一处理Dialog的显示和关闭，避免重复代码，也更容易维护。

内容的提问来源于stack exchange，提问作者Crazy Horse Hugo