少用 if 进行普通意义上的参数验证

每次在写 public 方法的时候第一件事就是参数验证。以往,都会有这样的代码出现:

if (oneOfTheArgument == null)
{
    throw new ArgumentNullException(
        "oneOfTheArgument",
        StringTable.ErrorClassNameArgNull);
}

但是方法的参数表有的时候比较长,逻辑判断的组合也比较多,此时这种语句的数目有时多大6-7个。这样的代码喧宾夺主,可读性差(从纯数据的角度而言,圈复杂度也会比较高)。于是目前全部换成了这种写法:

oneOfTheArgument 是 null 那么抛出 ArgumentNullException.
Error.Argument.IsNull(oneOfTheArgument);

// 带自定义消息提示的重载.
Error.Argument.IsNull(oneOfTheArgument, StringTable.ErrorClassNameArgNull); 

相应的还可以有:

// 查看 value 是否为 null,或者仅包含空白字符,如果为 null,
// 则抛出 ArgumentNullException,如果仅包含空白字符或者是
// string.Empty,则抛出 ArgumentException.
Error.Argument.IsNullOrWhitespaceString(value, parameterName, message);

// 如果 value 小于 minValue,则抛出 ArgumentOutOfRangeException.
Error.Argument.LessThan(value, minValue, message);

// 如果 value 在一个指定范围之外,则抛出 ArgumentOutOfRangeException.
Error.Argument.OutOfRange(value, minIncl, maxIncl, message);

// 其他还有很多...

这样,代码显得整洁了许多。

posted @ 2011-01-12 20:55  TW-刘夏  阅读(686)  评论(3编辑  收藏  举报