计算销售佣金和折扣后的价格

Question

代码已经起作用了，但是我只想要一些改进的提示。

import javax.swing.JOptionPane;
import java.text.DecimalFormat;
public class Calculator
{
    public static void main(String[] args)
    {
        double initialPrice = 0.00;
        double commissionRate = 0;
        double discountRate = 0;
        initialPrice = Double.parseDouble(JOptionPane.showInputDialog(null, "What is the initial price total of the sale?"));
        commissionRate = Double.parseDouble(JOptionPane.showInputDialog(null, "What is the percentage amount of the sales commission? \nFor example for 20%, type 20."));
        discountRate = Double.parseDouble(JOptionPane.showInputDialog(null, "What is the customer's discount rate? \nFor example for a 15% discount, type 15."));
        JOptionPane.showMessageDialog(null, "The final cost to the customer is \n$" + computeFinalPrice(initialPrice, commissionRate, discountRate) + ".");
    }
    public static String computeFinalPrice (double initialPrice, double commissionRate, double discountRate)
    {
        double finalPrice;
        DecimalFormat d = new DecimalFormat("0.00");
        finalPrice = (initialPrice*(commissionRate+100)*.01)*((100-discountRate)*.01);      
        return (d.format(finalPrice));
    }

Answer 1

有趣的是，请考虑以下几点：

• 对于额外的DRYness，您可以创建一个助手函数来获得一个双私有的双getDouble( String )
• computeFinalPrice可能应该是私有的
• 如果您希望computeFinalPrice可重用，那么返回一个double并使其公开可能更有意义。
• 一家公司可能宁愿计算折扣后的佣金，否则销售人员会对提供99%的折扣感到非常兴奋;)
• 您可以用main()这样的零初始化双数，也可以在computeFinalPrice中不喜欢。
• 正如另一位评论员所提到的，考虑长而不是双倍，因为双倍可能存在舍入问题。
• 我会将(initialPrice*(commissionRate+100)*.01)*((100-discountRate)*.01)；= finalPrice写成finalPrice = initialPrice\*(commissionRate+100)\*.01\*(100-discountRate)\*.01;，如果我想搅乱代码审查员，我甚至可能会写finalPrice = initialPrice\*(commissionRate+100)\*(100-discountRate)\*.0001;

Answer 2

请注意:不要在需要精确结果的地方使用浮点变量：

• 有效Java，第二版，第48项:如果需要确切答案，请避免浮动和加倍
• 为什么不使用双倍或浮动来表示货币呢？

幸运的是，Java有BigDecimal。