【问题标题】:BigDecimal not keeping actual value when being returned from Java method从 Java 方法返回时,BigDecimal 不保留实际值
【发布时间】:2018-10-30 19:53:49
【问题描述】:

我正在用 Java 制作货币转换应用程序。其他一些很棒的 StackOverflowians 建议我阅读 BigDecimal 以替换 double 以解决任何精度问题。

我有两个方法系统;它从起始货币转换为美元,然后将美元价值转换为目标货币。

注意,我的转化率是这样存储的:

// Conversion Rates - START (as of October 30, 2018 @ 3:19 AM)
// Rates obtained from exchange-rates.org

//Convert to United States Dollar rates
private final BigDecimal CAD_TO_USD = new BigDecimal(0.76135);
private final BigDecimal EUR_TO_USD = new BigDecimal(1.1345);
private final BigDecimal YEN_TO_USD = new BigDecimal(0.008853);
// Conversion Rates - END

在我用它们各自的 BigDecimal 替换我的双打后 - 我决定对其进行测试,看看效果如何。

我的测试器类运行以下方法来启动转换过程。

public BigDecimal convert()
{
    BigDecimal value;

    value = convertToUSD(); //Converts the current currency into USD 
    value = convertFromUSD(value);  //Converts the previous USD currency value into the destination currency

    return value;
}

当我输入我的样本变量(将 2.78 日元转换为加元)时,我逐步完成了整个过程,发现一切正常,直到我返回一个值。

从前面提到的方法中,convertToUSD() 被运行并且编码如下

private BigDecimal convertToUSD()
{
    switch (fromCurrency)
    {
        case "USD":
            return fromQuantity.multiply(new BigDecimal(1));

        case "CAD":
            return fromQuantity.multiply(CAD_TO_USD);

        case "EUR":
            return fromQuantity.multiply(EUR_TO_USD);

        case "YEN":
            return fromQuantity.multiply(YEN_TO_USD);
    }

    return new BigDecimal(0);
}

所有值都正确传递,它逐步向下到正确的大小写(“YEN”),变量窗格显示“fromQuantity”BigDecimal 的 intCompact 值为 278(这对我来说很有意义)

一旦断点返回到“convert”方法,它就会变得一团糟。它不是返回2.78 * 0.008853 = 0.0246,而是返回-9223372036854775808

这会导致所有其他计算产生错误。

我是使用 BigDecimal 的新手,所以我可能会犯一个完全明显的错误;但是我很高兴学习,所以我向你们寻求建议:)

感谢任何帮助。

【问题讨论】:

  • 不要使用带双精度的构造函数。使用带字符串的那个。 double 值不会像您想象的那样映射到 BigDecimal。不确定这是你唯一的问题,但它会让你的开始很糟糕。
  • -9223372036854775808 的 intCompact 值为 Long::MIN_VALUE。如果它具有该值,则意味着不应使用它来计算 BigDecimal 值,并且 BigInteger intVal 包含 BigDecimal 的实际数字。因此,尽管您使用双精度数来初始化 BigDecimal,但它们的值可能不准确,但不应与事实相差太远。 不管怎样,intCompact 中的值在这里没有意义。该值只是忽略 intCompact 的标志。。您可以使用System.out.println(value); 检查 BigDecimal 的实际值。

标签: java precision bigdecimal


【解决方案1】:

tl;博士

使用 String,而不是 double 文字。

new BigDecimal( "2.78" )           // Pass "2.78" not 2.78
.multiply(
    new BigDecimal( "0.008853" )   // Pass "0.008853" not 0.008853
)
.toString()

0.02461134

不要传递浮点类型

BigDecimal 类的重点是避免floating-point 技术中的inherent inaccuraciesfloat/Floatdouble/Double 等浮点类型以牺牲精度换取执行速度。相比之下,BigDecimal 速度慢但准确。

您的代码:

new BigDecimal( 0.76135 )
new BigDecimal( 1.1345 )
new BigDecimal( 0.008853 )

…正在传递一个 double 原始文字。在编译期间,您键入的文本 0.76135 被解析为一个数字,特别是 double(一个 64 位浮点值)。那时你已经引入了这种类型固有的不准确性。换句话说,从0.76135 生成的double 可能不再完全是0.76135

让我们在实例化后立即转储您的 BigDecimal 实例。

System.out.println( new BigDecimal( 0.76135 ) );    // Passing a `double` primitive.
System.out.println( new BigDecimal( 1.1345 ) );
System.out.println( new BigDecimal( 0.008853 ) );

0.7613499999999999712230192017159424722194671630859375

1.13450000000000006394884621840901672840118408203125

0.0088529999999999997584154698415659368038177490234375

因此,通过创建 double 数值,您调用了浮点技术,并引入了不准确性。

使用字符串

解决方案?使用字符串,完全避免使用 double 类型。

在这些输入周围加上一些双引号,然后

System.out.println( new BigDecimal( "0.76135" ) );  // Passing a `String` object.
System.out.println( new BigDecimal( "1.1345" ) );
System.out.println( new BigDecimal( "0.008853" ) );

0.76135

1.1345

0.008853

示例

您期望2.78 * 0.008853 = 0.0246。让我们试试吧。

BigDecimal x = new BigDecimal( "2.78" );
BigDecimal y = new BigDecimal( "0.008853" );
BigDecimal z = x.multiply( y );
System.out.println( x + " * " + y + " = " + z );

2.78 * 0.008853 = 0.02461134

接下来,您应该学习使用 BigDecimal 进行舍入和截断。已经在 Stack Overflow 上介绍过很多次了。

【讨论】:

  • 这准确地解释了代码错误的原因,但也应该解释如何修复它。
  • BigDecimal 的重点不是 floatdouble 不准确,而是后两者使用二进制算术,BigDecimal 使用十进制算术,这更有意义比二进制算术还要多的钱。一般来说,将二进制算术应用于打算成为小数的数字有时会导致不直观的结果,正如BigDecimal(double) 构造函数所示。
  • @PeterO。 “不直观的结果”=“不准确”。虽然我在学术意义上理解你的观点,但归根结底,我的客户只关心发票是否正确总计、薪水是否正确以及分类帐余额。
  • @Basil :这是我的观点:二进制和十进制算术不应该混合使用。
【解决方案2】:

问题是您假设intCompact 代表您的浮点数的非浮点版本。在某些情况下会,但在大多数情况下不会。

例如,我试图将您的问题减少到最低限度。

import java.math.BigDecimal;

class Main {
  public static void main(String[] args) {
    final BigDecimal YEN_TO_USD = new BigDecimal(0.008853);
    BigDecimal value = new BigDecimal(2.78);

    value = value.multiply(YEN_TO_USD);
    System.out.println(value);
  }
}

println 换行我得到以下信息:

您会看到intCompact 与您的相同 (-9223372036854775808)。但本例中的stringCache 是预期值。

请参阅 Basil 的回答,了解为什么应该使用字符串而不是双精度来构造 BigDecimal。

【讨论】:

    猜你喜欢
    • 2018-11-19
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    相关资源
    最近更新 更多