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”。

太有才了！

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

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