实践中的重构28_小心怀疑类库

一般而言，类库的使用频率较高，场景较多，隐藏的bug就较少。
因此，当我们发现代码中有问题的时候，首先应该怀疑应用的代码，只有确定了应用的正确性后，再去排查类库的bug。

        Money income = new Money(0);
        Money outgo = new Money(0);
        for (Money money : moneyList) {
            if (money.isIncome()) {
                income = income.add(money);
            } else {
                outgo = outgo.add(money);
            }
        }

        Money total = income == new Money(0) ? outgo.add(income) : income
                .add(outgo);

该段代码是摘自一个方法。该方法接收一个Money的列表，需要计算收入总额，支出总额，总额。Code review的时候，很明显，最后一行很可疑。
计算总额照理只需要income.add(outgo)就好了，不明白为什么写成这样。
和开发沟通后，他解释说是因为在自己测试中，发现new Money(0).add(new Money(100))的结果金额为0，new Money(100).add(new Money(0))的金额为100。于是就有了最后一行代码。
Money属于类库中的类，在系统中被大量使用，因此，我认为不会有这种问题。要求开发重现一下，开发重现不了，看开发重现的方式估计是手误为：

        Money i = new Money(0);
        Money j = new Money(100);
        System.out.println(i.add(i));

抛开这个手误不谈。加法是满足交换律的，如果发现Money类确实存在这个bug，也应该是通知Money类的维护者Money存在一个bug，请维护者修正Money类中的这个bug，而不是使用方使用奇怪的方式试图绕过该bug。

评论

1 楼 chenchao051 2011-07-24  

有点同感的。。。