打字猴:1.700438574e+09
1700438574 编写高质量代码:改善Java程序的151个建议 [:1700438072]
1700438575 编写高质量代码:改善Java程序的151个建议 建议5:别让null值和空值威胁到变长方法
1700438576
1700438577 上一建议讲解了变长参数的重载问题,本建议还会继续讨论变长参数的重载问题。上一建议的例子是变长参数的范围覆盖了非变长参数的范围,这次我们从两个都是变长参数的方法说起,代码如下:
1700438578
1700438579 public class Client{
1700438580
1700438581 public void methodA(String str, Integer……is){
1700438582
1700438583 }
1700438584
1700438585 public void methodA(String str, String……strs){
1700438586
1700438587 }
1700438588
1700438589 public static void main(String[]args){
1700438590
1700438591 Client client=new Client();
1700438592
1700438593 client.methodA(“China”,0);
1700438594
1700438595 client.methodA(“China”,“People”);
1700438596
1700438597 client.methodA(“China”);
1700438598
1700438599 client.methodA(“China”,null);
1700438600
1700438601 }
1700438602
1700438603 }
1700438604
1700438605 两个methodA都进行了重载,现在的问题是:上面的代码编译通不过,问题出在什么地方?看似很简单哦。
1700438606
1700438607 有两处编译通不过:client.methodA(“China”)和client.methodA(“China”,null),估计你已经猜到了,两处的提示是相同的:方法模糊不清,编译器不知道调用哪一个方法,但这两处代码反映的代码味道可是不同的。
1700438608
1700438609 对于methodA(“China”)方法,根据实参“China”(String类型),两个方法都符合形参格式,编译器不知道该调用哪个方法,于是报错。我们来思考这个问题:Client类是一个复杂的商业逻辑,提供了两个重载方法,从其他模块调用(系统内本地调用或系统外远程调用)时,调用者根据变长参数的规范调用,传入变长参数的实参数量可以是N个(N>=0),那当然可以写成client.methodA(“china”)方法啊!完全符合规范,但是这却让编译器和调用者都很郁闷,程序符合规则却不能运行,如此问题,谁之责任呢?是Client类的设计者,他违反了KISS原则(Keep It Simple, Stupid,即懒人原则),按照此规则设计的方法应该很容易调用,可是现在在遵循规范的情况下,程序竟然出错了,这对设计者和开发者而言都是应该严禁出现的。
1700438610
1700438611 对于client.methodA(“china”,null)方法,直接量null是没有类型的,虽然两个methodA方法都符合调用请求,但不知道调用哪一个,于是报错了。我们来体会一下它的坏味道:除了不符合上面的懒人原则外,这里还有一个非常不好的编码习惯,即调用者隐藏了实参类型,这是非常危险的,不仅仅调用者需要“猜测”该调用哪个方法,而且被调用者也可能产生内部逻辑混乱的情况。对于本例来说应该做如下修改:
1700438612
1700438613 public static void main(String[]args){
1700438614
1700438615 Client client=new Client();
1700438616
1700438617 String[]strs=null;
1700438618
1700438619 client.methodA(“China”,strs);
1700438620
1700438621 }
1700438622
1700438623 也就是说让编译器知道这个null值是String类型的,编译即可顺利通过,也就减少了错误的发生。
[ 上一页 ]  [ :1.700438574e+09 ]  [ 下一页 ]