标签:
是的,伟仔我到现在还在调bug!
伟仔是有代码洁癖的,容不得半点瑕疵。
在重构一个类(这个类是其他童鞋的作品,我这么给人家重构,是不是讨厌呢?)时,把一些不好的命名修改了,把不用的注释删掉,把不当的判断修整,把可以作为函数局部变量的放回到相应函数中,等等等等。
看着清爽的代码,我高兴的喝口水。
代码提交后,散散步。
回来后,觉得还是测试一下吧。虽然只是重构了无关痛痒的一些地方,但规矩还是要遵守的。(其实还是违规了,没测试就提交代码本身就要受到惩罚的!但谁让伟仔这么自信呢?)
编译。。。
运行。。。
我擦,刚刚还好好的读卡程序,怎么读不出来内容啦??!
仔细检查我的提交,发现没有修改影响到读卡。
那就奇怪了,那么多简单的修改真的不至于影响大功能。
还是回退代码吧。新拉一个branch下代码,并回退到前一个版本。
噫?还是不好用。好好看看?仔细研究了下,前一个版本是好用的。
还是我这次修改引起的。
很是不理解,带着不爽,把代码慢慢还原排查。经历了多次,仍然读不出来。
我觉得这事情很有意思了,我决定一探究竟。
在我觉得绝不可能的时候,把一个局部变量还原成类成员变量时,奇迹发生了!好用了!
这让我太不可思议了。
好吧,仔细读读代码吧。发现很长的一个方法,里面隐藏了一个递归调用,导致返回值为空!!!
对,就是那个类成员变量。
这是怎么回事呢?让我写一份伪代码说明下:
public class Recursive {
//static String result;
static String callme(int input) {
String result = "";
//result = "";
if(input == 1) {
callyou(1);
} else if(input ==2 ) {
callyou(2);
} else if(input == 3) {
result = "3";
System.out.println("input ==3: "+result);
return result;
}
System.out.println("return: "+result);
return result;
}
static void callyou(int input) {
callme(input+1);
}
public static void main(String[] args) {
String answer = callme(1);
}
}
请问:
1、callme会返回几次?
2、answer的值是什么?
因为方法被调用了三次,那么会对应的返回三次。
这里的answer值为“”而不是“3”。
太有才了!
本来递归调用就带来了阅读代码的成本,而在这种有返回值的方法中用了递归的变种,这在复杂的代码中更是增加了无穷的隐患。
伟仔表示,今天算是掉坑里了。
标签:
原文地址:http://blog.csdn.net/lincyang/article/details/51334916