C# 从事件处理程序访问Dispatchermer时，如何修复偶发的空引用异常？

如何使此调度计时器成为一行代码而不是此代码

       private DispatcherTimer timer1;

       public void InitTimer()
        {
            timer1 = new System.Windows.Threading.DispatcherTimer();
            timer1.Tick += new EventHandler(timer1_Tick);
            timer1.Interval = TimeSpan.FromMilliseconds(2000);
            timer1.Start();
        }

x问题？当我取消选中它以禁用它时，它会得到一个空错误

       private DispatchTimer timer1; //<- Main part of the problem?

        public void InitTimer()
        {
            timer1 = new System.Windows.Threading.DispatcherTimer();
            timer1.Tick += new EventHandler(timer1_Tick);
            timer1.Interval = TimeSpan.FromMilliseconds(2000); 
            timer1.IsEnabled = true;
        }

       private void timer1_Tick(object sender, EventArgs e)
        {

            if (AS.Vis.Visibility != Visibility.Visible == false)
            {
                AS.Vis.Visibility = Visibility.Hidden;
                timer1.IsEnabled = false;
            }
            else
            {
                AS.Vis.Visibility = Visibility.Visible;

                Comp();


            }
        }

        private void AKS_Checked(object sender, RoutedEventArgs e)
        {
            InitTimer();
        AS.Vis.Visibility = Visibility.Visible;
            timer1.IsEnabled = true;// <-Also part of the problem

        }

        private void AKS_Unchecked(object sender, RoutedEventArgs e)
        {
            AS.Vis.Visibility = Visibility.Hidden;

            timer1.IsEnabled = false; //<-The problem
        }

---

专用调度计时器计时器1// 如果这是真正的问题：

我只是在尝试禁用计时器时得到一个空引用

然后尝试此操作，以便更早地初始化计时器变量

   private DispatchTimer timer1 = new System.Windows.Threading.DispatcherTimer();

   public void InitTimer()
    {
        timer1.Tick += new EventHandler(timer1_Tick);
        timer1.Interval = TimeSpan.FromMilliseconds(2000);
        timer1.Start();
    }

这将在创建类时，在调用InitTimer（或禁用函数）之前初始化变量。
创建一个新的
Dispatchermer
实例，而不停止最近的调用，也不取消订阅
Dispatchermer.Tick
事件，使所有计时器永远运行，并永远执行回调。这就是你目前正在做的事情。每次调用
AKS_Checked
都会创建一个新的
dispatchermer
，而不会停止正在运行的dispatchermer，也不会清理事件处理程序

以下示例是代码的重构版本，并保证只有一个计时器实例。

它还通过正确初始化私有计时器属性来修复空引用问题

dispatchermer.Start
重置计时器并启动它

partial class MainWindow : Window
{
  private DispatcherTimer Timer { get; set; }

  public void MainWindow()
  {
    InitializeComponent();
    InitTimer();
  }

  private void InitTimer()
  {
    this.Timer = new System.Windows.Threading.DispatcherTimer();
    this.Timer.Tick += OnTick;
    this.Timer.Interval = TimeSpan.FromMilliseconds(2000); 
  }

  private void OnTick(object sender, EventArgs e)
  {
    if (AS.Vis.Visibility == Visibility.Visible)
    {
      AS.Vis.Visibility = Visibility.Hidden;
      this.Timer.Stop();
    }
    else
    {
      AS.Vis.Visibility = Visibility.Visible;
      Comp();
    }
  }

  private void AKS_Checked(object sender, RoutedEventArgs e)
  {
    AS.Vis.Visibility = Visibility.Visible;

    // Reset and start the timer
    this.Timer.Start();
  }

  private void AKS_Unchecked(object sender, RoutedEventArgs e)
  {
    AS.Vis.Visibility = Visibility.Hidden;
    this.Timer.Stop();
  }
}

使用（同时设置
IsEnabled=true
而不是调用
Start（）
）。但对于事件处理程序，您是。为什么要这样做？它的可读性非常好。代码越少=/=性能越好。少不是最好的。把它缩短一点，你什么也得不到。如果代码让你恼火，只需折叠代码区域，使其显示在加号（+）下。啊-好的老XY问题。下次，提前共享该上下文。创建一个新计时器而不停止最近的事件，也不取消对Tick事件的订阅，将使所有计时器永远运行，并永远执行回调。这就是你目前正在做的事情。每次调用AKS_Checked都会创建一个新的计时器，而不会停止正在运行的计时器，也不会清理事件处理程序。你必须更加小心。我认为这个答案是不成熟的——直到我们看到不起作用的代码。我的意思是，你几乎可以肯定是对的，但如果是这样的话，就会有数千个适用的副本。请记住，这会导致多个事件处理程序附加到Tick事件，因为InitTimer在其当前实现中可能被多次调用。事件订阅和字段初始化应该移到构造函数中，以确保它只执行一次。@BionicCode-很好，我完全同意。如果您认为有帮助，请随意编辑答案。在InitTimer下，我收到一个错误，它说
this.Timer=new System.Windows.Threading.dispatchermer（）
在this.Timer部分是只读的。抱歉，我忘了添加属性设置程序。我已经修好了。现在一切都应该正常了。我还将
InitTimer
private设置为private。这可以正常工作，但您在“private DispatchTimer{get；set；}”中有一个输入错误，谢谢。我刚纠正了打字错误。