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

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

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

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 1
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论 1
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值