线程仍尝试访问已释放窗体问题求助

首先，我得指出几个导致你问题的核心原因：

1. 异常捕获范围不对：你的try-catch块包裹的是下载线程的入口代码，但DownloadProgressChanged和DownloadFileCompleted的回调是由WebClient的内部线程池线程触发的，不在你的try块范围内，所以这些回调里抛出的ObjectDisposedException根本不会被捕获到。

2. 控件状态检查存在竞态条件：你在回调里判断!pbDownloadStatus.IsDisposed，但从判断到执行Invoke的瞬间，控件可能已经被释放了，这中间的时间差会导致异常。

3. 线程终止方式错误：使用Thread.Abort()是非常危险的操作，它会强制终止线程，可能导致资源泄漏、数据损坏，甚至死锁，而且并不能保证线程立即停止。

4. 不必要的手动Dispose：窗体在关闭时会自动调用Dispose()，你手动调用反而可能导致控件提前释放，引发问题。

修复方案 #

方案1：改用async/await（推荐，最简洁安全） #

WinForms里处理异步操作，async/await是最佳实践，它会自动处理线程上下文切换，而且不需要手动管理线程，避免了跨线程访问控件的很多问题。

修改后的代码示例：

private CancellationTokenSource _cts;

private async void btnStartDownload_Click(object sender, EventArgs e)
{
    _cts = new CancellationTokenSource();
    string[] URLs = { "http://test1.com/", "http://test2.com/", "http://test3.com/" };
    
    try
    {
        using (WebClient wc = new WebClient())
        {
            wc.DownloadProgressChanged += (s, e) =>
            {
                if (!_cts.IsCancellationRequested)
                {
                    pbDownloadStatus.Value = e.ProgressPercentage;
                    lbPercentage.Text = $"{e.ProgressPercentage}%";
                }
            };
            
            wc.Proxy = WebProxy.GetDefaultProxy();
            wc.Headers.Add(header);
            
            foreach (var url in URLs)
            {
                if (_cts.IsCancellationRequested) break;
                await wc.DownloadFileTaskAsync(new Uri(url), $"C:\\Test\\URL{Array.IndexOf(URLs, url)}");
            }
        }
    }
    catch (OperationCanceledException)
    {
        MessageBox.Show("下载已取消");
    }
    catch (Exception ex)
    {
        MessageBox.Show($"下载出错：{ex.Message}");
    }
    finally
    {
        pbDownloadStatus.Value = 0;
        lbPercentage.Text = "0%";
    }
}

private void frmDownload_FormClosing(object sender, FormClosingEventArgs e)
{
    _cts?.Cancel();
}

方案2：改进原有线程的实现（如果坚持用Thread） #

如果必须使用Thread，需要解决竞态条件和异常捕获的问题：

1. 用CancellationToken代替abortThread标志，更可靠。
2. 在回调里使用更安全的控件访问方式，先检查窗体的IsHandleCreated，并且用Invoke前再次确认。
3. 取消WebClient的异步操作，而不是Abort线程。

修改后的代码示例：

private Thread _downloader;
private CancellationTokenSource _cts;

private void frmDownload_FormClosing(object sender, FormClosingEventArgs e)
{
    // 取消下载
    _cts?.Cancel();
    // 等待线程安全退出
    if (_downloader != null && _downloader.IsAlive)
    {
        _downloader.Join(2000); // 等待2秒，超时则放弃
    }
}

private void StartDownload()
{
    _cts = new CancellationTokenSource();
    _downloader = new Thread(() =>
    {
        string[] URLs = { "http://test1.com/", "http://test2.com/", "http://test3.com/" };
        try
        {
            using (WebClient wc = new WebClient())
            {
                wc.DownloadProgressChanged += (s, e) =>
                {
                    // 检查是否已取消
                    if (_cts.IsCancellationRequested) return;
                    
                    // 安全访问控件：先判断窗体是否存在，再Invoke
                    if (this.IsHandleCreated)
                    {
                        this.Invoke((MethodInvoker)(() =>
                        {
                            // 在UI线程再次检查控件状态
                            if (!pbDownloadStatus.IsDisposed && !lbPercentage.IsDisposed)
                            {
                                pbDownloadStatus.Value = e.ProgressPercentage;
                                lbPercentage.Text = $"{e.ProgressPercentage}%";
                            }
                        }));
                    }
                };

                wc.DownloadFileCompleted += (s, e) =>
                {
                    if (_cts.IsCancellationRequested) return;
                    
                    if (this.IsHandleCreated)
                    {
                        this.Invoke((MethodInvoker)(() =>
                        {
                            if (!pbDownloadStatus.IsDisposed && !lbPercentage.IsDisposed)
                            {
                                pbDownloadStatus.Value = 0;
                                lbPercentage.Text = "0%";
                            }
                        }));
                    }
                };

                wc.Proxy = WebProxy.GetDefaultProxy();
                wc.Headers.Add(header);

                foreach (var url in URLs)
                {
                    if (_cts.IsCancellationRequested) break;
                    var sync = new object();
                    lock (sync)
                    {
                        // 修正拼写错误：DownloadFileAsync
                        wc.DownloadFileAsync(new Uri(url), $"C:\\Test\\URL{Array.IndexOf(URLs, url)}", sync);
                        Monitor.Wait(sync);
                    }
                }
            }
        }
        catch (Exception ex)
        {
            if (!_cts.IsCancellationRequested)
            {
                this.Invoke((MethodInvoker)(() => MessageBox.Show($"出错：{ex.Message}")));
            }
        }
    });
    _downloader.Start();
}

额外注意点 #

• 永远不要用Thread.Abort()来终止线程，这是一种非常粗暴且不安全的方式，应该用协作式的取消机制（比如CancellationToken）。
• 跨线程访问控件时，Invoke是同步调用，如果控件已经被释放，还是可能抛出异常，所以在UI线程内部再次检查控件状态是必要的。
• WebClient的DownloadFileAsync需要正确传递用户状态，你原来的代码里lock (e.UserState)要确保e.UserState是你传递的sync对象，避免死锁。

内容的提问来源于stack exchange，提问作者Some