4

this.value1并且c.value1都可以为空或非空。所以一共4个组合来测试。value2也可以为空或非空。

下面的 if-then-else 是否可以用更短的东西代替,比如使用三元运算符( if then else 使用? :运算符) - 对于这种特定情况,这是否是一个不好的做法,因为我们正在测试value1and的 4 种组合value2

     public override bool Equals(object obj)
     {
        bool value1_check = false;
        bool value2_check = false;
        var c = obj as ObjectType;

        if (this.value1 != null)
               value1_check = this.value1.Equals(c.value1);
        else if ((this.value1 == null) && (c.value1 == null))
               value1_check = true;
        else if ((this.value1 == null) && (c.value1 != null))
              value1_check = c.value1.Equals(this.value1);

        if (this.value2 != null)
               value2_check = this.value2.Equals(c.value2);
        else if ((this.value2 == null) && (c.value2 == null))
               value2_check = true;
        else if ((this.value2 == null) && (c.value2 != null))
              value2_check = c.value2.Equals(this.value2);

        return (value1_check && value2_check);
     }
4

3 回答 3

8

您可以调用Object.Equals(),它已经完成了所有这些。

return Equals(this.Value1, c.Value1)
    && Equals(this.Value2, c.Value2);
于 2012-05-15T15:53:45.343 回答
4

实际上,您可能想要?? 运算符

var lhs= this.value1 ?? c.value1 ?? null;
var rhs = c.value1 ?? this.value1 ?? null;
var value1Check = lhs == rhs

应该做和你一样的事情,但可读性几乎降低了 100%!

于 2012-05-15T15:57:43.453 回答
0

如果您仍然想知道三元选项。

 value1_check= this.value1!=null? this.value1.Equals(c.value1):(c.value1!=null?c.value.Equals(this.value):value1_check=true);
于 2014-06-10T17:14:00.527 回答