MVVM架构中ViewModel与Model通信方式及现有实现合理性咨询

我希望修改Model.Street及Model的其他属性，但尝试的所有方法要么代码冗长、不合理，要么过于复杂。我并非想偷懒，只是习惯了更简洁的实现方式。

尝试的实现方式 #

1. 直接包装Model属性 #

这种方式感觉只是重复编写了Model的内容：

public class CustomerInfoViewModel : BaseViewModel
{
    public CustomerInfo Model { get; set; }
    public CustomerInfoViewModel()
    {
        Model = new CustomerInfo();
    }
    public CustomerInfoViewModel(CustomerInfo customerInfo)
    {
        Model = customerInfo;
    }

    public string Street
    {
        get
        {
            return Model.Street;
        }
        set
        {
            Model.Street = value;
            OnPropertyChanged();
        }
    }
}

2. 使用字段+通用Setter方法 #

这种方式需要额外添加字段，感觉不合理：

public class CustomerInfoViewModel : BaseViewModel
{
    public CustomerInfo Model { get; set; }

    public CustomerInfoViewModel()
    {
        Model = new CustomerInfo();
    }

    public CustomerInfoViewModel(CustomerInfo customerInfo)
    {
        Model = customerInfo;
    }

    // 通用字段设置方法
    private void SetField<T>(ref T field, T value, [CallerMemberName] string propertyName = null)
    {
        if (!EqualityComparer<T>.Default.Equals(field, value))
        {
            field = value;
            OnPropertyChanged(propertyName);
        }
    }

    private string _street;

    public string Street
    {
        get => _street;
        set => SetField(ref _street, value);
    }

    // 其他属性也需要遵循相同模式添加字段和属性
}

我的疑问 #

1. 我是否误解了MVVM的理念？
2. 上述两种实现是否可行？
3. 有没有更简洁的优解决方案？

解答 #

1. 你没有误解MVVM的核心理念 #

MVVM的核心是分离视图逻辑和业务数据，ViewModel作为中间层，负责暴露视图需要的数据和命令，同时处理视图的交互逻辑。你尝试的两种方式都符合这个核心，只是在实现简洁性上有优化空间。

2. 两种实现的可行性分析 #

• 第一种直接包装Model属性：完全可行，是MVVM中常见的实现方式。优点是不需要额外维护字段，直接复用Model的状态；缺点是当Model属性较多时，会产生大量重复的包装代码，显得冗长。
• 第二种字段+通用Setter：本身可行，但你当前的实现存在数据不一致问题——没有将ViewModel字段的变更同步回Model。如果要使用这种方式，必须在Setter中同步更新Model的对应属性，修正后的代码如下：

  public string Street
  {
      get => _street;
      set
      {
          SetField(ref _street, value);
          Model.Street = value; // 同步到Model
      }
  }
  

  这种方式的优点是简化了属性变更通知逻辑，但缺点是需要维护ViewModel字段与Model的同步，增加了额外的维护成本，仅在ViewModel需要对数据做额外处理（比如格式转换、验证）时更有意义。

3. 更优的简洁解决方案 #

针对"简洁包装Model属性"的需求，有几种高效的优化方式：

方案一：在BaseViewModel中实现通用Model属性设置方法

通过表达式树和CallerMemberName，封装通用的Model属性更新逻辑，避免重复编写PropertyChanged代码：

public class BaseViewModel : INotifyPropertyChanged
{
    public event PropertyChangedEventHandler PropertyChanged;

    protected void OnPropertyChanged([CallerMemberName] string propertyName = null)
    {
        PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
    }

    // 通用Model属性设置方法
    protected void SetModelProperty<T>(T value, Expression<Func<T>> propertyExpression)
    {
        var memberExpr = propertyExpression.Body as MemberExpression;
        if (memberExpr == null) throw new ArgumentException("无效的属性表达式");

        var propertyInfo = memberExpr.Member as PropertyInfo;
        if (propertyInfo == null) throw new ArgumentException("表达式指向非属性成员");

        var targetObj = ((ConstantExpression)memberExpr.Expression).Value;
        var currentValue = (T)propertyInfo.GetValue(targetObj);

        if (!EqualityComparer<T>.Default.Equals(currentValue, value))
        {
            propertyInfo.SetValue(targetObj, value);
            OnPropertyChanged(memberExpr.Member.Name);
        }
    }
}

在ViewModel中使用时只需一行代码：

public class CustomerInfoViewModel : BaseViewModel
{
    public CustomerInfo Model { get; set; }

    public CustomerInfoViewModel() => Model = new CustomerInfo();
    public CustomerInfoViewModel(CustomerInfo customerInfo) => Model = customerInfo;

    public string Street
    {
        get => Model.Street;
        set => SetModelProperty(value, () => Model.Street);
    }
}

方案二：自动生成包装代码

如果项目中有大量Model需要包装，可以使用T4模板或Roslyn源代码生成器自动生成ViewModel的包装属性，完全避免手动编写重复代码。这类工具可以扫描Model类的属性，自动生成包含变更通知和Model同步逻辑的ViewModel属性。

方案三：让Model实现INotifyPropertyChanged

如果业务场景允许，可直接让Model实现INotifyPropertyChanged接口，ViewModel直接暴露Model给视图，无需额外包装属性。但这种方式会让Model包含UI相关的变更通知逻辑，可能违反单一职责原则，适合简单场景或小型项目。

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