WPF中如何通过复选框正确启停Timer，避免异常与重复执行

你的问题核心出在Timer的作用域管理不当和重复订阅事件导致多次触发这两个点上，我来一步步帮你修正：

问题根源拆解 #

1. 无法访问已释放对象：你把rsTimer声明为方法内的局部变量，取消勾选时Dispose了这个实例，但下次勾选时如果不小心复用了已释放的引用（或者变量作用域混乱），就会抛出异常。而且局部变量会在方法执行完后被回收，根本没法持续控制同一个Timer实例。
2. 重复添加内容：如果不Dispose，每次勾选都会创建新的Timer，旧的Timer没被停止还在后台运行；再加上每次启动都重新订阅Elapsed事件，导致同一个Timer触发一次就会执行多次UpdateList，自然会重复添加内容。

修正后的完整代码 #

首先把rsTimer改成类级别的私有字段，确保整个类生命周期内只有一个可控的Timer实例：

// 在类的顶部声明Timer字段，不要放在方法内部
private Timer _rsTimer;

private void checkconnect_CheckedChanged(object sender, EventArgs e)
{
    if (checkconnect.Checked)
    {
        // 仅在Timer未初始化时创建实例并订阅事件
        if (_rsTimer == null)
        {
            _rsTimer = new Timer();
            _rsTimer.Interval = 1000;
            // 事件只订阅一次，避免重复触发
            _rsTimer.Elapsed += OnTimedEvent;
        }
        
        // 更新UI状态
        UpdateStatusLabels("Checking");
        _rsTimer.Start();
    }
    else
    {
        if (_rsTimer != null)
        {
            _rsTimer.Stop();
            // 不要立刻Dispose，留着下次复用；如果确定不再使用，可在窗口关闭时统一处理
            // _rsTimer.Dispose();
            // _rsTimer = null;
        }
        
        // 更新UI状态
        UpdateStatusLabels("Idle");
    }
}

// 把事件处理方法抽出来，作为类的成员方法
private void OnTimedEvent(object source, ElapsedEventArgs elapsed)
{
    try
    {
        // WPF中UI操作必须在主线程执行，这里用Dispatcher切回主线程
        Application.Current.Dispatcher.Invoke(() =>
        {
            UpdateList("test");
        });
    }
    catch (Exception ex)
    {
        Console.WriteLine(ex);
        // 这里不要throw，否则会导致Timer停止触发后续事件
    }
}

// 抽离重复的UI更新逻辑，避免冗余代码
private void UpdateStatusLabels(string status)
{
    Shorestatuslbl.Text = status;
    buoystatuslbl.Text = status;
    nistatuslbl.Text = status;
}

关键优化点说明 #

• 类级别的Timer实例：确保每次启停都是同一个Timer，避免重复创建和已释放对象的问题。
• 单次事件订阅：只在Timer初始化时订阅一次Elapsed事件，彻底解决重复触发UpdateList的问题。
• WPF线程安全：Timer的Elapsed事件在后台线程触发，必须用Dispatcher.Invoke将UI操作切回主线程，否则会抛出跨线程操作异常。
• 合理的Dispose时机：不要在取消勾选时立刻Dispose，建议在窗口的Closed事件里统一清理资源：

  private void Window_Closed(object sender, EventArgs e)
  {
      _rsTimer?.Dispose();
  }
  

这样修改后，重新勾选复选框不会再出现已释放对象的异常，也不会重复向列表添加内容了。

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