重構警衛條款

在您的班級中管理guard clause爆炸時，人們會採取哪些方法（如果有）？例如：重構警衛條款

public void SomeMethod<T>(string var1, IEnumerable<T> items, int count) 
{ 
    if (string.IsNullOrEmpty(var1)) 
    { 
     throw new ArgumentNullException("var1"); 
    } 

    if (items == null) 
    { 
     throw new ArgumentNullException("items"); 
    } 

    if (count < 1) 
    { 
     throw new ArgumentOutOfRangeException("count"); 
    } 

    ... etc .... 
}

在我目前正在研究的項目中，有許多類在公共方法上有一組類似的guard子句。

我知道的.NET 4.0代碼契約的但是這不是我們球隊目前的一個選項。

來源

2009-10-20 Kane

很多項目，我已經看到了使用靜態Guard類。

public static class Guard { 
    public static void ArgumentIsNotNull(object value, string argument) { 
     if (value == null) 
      throw new ArgumentNullException(argument); 
    } 
}

它使代碼更清潔，在我看來。

Guard.ArgumentIsNotNull(arg1, "arg1");

來源

2009-10-20 23:40:52

我只是發佈相同的東西。唯一的問題是它把這個方法放在堆棧跟蹤的頂端，而不是頂端的方法，而不是它那麼大。這種模式顯然可以用於不同的類型來檢查一系列的值，等等...... – 2009-10-20 23:42:59

是的，這是我曾經遇到過的唯一問題。雖然很容易找到原始的調用方法。 – 2009-10-20 23:46:19

這與模擬代碼合同的類基本相同。 – 2009-10-21 03:25:38

如果你不想下去代碼契約路線，簡化它的一種方式是去除括號：

public void SomeMethod<T>(string var1, IEnumerable<T> items, int count) 
{ 
    if (string.IsNullOrEmpty(var1)) 
     throw new ArgumentNullException("var1"); 

    if (items == null) 
     throw new ArgumentNullException("items"); 

    if (count < 1) 
     throw new ArgumentOutOfRangeException("count"); 

    ... etc .... 
}

除此之外，還有一些方法，你可以模擬代碼契約，如果你的反對意見是，.NET 4.0是不是黃金時間尚未：

http://geekswithblogs.net/Podwysocki/archive/2008/01/22/118770.aspx

來源

2009-10-20 23:35:49

看在上帝的份上，不要這樣做！如果沒有大括號的陳述是一個很好的方式來解決問題。 – EricRRichards 2018-02-27 15:04:06

@EricRRichards：* [聳聳肩] *坦率地說，如果程序員不能在沒有大括號的情況下保持代碼不變（特別是像這樣的不起眼的代碼），他們可能應該回到學校。 – 2018-02-27 15:47:35

你可能會考慮重構以Introduce a Null Object。

來源

2009-10-21 00:03:50

同時有關於這個位置的優秀文章：http://haacked.com/archive/2013/01/05/mitigate-the-billion-dollar-mistake-with-aspects.aspx/

我會考慮使用NullGuard.Fody因爲我很興奮Fodys能力

來源