12

我正在尝试创建一个简单的 Clamp（这样我就可以绑定任何可比较的值......主要用于 int、double 等数字类型）

问题是如果我执行以下操作会出现错误，但根据 MSDN IComparable 的 CompareTo 应该能够处理空值。
引用：“根据定义，任何对象比较大于空，两个空引用比较彼此相等。”

public static T Clamp<T>(this T value, T min, T max)
    where T : IComparable<T>
{
    if (value.CompareTo(max) > 0)
        return max;

    if (value.CompareTo(min) < 0)
        return min;

    return value;
}



private Int32? _zip;
public Int32? Zip
{
    get
    {
        return _zip;
    }
    set
    {
        _zip = value.Clamp<Int32?>(0, 99999);
    }
}
4

2 回答 2

15

正如@LBushkin Nullable< T > 或 T 所说的那样？不实现 IComparable 接口。给定的解决方案是好的，但是我更喜欢在一个专门的类中使用可空比较逻辑，遵循单一职责原则，并且也可以用于比较任何可空类型。

例如，您可以像这样创建一个通用的 Nullable 类型比较器类：

public class NullableComparer<T> : IComparer<Nullable<T>>
      where T : struct, IComparable<T>
{

     public int Compare(Nullable<T> x, Nullable<T> y)
     {
        //Compare nulls acording MSDN specification

        //Two nulls are equal
        if (!x.HasValue && !y.HasValue)
            return 0;

        //Any object is greater than null
        if (x.HasValue && !y.HasValue) 
            return 1;

        if (y.HasValue && !x.HasValue)
            return -1;

        //Otherwise compare the two values
        return x.Value.CompareTo(y.Value);
     }

}

在这种情况下，您将像这样使用此类：

public static T? Clamp<T>(this T? value, T? min, T? max)
    where T : struct
{
    var comparer = new NullableComparer<T>();

    if (comparer.Compare(value, max) > 0)
        return max;

    if (comparer.Compare(value, min) < 0)
        return min;

    return value;
}

方便保存在您的助手库中。

希望能帮助到你！

于 2010-07-20T22:09:24.800 回答
7

请记住，Int32?是 的简写Nullable<Int32>。由于Nullable<T>未实现IComparable<T>，因此您的结构化代码将无法编译。

但是，您可以重载该方法：

public static T? Clamp<T>(this T? value, T? min, T? max) 
    where T : struct, IComparable<T> 
{ 
    // your logic...
}

当然，如果您打算使用可空类型，则必须定义如何钳位null值...

如果您实际上不需要钳位null值，则首先在属性 getter 中检查 null 可能会更简单：

public Int32? Zip
{
   ...
   set
   {
       _zip = value == null ? value : value.Value.Clamp<Int32>(0,99999);
   }

或者更好的是，将其作为附加重载实现的一部分Clamp...

于 2010-07-20T21:41:02.627 回答