WinForm倒计时Label显示跳变求助：数值反复跳回初始值

在WinForms程序中，通过Radar类的构造函数传入Form中的Label控件实现倒计时显示。程序正常运行时倒计时正常，但点击按钮启动下载流程后，Label显示的倒计时出现跳变（如50→40→50→39→50→38）。

Form1中启动下载流程的代码：

private async void btnDownload_Click(object sender, EventArgs e)
{
    // Display folder selection dialog
    using (var folderDialog = new FolderBrowserDialog())
    {
        if (folderDialog.ShowDialog() == DialogResult.OK)
        {
            lblStatus.Text = "Waiting for download to start";
            txtFolder.Text = folderDialog.SelectedPath;
            radar = new Radar(lblNextTimeDownload, txtFolder.Text);
            radar.ProgressChanged += Radar_ProgressChanged;
            radar.DownloadCompleted += Radar_DownloadCompleted;

            // Subscribe to the DownloadStarted event
            radar.DownloadStarted += Radar_DownloadStarted;

            // Start asynchronous download
            await radar.DownloadAndProcessImagesAsync();
        }
    }
}

原因分析 #

1. 多实例冲突：每次点击下载按钮都会创建新的Radar实例，但旧实例并未被销毁，其内部的Timer仍在后台运行，两个实例同时修改同一个Label控件的文本，导致显示跳变。
2. UI线程阻塞：TimerTick事件中调用DownloadAndProcessImagesAsync().Wait()会阻塞UI线程，造成Timer Tick事件堆积，进一步加剧显示异常。
3. 倒计时重置冲突：新实例初始化时会将nextDownloadTime重置为当前时间+5分钟，而旧实例仍基于原有时间更新倒计时，两者的时间差导致Label文本反复跳变。

解决方法 #

1. 清理旧实例，避免多Timer并行 #

在创建新Radar实例前，停止并销毁旧实例的Timer，确保只有一个实例控制Label：

private async void btnDownload_Click(object sender, EventArgs e)
{
    using (var folderDialog = new FolderBrowserDialog())
    {
        if (folderDialog.ShowDialog() == DialogResult.OK)
        {
            lblStatus.Text = "Waiting for download to start";
            txtFolder.Text = folderDialog.SelectedPath;
            
            // 清理旧实例
            radar?.Dispose();
            
            radar = new Radar(lblNextTimeDownload, txtFolder.Text);
            radar.ProgressChanged += Radar_ProgressChanged;
            radar.DownloadCompleted += Radar_DownloadCompleted;
            radar.DownloadStarted += Radar_DownloadStarted;

            await radar.DownloadAndProcessImagesAsync();
        }
    }
}

给Radar类实现IDisposable接口，用于安全释放Timer：

public class Radar : IDisposable
{
    private bool disposed = false;
    // 已有成员...

    public void Dispose()
    {
        Dispose(true);
        GC.SuppressFinalize(this);
    }

    protected virtual void Dispose(bool disposing)
    {
        if (disposed) return;
        
        if (disposing)
        {
            timer?.Stop();
            timer?.Dispose();
        }
        
        disposed = true;
    }

    ~Radar()
    {
        Dispose(false);
    }
}

2. 避免UI线程阻塞 #

将TimerTick中的同步等待改为异步等待，防止UI线程卡死：

private async void TimerTick(object sender, EventArgs e)
{
    TimeSpan remainingTime = nextDownloadTime - DateTime.Now;

    lblNextTimeDownload.Invoke((MethodInvoker)delegate
    {
        lblNextTimeDownload.Text = "Next download in: " + remainingTime.ToString(@"mm\:ss");
    });

    if (remainingTime <= TimeSpan.Zero)
    {
        timer.Stop();
        PrepareLinks();
        // 使用await替代Wait()，避免阻塞UI线程
        await DownloadAndProcessImagesAsync();
        nextDownloadTime = DateTime.Now.AddMinutes(5);
        timer.Start();
    }
}

3. 单实例控制 #

如果业务不需要并行下载，可通过单例模式确保程序中始终只有一个Radar实例，从根源避免多实例冲突。

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