等效縮寫

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") };