少用 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); // 其他还有很多...
这样,代码显得整洁了许多。