我有以下内容,想知道初始测试是否过大:
static void Main(string[] args) {
if (args.Length == 0 || args == null) {
//do X
}
else {
//do Y
}
}
换句话说,我要问的是 args.Length 是否有可能为零,或者 args 为空......或者这些条件之一是否就足够了?
好吧,Main
它被定义为永远不会用null
参数调用。如果它确实以某种方式接收到一个空参数,那么你的环境已经被破坏了,无论你做什么,所有的赌注都被取消了,所以通过检查null
.
另一方面,如果您确实检查了 null,那么代码的读者和维护者将不得不理解为什么。为什么最初的程序员要进行如此无用的检查?他知道我们不知道的事情吗?我们不能只是删除它,因为他可能发现了一些奇怪的角落案例错误!
换句话说,您正在增加程序的复杂性,并绊倒代码的未来读者。不要那样做。那个未来的用户可能是你。让你未来的自己快乐,写出有意义的代码。
但是,在这种空检查确实有意义的情况下,它必须是最左边的条件。
在这样的测试中:args.Length == 0 || args == null
,首先args.Length
被评估,如果失败,则与. 换句话说,如果为 null,您的代码将抛出异常。它应该是args
null
args
args == null || args.Length == 0
做额外的控制从来都不是坏主意,如果这不是高性能,非常常用的功能。所以我想说,不,这不是矫枉过正。
还有一件事:首先检查null
,然后检查Length
如果没有输入数据,则 args.Length 等于 0,但不为 null。如果有任何输入数据,则 agrs.Length 等于输入参数的计数。总之,args 不能为空,但长度可以为零。
PS 总是首先检查 null
if (args == null)
{
Console.WriteLine("args is null"); // Check for null array
}
else
{
if (args.Length == 0)
{
//do X
}
else
{
//do Y
}
}