ViewModel属性栈溢出/递归防护及宽高比保持功能实现难题

我之前也碰到过一模一样的问题！双向同步宽高比的时候，两个控件的变更事件互相触发，就算加了递归防护，代码还是写得重复又啰嗦，特别闹心。结合你的场景（已经用了OneWay绑定和递归防护），可以试试这几个优化方向：

1. 提取共享计算逻辑，消除代码重复 #

把两个事件里重复的比例计算逻辑抽成一个独立的私有方法，不管是改宽度还是高度，都调用这个方法来计算对应的值。这样两个事件处理函数里就只剩调用逻辑，不会再冗余。

比如可以写这样的方法：

private void UpdateProportionalDimension(bool isWidthChanged, double newValue)
{
    if (_isUpdating) return; // 你的递归防护标志
    
    try
    {
        _isUpdating = true;
        var aspectRatio = _originalWidth / _originalHeight; // 假设你已经保存了基准宽高比
        
        if (isWidthChanged)
        {
            // 根据新宽度计算对应高度
            Height = newValue / aspectRatio;
        }
        else
        {
            // 根据新高度计算对应宽度
            Width = newValue * aspectRatio;
        }
    }
    finally
    {
        _isUpdating = false; // 确保标志位一定会重置
    }
}

然后你的两个变更事件就可以简化成：

private void WidthControl_ValueChanged(object sender, EventArgs e)
{
    UpdateProportionalDimension(isWidthChanged: true, newValue: WidthControl.Value);
}

private void HeightControl_ValueChanged(object sender, EventArgs e)
{
    UpdateProportionalDimension(isWidthChanged: false, newValue: HeightControl.Value);
}

2. 把逻辑移到ViewModel属性层面（更贴合MVVM） #

既然你用了ViewModel和OneWay绑定，其实可以把同步逻辑直接写到Width和Height属性的setter里，这样就不用在后台绑定控件的变更事件了，更符合MVVM的设计思路。

示例代码：

private double _width;
public double Width
{
    get => _width;
    set
    {
        if (_isUpdating || Math.Abs(value - _width) < 1e-6) return; // 防递归+值无变化时跳过
        
        _isUpdating = true;
        _width = value;
        Height = _width / _aspectRatio; // 同步计算高度
        OnPropertyChanged(nameof(Width));
        _isUpdating = false;
    }
}

private double _height;
public double Height
{
    get => _height;
    set
    {
        if (_isUpdating || Math.Abs(value - _height) < 1e-6) return;
        
        _isUpdating = true;
        _height = value;
        Width = _height * _aspectRatio; // 同步计算宽度
        OnPropertyChanged(nameof(Height));
        _isUpdating = false;
    }
}

这样UI绑定到这两个属性后，不管修改哪一个，另一个都会自动同步，而且代码完全没有冗余，递归防护也整合在属性setter里了。

3. 用单一事件处理函数（如果控件类型一致） #

如果你的宽度和高度控件是同一种类型（比如都是NumericUpDown），还可以把两个事件绑定到同一个处理函数里，通过sender来判断是哪个控件触发的，进一步减少代码量：

private void DimensionControl_ValueChanged(object sender, EventArgs e)
{
    var control = sender as NumericUpDown;
    if (control == null) return;
    
    bool isWidthChanged = control == WidthControl;
    UpdateProportionalDimension(isWidthChanged, control.Value);
}

这样只需要写一个事件处理函数就够了。

这些方法应该能帮你彻底解决代码冗余的问题，我当时就是用属性层面的逻辑优化搞定的，代码一下子清爽了好多！

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