《编写高质量代码：改善Java程序的151个建议》 建议5

建议5 别让null值和空值威胁到变长的方法

思考一下，下面代码的逻辑：

public void methodA(String str, Integer...is)
{ ... }
public void methodA(String str, String...strs)
{ ... }

methodA("China", 0);
methodA("China", "People");
methodA("China");
methodA("China", null);

两个methodA都进行了重载，现在的问题是：上面的代码编译通不过，问题出在什么地方？

有两处编译通不过：methodA(“China”); 和 methodA("china", null); 提示是“方法模糊不清”，编译器不知道调用哪一个方法，但这两处代码反映的代码味道可是不同的。

1. 对于methodA("China");这个调用，两个方法都符合这个形参格式，所以编译器报错。因为变长参数的实参数量可以是N个(N>=0)

为什么程序符合规则却不能运行，谁之责任？

责任应该是这段代码的设计者，他违反了 KISS 原则(Keep It Simple, Stupid, 即懒人原则)。致使程序竟然出错了，这对设计者和开发者而言都是应该严禁出现的。

2. 对于methodA("China", null);这个调用，因为直接量 null 是没有类型的，还是会造成不知道调用哪个的情况，所以编译器报错。

以上代码的坏味道：

除了不符合上面的懒人原则外，这里还有一个非常不好的编码习惯，即调用者隐藏了实参类型，这是非常危险的，不仅仅调用者需要“猜测”该调用哪个方法，而且被调用者也可能产生内部逻辑混乱的情况。对于本例来说应该做如下修改：

String[] strs = null;
methodA("China", strs);

也就是说 让编译器知道这个null值是string类型的，编译器即可顺利通过，也就减少了错误的发生。