无法使用 MVVM-Light 挂钩 PropertyChanged 事件

Question

您好，创建了我的第一个基于 MVVM 的 WPF 应用程序，并试图弄清楚为什么我无法挂钩到依赖属性的 PropertyChanged 事件。

父视图模型中的代码：

void createClients()
{
    var clients = from client in Repository.GetClients()
                  select new ClientViewModel(Repository, client);
    foreach (var client in clients)
    {
        client.PropertyChanged += onClientPropertyChanged;
    }
    Clients = new ViewableCollection<ClientViewModel>(clients);
    Clients.CollectionChanged += onClientsCollectionChanged;
}

// Never gets called
void onClientPropertyChanged(object sender, PropertyChangedEventArgs e)
{
    if (e.PropertyName == "Name")
    {
         //...
    }
}

ViewableCollection 是 ObservableCollection 的简单扩展，用于封装 View。

在ClientViewModel 中，setter 被调用，但 RaisePropertyChanged 没有像我预期的那样工作，因为onClientPropertyChanged 没有被调用。两个视图模型都继承自 ViewModelBase。

public string Name
{
    get { return client.Name; }
    set
    {
        if (value == client.Name) return;
        client.Name = value;
        RaisePropertyChanged("Name");
    }
}

如果我将PropertyChanged 连接到ClientViewModel 内的一个方法，那么它就会被触发，所以我很难理解为什么这在父视图模型中不起作用。我哪里错了？

Answer 1

This SO question 解释问题； ObservableCollection 保护 PropertyChanged 事件。

一种解决方案是使用 MVVM-Light Messenger：

void createClients()
{
    var clients = from client in Repository.GetClients()
                  select new ClientViewModel(Repository, client);
    Clients = new ViewableCollection<ClientViewModel>(clients);
    Clients.CollectionChanged += onClientsCollectionChanged;
    Messenger.Default.Register<PropertyChangedMessage<string>>(this, (pcm) =>
    {
        var clientVM = pcm.Sender as ClientViewModel;
        if (clientVM != null && pcm.PropertyName == "Name")
        {
            // ...
        }
    });
}

createClients() 应该重构，但为了与问题代码保持一致，我将其保留在那里。然后对属性设置器稍作改动：

public string Name
{
    get { return client.Name; }
    set
    {
        if (value == client.Name) return;
        string oldValue = client.Name;
        client.Name = value;
        RaisePropertyChanged<string>("Name", oldValue, value, true);
    }
}