间接财产通知

Question

这个问题可能看起来有点微不足道，但事实可能并非如此。我只是想知道以下两种情况中哪一种更适合间接属性通知，或者也许有更好的方法。

场景：

我有两个属性，第一个是一个名为 HoldingObject 的对象，第二个是一个名为 IsHoldingObject 的布尔值，如果 HoldingObject == null 则为 false，否则为 true。我只是想知道 IsHoldingObject 的最佳通知机制是什么：

案例 (A) - 通知 IsHoldingObject 从 HoldingObject 属性更改：

public class NotifyingClass1 : INotifyPropertyChanged
{
    private object _holdingObject;
    public object HoldingObject
    {
        get { return _holdingObject; }
        set
        {
            if (_holdingObject != value)
            {
                _holdingObject = value;
                NotifyPropertyChanged("HoldingObject");
                // Notify from the property that is being checked
                NotifyPropertyChanged("IsHoldingObject");
            }
        }
    }

    public bool IsHoldingObject { get { return this.HoldingObject == null; } }

    #region INotifyPropertyChanged Members

    public event PropertyChangedEventHandler PropertyChanged;

    private void NotifyPropertyChanged(string propertyName)
    {
        if (this.PropertyChanged != null)
            this.PropertyChanged(this, new PropertyChangedEventArgs(propertyName));
    }

    #endregion
}

案例 (B) - 通过从 HoldingObject 属性将其设置为 false 或 true，直接通知 IsHoldingObject 从 IsHoldingObject 更改：

public class NotifyingClass2 : INotifyPropertyChanged
{
    private object _holdingObject;
    public object HoldingObject
    {
        get { return _holdingObject; }
        set
        {
            if (_holdingObject != value)
            {
                _holdingObject = value;
                NotifyPropertyChanged("HoldingObject");

                // 1) Set the property here
                this.IsHoldingObject = _holdingObject != null;
            }
        }
    }

    private bool _isHoldingObject;
    public bool IsHoldingObject
    {
        get { return _isHoldingObject; }
        set
        {
            if (_isHoldingObject != value)
            {
                _isHoldingObject = value;

                // 2) Notify directly from the property
                NotifyPropertyChanged("IsHoldingObject");
            }
        }
    }

    #region INotifyPropertyChanged Members

    public event PropertyChangedEventHandler PropertyChanged;

    private void NotifyPropertyChanged(string propertyName)
    {
        if (this.PropertyChanged != null)
            this.PropertyChanged(this, new PropertyChangedEventArgs(propertyName));
    }

    #endregion
}

我个人倾向于第一个，因为它需要更少的代码，但我不确定这样做是否值得推荐。让我知道是否还有其他（更好）的方法。

谢谢！

Answer 1

我几乎总是选择第二个。这样，负责设置给定属性的所有代码都位于一个位置。在这种特殊情况下，IsHoldingObject 属性完全依赖于 HoldingObject 属性，因此它可能不适用。但总的来说，现在代码可能会减少，但是如果您稍后需要从其他地方更新该属性，会发生什么情况？

您所遇到的痛苦（代码的冗长）可以通过用于在视图模型中设置属性的辅助方法轻松减轻（这可以存在于基类中）：

protected bool SetValue<T>(ref T field, T value, string propertyName)
{
    if (!Equals(field, value))
    {
        field = value;
        NotifyPropertyChanged(propertyName);
        return true;
    }

    return false;
}

然后您的属性将变为：

private object _holdingObject;
public object HoldingObject
{
    get { return _holdingObject; }
    set
    {
        if (SetValue(ref _holdingObject, value, "HoldingObject"))            
            this.IsHoldingObject = _holdingObject != null;
    }
}

private bool _isHoldingObject;
public bool IsHoldingObject
{
    get { return _isHoldingObject; }
    private set { SetValue(ref _isHoldingObject, value, "IsHoldingObject"); }
}

Answer 2

在大多数情况下这并不重要。让我们看看

• 每次计算的选项。因此，每次 UI 请求该值时，您都会消耗 CPU。但考虑到每次 UI 需要每秒 100 秒请求该值，您才会注意到命中，需要多少 CPU。我怀疑你的用户界面会需要这个。
• 存储该值。这里你正在消耗内存。同样，除非您存储许多大型物体，否则这不是问题。

所以我想这取决于什么对你来说更具可读性。我更喜欢“每次都计算”。然后使用这个 http://code.google.com/p/notifypropertyweaver/ 它允许我编写此代码

public class NotifyingClass1 : INotifyPropertyChanged
{
    public object HoldingObject { get; set; }

    public bool IsHoldingObject { get { return HoldingObject == null; } }

    public event PropertyChangedEventHandler PropertyChanged;
}

并将其编译到程序集中（即导出依赖项）

public class NotifyingClass1 : INotifyPropertyChanged
{
    private object holdingObject;

    public event PropertyChangedEventHandler PropertyChanged;

    public virtual void OnPropertyChanged(string propertyName)
    {
        var propertyChanged = PropertyChanged;
        if (propertyChanged != null)
        {
            propertyChanged(this, new PropertyChangedEventArgs(propertyName));
        }
    }

    public object HoldingObject
    {
        get { return holdingObject; }
        set
        {
            if (holdingObject !=value)
            {
                holdingObject = value;
                OnPropertyChanged("HoldingObject");
                OnPropertyChanged("IsHoldingObject");
            }
        }
    }

    public bool IsHoldingObject { get { return (HoldingObject == null); } }
}

Answer 3

有趣的想法。我想，我会使用 BindingConverter 方法，它将 HoldingObject 转换为您喜欢的值。

我很好奇其他意见。

问候