WPF窗口动态切换UserControl时Logout事件失效问题求助

我一眼就看出问题出在哪了——你绑定事件的控件实例和实际显示的控件实例根本不是同一个！

问题根源拆解 #

看你的MainWindow构造函数：

1. 你创建了一个checkControl实例，还贴心地绑定了LogOutClick事件，但这个实例从来没被放到contentControl里显示过。
2. 当登录成功调用ChangeContentControl("Check")时，你又新建了一个CheckControl()实例赋值给内容控件。这个新实例完全是“裸奔”的，没有绑定任何LogOutClick事件，所以点击Logout按钮时，LogOutClick自然是null，触发不了切换逻辑。

两种可行的解决方案 #

方案1：复用预先创建的控件实例（推荐）

把控件实例提升为MainWindow的类字段，这样构造函数里绑定好事件的实例，就是后续切换时显示的实例，事件不会丢失：

// 把控件实例定义为MainWindow的私有字段
private LogInControl _logInControl;
private CheckControl _checkControl;

public MainWindow()
{
    InitializeComponent();
    
    // 初始化控件并绑定事件
    _logInControl = new LogInControl();
    _checkControl = new CheckControl();
    
    _logInControl.OnSuccessfulLogin += (sender, e) => ChangeContentControl("Check");
    _checkControl.LogOutClick += (sender, e) => ChangeContentControl("LogIn");
    
    // 默认显示登录控件
    contentControl.Content = _logInControl;
}

public void ChangeContentControl(string kindContentControl)
{
    switch (kindContentControl)
    {
        case "LogIn":
            contentControl.Content = _logInControl;
            break;
        case "Check":
            contentControl.Content = _checkControl;
            break;
    }
}

这种方式还能避免重复创建控件的性能开销，适合不需要重置控件状态的场景。

方案2：每次创建新控件时绑定事件（适合需要重置状态的场景）

如果每次切换都需要全新的控件（比如登录后要清空输入框，登出后要重置检查控件的状态），那就在创建新实例的同时绑定事件：

public MainWindow()
{
    InitializeComponent();
    
    // 初始化默认登录控件并绑定事件
    var logInControl = new LogInControl();
    logInControl.OnSuccessfulLogin += (sender, e) => ChangeContentControl("Check");
    contentControl.Content = logInControl;
}

public void ChangeContentControl(string kindContentControl)
{
    switch (kindContentControl)
    {
        case "LogIn":
            var logInControl = new LogInControl();
            logInControl.OnSuccessfulLogin += (sender, e) => ChangeContentControl("Check");
            contentControl.Content = logInControl;
            break;
        case "Check":
            var checkControl = new CheckControl();
            checkControl.LogOutClick += (sender, e) => ChangeContentControl("LogIn");
            contentControl.Content = checkControl;
            break;
    }
}

这样每次新创建的控件都有对应的事件绑定，不会出现事件为null的情况。

额外优化小建议 #

• 别用字符串当切换标识了，容易打错！定义一个枚举会更健壮：

  public enum ControlType
  {
      LogIn,
      Check
  }
  

  然后把ChangeContentControl的参数改成这个枚举，代码可读性和安全性都会提升。
• 长期来看，建议学习WPF的MVVM模式，用视图模型来管理视图切换，能彻底避免这类实例管理的问题，这也是WPF开发的最佳实践。

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