重构警卫条款

在您的班级中管理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能力

来源