方法不完全奏效？可能是什么错误？

Question

我只是在做一些项目，我找不到以下代码有什么问题：

double getInterest(double amt,double tax){  
        double diff=0;
        double refund=0;
        double interest=0;
        if(amt>=(tax*0.4)){
           diff=amt-(tax*0.4);
           System.out.println(" "+diff);  //This is not working.Nothing gets printed.
           refund=diff;                   //The value is not being assigned to refund.
        }                                 //refund remains to be 0 throughout.
         interest=diff*0.01*12;
         if(refund>0){
            JOptionPane.showMessageDialog(null,"Your refund is: "+refund);
         }
        return interest;
}

我无法找出代码中的任何错误。但它仍然不能完全工作。
错误以cmets形式在代码中给出。
有人帮我找出错误并解决这个问题吗？

我不知道为什么diff 的值没有被打印出来，以及为什么diff 的值没有被分配给变量refund。

我在 NetBeans 中工作，它没有给出任何类型的编译时错误。

我尝试使用println 来检查diff 和refund 的值，但是这些值没有被打印出来。

Answer 1

如果if 块中的语句从未被执行，这意味着在参数满足条件amt >= (tax*0.4) 的情况下永远不会调用此方法。所以真正的问题是，你传递给这个方法的值是什么？

Answer 2

您的包围 if(amt>=(tax*0.4){ 行在第一个 { 括号之前缺少一个结束 ) 括号。确保检查您的包围并确保您在选项菜单中打开了所有错误（如果可以的话）。

由于括号错误，此代码不应编译。检查 Netbeans 是否正在编译正确的项目，因为这应该在编译时发疯。

Answer 3

在处理货币时应始终使用BigDecimal 类型以避免double precision issues。

这是BigDecimal中重写的代码

BigDecimal getInterest(final BigDecimal amount, final BigDecimal tax) {
    BigDecimal diff = BigDecimal.ZERO;
    BigDecimal refund = BigDecimal.ZERO;
    BigDecimal reducedTax = new BigDecimal("0.4").multiply(tax);
    System.out.println("Amount is " + amount + ". Tax times 0.4 is " + tax + ".");
    if (amount.compareTo(reducedTax) >= 0) {
        diff = amount.subtract(reducedTax);
        System.out.println(diff);
        refund = diff;
    }
    BigDecimal interest = diff.multiply(new BigDecimal("0.01")).multiply(new BigDecimal("12"));
    if (refund.compareTo(BigDecimal.ZERO) > 0) {
        JOptionPane.showMessageDialog(null, "Your refund is: " + refund);
    }
    return interest;
}

是的，我知道它更丑陋。它也会慢很多。但它是新的正确。

关于您的问题，大概代码没有输入if，因为amount 不大于tax * 0.4 - 如果您确定是这样，那么您可能已经遇到了双精度问题。

我在if 之前添加了一个println，因此您可以在测试发生之前查看这些值。