2012-04-02 83 views
-2

代码是 - 在里面创建一个calcTotal,calcSalesTax和calcSubTotal方法的计算类。应提示用户输入数量和价格,税金为0.7。我需要在Main中调用这个类,然后输出小计,税和总额。创建一个简单的计算类,输出Java中的总数

到目前为止,这是我在calulator类

package com.Nick.Calculator; 

public class Calculator { 
    public double calcSubTotal(double amount, double qty){ 
     double subTotal; 
     subTotal=qty*amount; 
     return subTotal; 
    } 

    public static double calcSalesTax(double subTotal, double taxAmount){ 
     double tax=0.7; 
     taxAmount=subTotal*tax; 
     return taxAmount; 
    } 

    public static double calcTotal(double subTotal, double taxAmount){ 
     double total; 
     total=subTotal+taxAmount; 
     return total; 
    } 
} 

请问什么都需要在这个类中去,或者我可以提示主用户,以及如何我正确地调用这些函数主要?谢谢

+0

正在做作业吗?你应该给它加上标记... – 2012-04-02 18:13:22

+0

为什么你在第二个函数中有“taxAmount”函数参数?你永远不会使用它的价值,但你改变它......永远不要这样做! – dexametason 2012-04-02 18:15:32

+0

谢谢你们!我得到它的工作! – 2012-04-05 19:49:24

回答

0

该类看起来不错,我会保持逻辑从任何用户提示分开。最快的方法是在你的主要提示用户。在你的主实例化一个新的计算器类:

//beginning of main, prompt user and capture input feed into below lines 
Calculator taxCalc = new Calculator(); 
double subTotal = taxCalc.calcSubTotal(amount, qty); 
double taxAmount = Calculator.calcSalesTax(subTotal, taxAmount); 
//... you should get the idea from here, print the result back to the user 

一些意见寿,为什么你的方法计算器非静态和静态剩下一个?我会让他们都不是静态的,因为为他们编写测试会更容易。或者有些人可能会认为逻辑太简单了,静态会很好。你的电话,但要去一个或另一个。同样在你的calcSalesTax调用中,你将结果设置为传入的参数。这通常是不好的做法,我会直接返回值或为它创建一个新对象。

1

如果课程的目的只是为了计算,不要要求用户在课堂上输入。坚持你的课堂的目的。