2

申請ステータスが独身の人の税金を計算するこの方法があります。しかし、これを for ループに変換したり、while ループの代わりに Array を使用したりするのに苦労しています。コードを簡略化して見栄えを良くしたい。助言がありますか？

public void calculateTax()
    {
        //The constant fields for the tax rate
        final double TAXRATE_10 = 0.1;     //10%
        //This is the tax rate percent on the tax 15%
        final double TAXRATE_15PERCENT = 0.15;
        //This is the tax rate percent on the tax 25%
        final double TAXRATE_25PERCENT = 0.25;
        //This is the tax rate percent on the tax 28%
        final double TAXRATE_28PERCENT = 0.28;
        //This is the tax rate percent on the tax 33%
        final double TAXTRATE_33PERCENT = 0.33;
        //This is the tax rate percent on the tax 35%
        final double TAXRATE_35PERCENT = 0.35;

        //constant numbers for tax boundaries.
        final int NOTRICH = 8700;
        final int MIDDLECLASS = 35350;
        final int SORTOFRICH = 85650;
        final int RICH = 178650;
        final int FORSURERICH = 388350;

        //Variables for taxable income, and tax calculation.
        long taxableIncome = income - deduction -(numberOfExemption * VAlUEOFEXEMPTION);
        double cumulatedTax = 0;

        //Calculate the Tax
        while(taxableIncome != 0)
        {
            if(taxableIncome > FORSURERICH)
            {
                cumulatedTax += ((taxableIncome-FORSURERICH) * TAXRATE_35PERCENT);
                taxableIncome = (long)FORSURERICH;
            }
            else if(taxableIncome > RICH)
            {
                cumulatedTax += ((taxableIncome-RICH) * TAXTRATE_33PERCENT);
                taxableIncome = (long)RICH;
            }
            else if(taxableIncome > SORTOFRICH)
            {
                cumulatedTax += ((taxableIncome-SORTOFRICH) * TAXRATE_28PERCENT);
                taxableIncome = (long)SORTOFRICH;
            }
            else if(taxableIncome > MIDDLECLASS)
            {
                cumulatedTax += ((taxableIncome-MIDDLECLASS) * TAXRATE_25PERCENT);
                taxableIncome = (long)MIDDLECLASS;
            }
            else if(taxableIncome > NOTRICH)
            {
                cumulatedTax += ((taxableIncome-NOTRICH) * TAXRATE_15PERCENT);
                taxableIncome = (long)NOTRICH;
            }
            else
            {
                cumulatedTax += ((taxableIncome) * TAXRATE_10);
                taxableIncome = 0;
            }
        }
4

4 に答える 4

12

これはどう？よりオブジェクト指向的に考える。私はいくつかのクラスでそれを行いましたが、アイデアがあれば自分で機能を追加できます;)

Decorator Patternを使用して実装します。

共通インターフェース。

public interface TaxCalculator {

    Double calculate(Double tax);

}

すべての税金の基本計算。

public class TaxCalculatorBase implements TaxCalculator{

    @Override
    public Double calculate(Double tax) {
        return  tax * TAXRATE_10;
    }

}

デコレータ抽象クラス

public abstract class TaxCalculatorDecorator implements TaxCalculator{

    private final TaxCalculator decoratee;

    /**
     * @param decoratee
     */
    public TaxCalculatorDecorator(TaxCalculator decoratee) {
        super();
        this.decoratee = decoratee;
    }

    @Override
    public Double calculate(Double tax) {
        Double returnValue = decoratee.calculate(tax); 
        return taxCalculate(returnValue);
    }

    protected abstract Double taxCalculate(Double tax);


}

そしてデコレータの具象クラス。例として2つだけやった

public class NotRichTaxCalculator extends TaxCalculatorDecorator{

    public NotRichTaxCalculator(TaxCalculator taxCalculator) {
            super(taxCalculator);
    }

@Override
protected Double taxCalculate(Double tax) {
    return ((tax-NOTRICH) * TAXRATE_15PERCENT);
}

}

豊富な税計算機の並べ替え

public class SortOfRichTaxCalculator extends TaxCalculatorDecorator{

    public SortOfRichTaxCalculator(TaxCalculator decoratee) {
        super(decoratee);
    }

    @Override
    protected Double taxCalculate(Double cumulatedTax) {
        return  ((cumulatedTax-SORTOFRICH) * TAXRATE_28PERCENT);;
    }



}

オブジェクトを作成する単純な Factory

public final class TaxCalculatorFactory {

    private TaxCalculatorFactory(){}

    public static TaxCalculator create(Double taxableIncome){

        TaxCalculator taxCalculator= null;      

          if(taxableIncome > SORTOFRICH)
         {
             taxCalculator = new SortOfRichTaxCalculator(new NotRichTaxCalculator(new TaxCalculatorBase()));
         }else if(taxableIncome > NOTRICH)
         {
             taxCalculator = new NotRichTaxCalculator(new TaxCalculatorBase());
         }
         else
         {
             taxCalculator =new TaxCalculatorBase();
         }

        return taxCalculator;
    }
}

クライアントコードでは、これを書くだけです。

TaxCalculator taxCalculator= TaxCalculatorFactory.create(tax);
Double acumulatedTaxes = taxCalculator.calculate(tax);
于 2013-06-08T01:53:11.290 に答える
6

税額区分と税率が 1:1 であるという事実を考慮してください。

final double[][] BRACKETS = {
    {388350.0, 0.35},
    {178650.0, 0.33},
    {85650.0, 0.28},
    {35350.0, 0.25},
    {8700.0, 0.15}
};

/* ... */

for (double[] bracket: BRACKETS) {
    if(taxableIncome > bracket[0]) {
        cumulatedTax += ((taxableIncome-bracket[0]) * bracket[1]);
        taxableIncome = (long)bracket[0];
    }
}
cumulatedTax += taxableIncome * 0.1;
taxableIncome = 0;

C スタイルの ALLCAPSCONSTANTS が本当に気に入った場合は、自由に宣言して、 my で使用したリテラルの代わりにそれらの名前を使用してdouble[][]ください。Java をネイティブに使用する場合は、TaxBracket クラスを定義します。:)

編集:コードの意図を誤解しました。私の編集はあなたが望むことをするべきだと思います。

于 2013-06-08T01:14:36.297 に答える
3

上記のアプローチは気に入っていますが、agarrett の例の代わりに EnumMap を使用して調査することをお勧めします。double[][]

于 2013-06-08T01:28:52.580 に答える
2

これはどう？

List<Integer> taxSlabs = new ArrayList<Integer>();
taxSlabs.put(388350); 
taxSlabs.put(178650); 
taxSlabs.put(85650); 
taxSlabs.put(35350); 
taxSlabs.put(8700); 

List<Double> taxRates = new ArrayList<Double>();
taxRates.put(0.35); 
taxRates.put(0.33); 
taxRates.put(0.28); 
taxRates.put(0.25); 
taxRates.put(0.15);


//Variables for taxable income, and tax calculation.
long taxableIncome = income - deduction -(numberOfExemption * VAlUEOFEXEMPTION);
double cumulatedTax = 0.0; 


for(int indx = 0; indx < taxSlabs.size(); indx++){
    int slabLimit = taxSlabs.get(indx).intValue(); 
    if(taxableIncome >=  slabLimit ){
        cumulatedTax += ((taxableIncome% slabLimit) * taxRates.get(indx).doubleValue());
        taxableIncome = (long)slabLimit;
    }
}
于 2013-06-08T01:25:06.327 に答える