为什么我的函数始终无法通过JUnit测试？

Question

这是我正在测试的课程之一。 它一直使calculateVolume（）方法失败，我不确定为什么。

   package shape3D;


public class Sphere implements Cloneable {
    private double myRadius;
    private double myVolume;
    private double mySurfaceArea;
    private final static double pi = 3.14;

    public static void main(String args[]){
        Sphere sphere = new Sphere();
    }
    public double calculateVolume(){
        myVolume = (4/3)*pi*(Math.pow(myRadius,3));
        return myVolume;
    }
     public double calculateSurfaceArea(){
         mySurfaceArea = ((4)*(pi)*(Math.pow(myRadius,2)));
         return mySurfaceArea;
     }
     public double getSurfaceArea(){
         return this.calculateSurfaceArea();
     }
     public double getVolume(){
         return this.calculateVolume();
     }
     public void setRadius(double radius2){
         myRadius = radius2;
     }

     public String toString(){
            return "Volume: " + this.getVolume() + "   Surface area " + this.getSurfaceArea();
        }
     public Sphere clone (){
            Sphere p = new Sphere();
            p.setRadius(myRadius);
            return p;
        }

}

这是我正在使用的JUnit测试用例

public class sphereTest {
    @Test
    public void testSphere(){
        shape3D.Sphere sphere = new shape3D.Sphere();
        sphere.setRadius(6);
        assertTrue(sphere.calculateSurfaceArea()== 452.16);
        assertTrue(sphere.calculateVolume()== 904.32);

computeSurfaceArea（）东西通过得很好，但是卷失败了，我不确定为什么。

Answer 1

你计算体积截断方程式的第一项时做整数除法1 。 更换

myVolume = (4 / 3) * pi * (Math.pow(myRadius, 3)); // 678.24

与

myVolume = (4 / 3.0) * pi * (Math.pow(myRadius, 3)); // 904.31

由于浮点数的不精确性，您仍然需要考虑期望值和计算值之间的差异。 您可以使用此版本的assertEquals允许使用增量值进行比较-替换

assertTrue(sphere.calculateVolume()== 904.32);

与

assertEquals(sphere.calculateVolume(), 904.32, .02);

Answer 2

计算

myVolume = (4/3)*pi*(Math.pow(myRadius,3));

使用整数运算： 4/3计算为1 。

更改为

myVolume = (4.0/3)*pi*(Math.pow(myRadius,3));

Answer 3

除了4/3整数问题（我没有发现）之外，它还比较：用这种方法比较两个Java double值并不安全。

如果使用assertEquals而不是assertTrue则可能会看到问题。 我敢打赌，它先计算4/3，然后将其截断。 即使将它们变成双倍，它仍然会这样做。

如本问题所述，将重载用于assertEquals。