WPF中调用CheckUserExist时Login始终为null的问题排查

嘿，我一眼就瞅出问题在哪了——你的UserService里的Login属性根本没拿到ViewModel里用户输入的值啊！

核心问题拆解 #

你看：

• 你在MainWindow里分别实例化了UserViewModel和UserService，这俩完全是独立的对象，没有任何数据关联
• View和ViewModel的Login绑定是正常的，但UserService自己的Login从来没被赋值过，所以调用CheckUserExist()时肯定是null
• 你之前试的UpdateSourceTrigger和RaisePropertyChanged都是在View和ViewModel之间的绑定上做文章，根本没碰Service这边的问题

几种可行的解决方案 #

方案1：调用方法时直接传值（最推荐） #

这是最简单直接的方式，不用改太多代码：
先修改按钮点击事件，把ViewModel里的Login传给Service的方法：

private void Button_Click(object sender, RoutedEventArgs e)
{
    // 把ViewModel里的实时Login值传进去
    userService.CheckUserExist(userViewModel.Login);
}

然后更新UserService的CheckUserExist方法，用参数代替自己的Login属性：

public void CheckUserExist(string login)
{
    using (PrincipalContext principalContext = new PrincipalContext(ContextType.Domain))
    {
        var user = UserPrincipal.FindByIdentity(principalContext, login);
        if (user == null)
        {
            UserMessageText = "用户不存在";
        }
        // 你原来代码里的空代码块记得删掉哦
    }
}

这样一来，Service不需要维护自己的Login属性，直接用调用方传递的参数，干净又符合单一职责。

方案2：让Service依赖ViewModel实例 #

如果你的Service需要多次用到ViewModel里的属性，可以把ViewModel注入到Service中：
先修改UserService的构造函数，接收ViewModel实例：

public class UserService
{
    private readonly UserViewModel _userViewModel;

    // 通过构造函数注入ViewModel
    public UserService(UserViewModel userViewModel)
    {
        _userViewModel = userViewModel;
    }

    public void CheckUserExist()
    {
        using (PrincipalContext principalContext = new PrincipalContext(ContextType.Domain))
        {
            // 直接用ViewModel里的Login值
            var user = UserPrincipal.FindByIdentity(principalContext, _userViewModel.Login);
            if (user == null)
            {
                UserMessageText = "用户不存在";
            }
        }
    }
}

然后在MainWindow里实例化Service时传入ViewModel：

UserViewModel userViewModel = new UserViewModel();
// 把ViewModel传给Service
UserService userService = new UserService(userViewModel);

这样Service就能实时获取ViewModel里的Login值了。

方案3：给Service加属性同步（不推荐） #

这种方式有点冗余，除非有特殊需求否则不建议用。思路是监听ViewModel的属性变化，同步到Service的Login上：
先让UserService实现INotifyPropertyChanged（其实没必要，但为了同步）：

public class UserService : INotifyPropertyChanged
{
    private string _login;
    public string Login
    {
        get { return _login; }
        set
        {
            _login = value;
            OnPropertyChanged(nameof(Login));
        }
    }

    public event PropertyChangedEventHandler PropertyChanged;
    protected void OnPropertyChanged(string propertyName)
    {
        PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
    }

    // 其余业务代码不变
}

然后在MainWindow的构造函数里添加同步逻辑：

public MainWindow()
{
    InitializeComponent();
    DataContext = userViewModel;

    // 监听ViewModel的Login变化，同步到Service
    userViewModel.PropertyChanged += (sender, e) =>
    {
        if (e.PropertyName == nameof(UserViewModel.Login))
        {
            userService.Login = userViewModel.Login;
        }
    };
}

但这种方式多了一层同步逻辑，不如前两种简洁高效。

总结 #

最推荐方案1，逻辑清晰，代码改动最少，还符合软件设计的单一职责原则——Service只负责处理业务逻辑，数据由调用方传递，避免了不必要的依赖。

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