为何非静态构造函数可给静态只读SmtpClient绑定SendCompletedEventHandler？

先把你的代码贴出来方便讨论：

private static readonly SmtpClient SmtpClient = new SmtpClient (...)
public Constructor() {
    SmtpClient.SendCompleted += new SendCompletedEventHandler(SendCompletedMethod);
    // 上述代码合法，但字段标记为static readonly，原因何在？
}

为什么这段代码是合法的？ #

你可能误解了static readonly的限制范围——它约束的是字段本身的引用不可变更，而不是字段指向的对象的内部状态不能修改。

SmtpClient是引用类型，static readonly只是保证你没法给SmtpClient这个字段重新赋值一个新的SmtpClient实例（比如写SmtpClient = new SmtpClient()会直接编译报错）。但绑定事件是在修改这个实例的内部事件列表，属于对象内部状态的变更，完全没有触碰字段的引用，所以编译器完全允许这种操作。

打个比方：static readonly字段就像一把焊死在墙上的挂钩，你不能把挂钩换成别的，但你可以往挂钩上挂东西、取东西——绑定事件就属于“挂东西”的操作。

构造函数执行两次会引发什么错误？ #

因为SmtpClient是静态成员，整个应用域里只会存在这一个实例。每次你的实例构造函数执行，都会给这个实例的SendCompleted事件多绑定一次SendCompletedMethod方法。

如果构造函数执行两次，那么当SendCompleted事件触发时，SendCompletedMethod会被连续调用两次；如果执行N次，就会被调用N次。这通常会导致重复执行业务逻辑、重复处理数据，甚至引发数据一致性问题，这就是你说的“引发错误”的根源。

怎么解决这个问题？ #

给你几个实用的解决方案：

• 最推荐：把事件绑定移到静态构造函数：静态构造函数只会在类第一次被访问时执行一次，从根源上避免重复绑定：

  private static readonly SmtpClient SmtpClient = new SmtpClient(...);
  
  // 静态构造函数，仅执行一次
  static YourClassName() {
      SmtpClient.SendCompleted += SendCompletedMethod;
  }
  

• 临时方案：先解绑再绑定：如果确实需要在实例构造函数里处理，可以先移除旧的绑定再添加新的（注意：如果有其他地方也绑定了这个事件，这种方式可能会误删其他绑定，谨慎使用）：

  public Constructor() {
      SmtpClient.SendCompleted -= SendCompletedMethod;
      SmtpClient.SendCompleted += SendCompletedMethod;
  }
  

• 重构方案：评估是否需要静态SmtpClient：如果你的类的每个实例都需要独立的邮件客户端，可以去掉static修饰符，让每个实例拥有自己的SmtpClient——不过SmtpClient通常复用性更好，静态实例的场景更常见。

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