如何简化这个C＃代码

Question

有没有更好的方式来写在vs2010 C＃中？

public bool IsAccept() 
    { 
     //check the status is accept 
     if (Status == null) 
      return false; 
     return Status.ToLower() == "accept"; 
    } 

    public bool IsRefer() 
    { 
     //check the status is refer 
     if (Status == null) 
      return false; 
     return Status.ToLower() == "refer"; 
    } 


    public bool IsAnyReviewState() 
    { 
     if (IsAccept() || IsRefer()) 
      return true; 
     return false; 
    } 

也许我还在学习C＃4中的简化方法。

Answer 1

public bool IsAccept() 
    { 
     return (Status == null)?false:(Status.ToLower() == "accept"); 
    } 

等等......

Answer 2

这个怎么样？

public bool IsAnyReviewState() 
{ 
    return new [] {"accept", "refer"}.Contains((Status??string.empty).ToString().ToLower()) 
} 

Answer 3

您可以使用短路评估来使代码更精确一些。 BTW。没有什么特别的C＃4。

public bool IsAccept() 
{ 
    //check the status is accept 
    return Status != null && Status.ToLower() == "accept"; 
} 

public bool IsRefer() 
{ 
    //check the status is refer 
    return Status != null && Status.ToLower() == "refer"; 
} 


public bool IsAnyReviewState() 
{ 
    return IsAccept() || IsRefer(); 
} 

Answer 4

1. 对于简单的存取这样的，物业语法比方法更好。
2. 使用String.Equals或String.IndexOf与StringComparison.OrdinalIgnoreCase而不是String.ToLower() ==。此外，如果您绝对必须对字符串进行规范化处理，那么请使用String.ToUpperInvariant，因为它会保留在执行ToLowerInvariant时丢失的某些信息，这会影响非拉丁语言，如土耳其语，这种语言具有特殊行为的I类字符。
3. 考虑使用String.IsNullOrEmpty
4. 请记住，布尔运算符本身返回布尔值，所以你可以大量简化你的IsAnyReviewState。

这是我会怎么写代码：

public bool IsAccept { 
    get { 
     return String.IsNullOrEmpty(this.Status) ? false : this.Status.Equals("accept", StringComparison.OrdinalIgnoreCase); 
    } 
} 

public bool IsRefer { 
    get { 
     return String.IsNullOrEmpty(this.Status) ? false : this.Status.Equals("refer", StringComparison.OrdinalIgnoreCase); 
    } 
} 

public bool IsAnyReviewState { 
    get { 
     return this.IsAccept || this.IsRefer; 
    } 
} 

Answer 5

我会用ternary operators像这样：

public bool IsAccept() 
    { 
     //check the status is accept 
     return Status == null ? false : Status.ToLower() == "accept"; 
    } 

Answer 6

就个人而言，我更喜欢使用string.Equals的字符串比较。在IsAccept()的情况下，我会写：

public bool IsAccept() 
{ 
    return string.Equals(Status, "accept", StringComparison.InvariantCultureIgnoreCase); 
} 

这将释放你写的防守null -checks并且不需要.ToLower()。

更少的代码：=快乐编码

编辑：请问Status属性需要一个string？也许你可以将它替换为枚举？

Answer 7

正如RvdK所说，这个问题应该在CodeReview。另外，我不确定您是否试图使代码更紧凑或更具可读性。

如果您试图使其更易读，唯一想到的就是修改return声明。

例如，而不是

return Status.ToLower() == "accept"; 

考虑

if(Status.ToLower() == "accept") 
     return true; 
    else 
     return false;