简单方程未通过单元测试

Question

我有一个接受 2 个整数作为参数并返回的方法：

 public int method(int a, int b){
   return Math.round((Math.abs(a-b)/3) - (Math.min(a,b)-1500)/10)
 }

我创建了单元测试，它通过了值 1400 和 1500（预期值为 43），但是对于 1459 和 1500 失败了。预期输出是 18，但是我的方法返回17. 我相信这可能与舍入有关，但是我看不到任何明显的错误。将 17.7(6) 舍入到 18 应该不会有任何问题。

编辑：

实际函数略有不同（我没有 Math.abs(ab) 但我因此定义了“diff”变量。我可以向你们保证，我将其声明为 double diff； - 我不知道为什么它变成 int diff :) 已解决，谢谢 :)

Answer 1

当将 int 作为参数传递时，Math.abs 和 Math.min 都返回 int，因此您的代码是进行整数除法而不是您期望的双除法。如果将 3 和 10 替换为 3.0 和 10.0，您的代码应该按您的预期工作。

Answer 2

这是因为它正在执行整数除法。对于 a = 1459、b = 1500：

Math.round((Math.abs(a-b)/3) - (Math.min(a,b)-1500)/10)
Math.round(Math.abs(41)/3 - (1459-1500)/10)
Math.round(41/3 - (-41)/10)
Math.round(13 - (-4))
Math.round(17)       // note that round will always get an integer as input
17

简单的解决方法是通过使用浮点数作为参数之一来强制其执行浮点除法：

Math.round((Math.abs(a-b)/3.0) - (Math.min(a,b)-1500)/10.0)

Answer 3

使用整数除法时期望出现双精度数肯定是本书中最古老的问题。 ;)

您的回合不会执行任何操作，因为您的整数除法会产生整数结果。

Answer 4

预期结果是 17。

|a-b| = 41
|a-b|/3 = 13

min(a,b) = 1459
min(a,b)-1500 = -41
(min(a,b)-1500)/10 = -4

13-(-4) = 17

我怀疑您不想进行整数除法。因此，您必须将 3 和 10 替换为 3.0 和 10.0。

Answer 5

您必须记住要考虑到这样一个事实：在 Java 中，当一个整数除以另一个整数时，结果是一个整数，而不是浮点数（任何余数都会被丢弃）。例如 6/4 == 1，而不是 1.5。

方法(1459, 1500)

Math.round((Math.abs(1459-1500)/3) - (Math.min(1459,1500)-1500)/10);
Math.round((41/3) - (1459-1500)/10);
Math.round(13 - (-41)/10); //41/3 == 13, not 13.6666
Math.round(13 - -4); //-41/10 == -4, not -4.1
Math.round(17);
17