IT痴汉的工作现状45-我掉入了陷阱

是的,伟仔我到现在还在调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”。

太有才了!
本来递归调用就带来了阅读代码的成本,而在这种有返回值的方法中用了递归的变种,这在复杂的代码中更是增加了无穷的隐患。

伟仔表示,今天算是掉坑里了。

你可能感兴趣的:(花园随笔,IT痴汉的工作现状)