比较装箱值类型

Question

今天我偶然发现了我写的一个有趣的错误。我有一组可以通过通用设置器设置的属性。这些属性可以是值类型或引用类型。

public void SetValue( TEnum property, object value )
{
    if ( _properties[ property ] != value )
    {
        // Only come here when the new value is different.
    }
}

当为此方法编写单元测试时，我发现值类型的条件始终为真。我没花很长时间就发现这是由于 装箱/拆箱。我也没有花很长时间将代码调整为以下内容：

public void SetValue( TEnum property, object value )
{
    if ( !_properties[ property ].Equals( value ) )
    {
        // Only come here when the new value is different.
    }
}

问题是我对这个解决方案并不完全满意。我想保留一个简单的参考比较，除非该值被装箱。

我当前想到的解决方案只是为装箱值调用 Equals() 。进行检查盒装价值观似乎有点矫枉过正。难道就没有更简单的方法吗？

Answer 1

如果您在处理值类型时需要不同的行为，那么您显然需要执行某种测试。您不需要显式检查装箱值类型，因为由于参数类型为object，所以所有值类型都将被装箱**。

此代码应满足您规定的条件：如果 value 是（装箱的）值类型，则调用多态 Equals 方法，否则使用 ==测试参考相等性。

public void SetValue(TEnum property, object value)
{
    bool equal = ((value != null) && value.GetType().IsValueType)
                     ? value.Equals(_properties[property])
                     : (value == _properties[property]);

    if (!equal)
    {
        // Only come here when the new value is different.
    }
}

（ ** 而且，是的，我知道 Nullable 是一种值类型，具有与装箱和拆箱相关的特殊规则，但这在这里几乎无关紧要。）

Answer 2

Equals() 通常是首选方法。

.Equals() 的默认实现对引用类型进行简单的引用比较，因此在大多数情况下，这就是您将得到的结果。 Equals() 可能已被重写以提供某些其他行为，但如果有人在类中重写了 .Equals() ，那是因为他们想要更改该类型的相等语义，如果您不这样做，最好让这种情况发生有一个令人信服的理由不这样做。当你的类认为两件事不同，而其他类都同意它们相同时，使用 == 绕过它可能会导致混乱。

Answer 3

由于输入参数的类型是object，因此您将始终在方法的上下文中获得一个装箱值。

我认为你唯一的机会是更改方法的签名并编写不同的重载。

Answer 4

怎么样：

if(object.ReferenceEquals(first, second)) { return; }
if(first.Equals(second)) { return; }

// they must differ, right?

更新

我意识到这在某些情况下不能按预期工作：

• 对于值类型，ReferenceEquals返回 false，因此我们回退到Equals< /code>，其行为符合预期。
• 对于 ReferenceEquals 返回 true 的引用类型，我们认为它们与预期“相同”。
• 对于 ReferenceEquals 返回 false 和 Equals 返回 false 的引用类型，我们认为它们与预期的“不同”。
• 对于 ReferenceEquals 返回 false 和 Equals 返回 true 的引用类型，即使我们想要“不同”，我们也会认为它们“相同”，

所以教训是“不要自作聪明”

Answer 5

我想

我想保留一个简单的参考比较，除非该值被装箱。

有点相当于

如果该值被装箱，我将进行非“简单参考比较”。

这意味着您需要做的第一件事是检查该值是否已装箱。

如果存在一种方法来检查对象是否是装箱值类型，那么它应该至少与您提供链接的“overkill”方法一样复杂，除非这不是最简单的方法。尽管如此，应该有一种“最简单的方法”来确定对象是否是装箱值类型。这种“最简单的方法”不太可能比简单地使用对象 Equals() 方法更简单，但我已经为这个问题添加了书签，以防万一。

（不确定我是否符合逻辑）