等效缩写

Question

与以下代码片段相当的速记是什么？

if (strValue == ""){ 
    throw new Exception("Mandatory 'strValue' parameter empty"); 
} 

Answer 1

它可能尽可能短，除非删除空格和大括号（并牺牲了可读性）。

至于正确性...这可能会更好：

.NET 4.0：

if (string.IsNullOrWhiteSpace(strValue)){ 
    throw new ArgumentException("Mandatory 'strValue' parameter empty"); 
} 

.NET < 4.0：

if (string.IsNullOrEmpty(strValue)){ 
    throw new ArgumentException("Mandatory 'strValue' parameter empty"); 
} 

另外请注意，这是不好的做法，简单地扔Exception - 从BCL中选择适当的异常类（如果存在的话）或自定义的类（如果不存在的话）会更好。 （谢谢@djacobson）

Answer 2

您可以使用代码合同。

您也可以使用string.IsNullOrWhitespace（）

Contract.Requires(string.IsNullOrEmpty(strValue), "** fancy localized message here ***"); 

Answer 3

if(strValue=="")throw new Exception("Mandatory 'strValue' parameter empty"); 

所有你能做的就是去掉括号和空格:)

Answer 4

它已经短。而不是做strValue ==“”，我会做String.Empty或String.NullOrEmpty，我不记得哪一个可用于.NET。

Answer 5

不会变得更短，但如果需要更少的行：

if (String.IsNullOrWhitespace(strValue)) throw new Exception("Mandatory 'strValue' parameter empty"); 

Answer 6

随着空校验，我想你想和使用的ArgumentException：

ThrowIfNullOrEmpty(strValue, "strValue"); 

... 

private void ThrowIfNullOrEmpty(string parameterValue, string parameterName) 
{ 
    if String.IsNullorEmpty(parameterValue) 
    { 
     throw new ArgumentException("Mandatory 'strValue' parameter empty", 
            parameterName); 
    } 
} 

显然只有当你在做多几次这样更加有用。

Answer 7

假设你正在尝试写更多的防守代码，你可以使用Trace.Assert

Trace.Assert(strValue != "", "Mandatory 'strValue' parameter is not empty"); 

或者你可以使用Fluent Validation库封装更复杂的验证。

Answer 8

你几乎就像你可以得到的一样短。我建议使用IsNullOrEmpty字符串函数来检查一个空字符串。另外，在您的异常处理中可能更具体，并抛出ArgumentException。

if (String.IsNullOrEmpty(strValue)) { throw new ArgumentException("strValue must not be null or empty") };