3

我有一段简单的代码，它的行为不正常。

这段代码试图添加一个 BigDecimals 数组，然后除以 array.length 以找到平均值。但是，算法的第一阶段未能正确地将数组相加在一起（在变量“sum”中）。

public BigDecimal getAverageHeight()
{
    BigDecimal sum = new BigDecimal(0);
    BigDecimal[] heights = getAllHeights();

    for (int a = 0; a < heights.length; a++)
    {
        sum.add(heights[a]);
        System.out.println("Height[" + a + "] = " + heights[a]);
        System.out.println("Sum = " + sum.setScale(2, BigDecimal.ROUND_HALF_UP));
    }        

    return sum.divide(new BigDecimal(heights.length));
}

输出如下：

Height[0] = 24  
Sum = 0.00  
Height[1] = 24  
Sum = 0.00  
Height[2] = 24  
Sum = 0.00  
Height[3] = 26  
Sum = 0.00  
Height[4] = 26  
Sum = 0.00  
Height[5] = 26  
Sum = 0.00

我确定这是一个简单的错误，但我已经厌倦了关注这个问题，提前谢谢。

4

3 回答 3

5

BigDecial.add() 返回总和，它不会改变它。做这个：

sum = sum.add(heights[a]);
于 2012-11-05T10:27:56.320 回答
4

BigDecimal对象是不可变的，其所有修改其值的方法都返回一个新BigDecimal对象。新创建的对象包含修改的值。

你需要做这样的事情：

sum = sum.add(heights[a]);

这也适用于setScale()anddivide()操作

于 2012-11-05T10:27:58.900 回答
1

将 sum 加法运算的结果值赋回变量 sum

for (int a = 0; a < heights.length; a++)
    {
        sum = sum + heights[a];
        System.out.println("Height[" + a + "] = " + heights[a]);
        System.out.println("Sum = " + sum.setScale(2, BigDecimal.ROUND_HALF_UP));
    }
于 2012-11-05T10:29:33.110 回答