一般而言,类库的使用频率较高,场景较多,隐藏的bug就较少。
因此,当我们发现代码中有问题的时候,首先应该怀疑应用的代码,只有确定了应用的正确性后,再去排查类库的bug。
该段代码是摘自一个方法。该方法接收一个Money的列表,需要计算收入总额,支出总额,总额。Code review的时候,很明显,最后一行很可疑。
计算总额照理只需要income.add(outgo)就好了,不明白为什么写成这样。
和开发沟通后,他解释说是因为在自己测试中,发现new Money(0).add(new Money(100))的结果金额为0,new Money(100).add(new Money(0))的金额为100。于是就有了最后一行代码。
Money属于类库中的类,在系统中被大量使用,因此,我认为不会有这种问题。要求开发重现一下,开发重现不了,看开发重现的方式估计是手误为:
抛开这个手误不谈。加法是满足交换律的,如果发现Money类确实存在这个bug,也应该是通知Money类的维护者Money存在一个bug,请维护者修正Money类中的这个bug,而不是使用方使用奇怪的方式试图绕过该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。