Счетчик долларов и центов

Question

Итак, я получил задание написать счетчик, который складывает определенное количество долларов и центов вместе.Нам дали тестовый класс, который используется для тестирования функциональности.

Нам дали следующие подсказки:

public int dollars () //The dollar count. 
ensure: this.dollars () >= 0

public int cents () //The cents count. 
ensure: 0 <= this.cents() && this.cents() <= 99

И:

public void add (int dollars, int cents) //Add the specified dollars and cents to this Counter. 
public void reset () //Reset this Counter to 0. 
ensure: this .dollars() == 0 && this.cents() == 0 

Это мойтекущий код:

public class Counter {

    private float count;

    public Counter() {
        count = 0;
    }

    public int dollars() {
        if (this.dollars() >= 0) {
            count = count + Float.parseFloat(this.dollars() + "." + 0);
        } return 0;
    }

    public int cents() {
        if (0 <= this.cents() && this.cents() <= 99) {
            count = count + Float.parseFloat(+0 + "." + this.cents());
        } else if (100 <= this.cents()) {
            count = count + Float.parseFloat(+1 + "." + (this.cents() - 100));
        }
        return 0;
    }

    public void add(int dollars, int cents) {
        dollars = this.dollars();
        cents = this.cents();
    }

    public void reset() {
        count = 0;  
    }

    /**
     * @param args
     */
    public static void main(String[] args) {
        // TODO Auto-generated method stub
    }
}

Я понимаю, что здесь я делаю ошибки (предположительно, на поплавке и пытаюсь вычислить долларовую и центовую часть поплавка).Но я не могу точно определить, где это не удается.

Answer 1

public final class Counter {

    private int cents;

    public int dollars() {
        return cents / 100;
    }

    public int cents() {
        return cents;    // if you want to retrun all centes
        // return cents % 100;    // if you want to return cents less than dollar
    }

    public void add(int dollars, int cents) {
        this.cents = dollars * 100 + cents;
    }

    public void reset() {
        cents = 0;
    }
}

Одно очень важное правило для финансового программирования: НИКОГДА НЕ ИСПОЛЬЗУЙТЕ FLOAT КАК ДЕНЬГИ СЧЕТЧИКА . У вас определенно есть проблемы с этим скоро или даже очень скоро. Посмотрите мой пример, на самом деле ваш счетчик может быть реализован очень просто, просто держа количество центов как int (как я вижу, у вас нет части центов).

P.S. Один трюк на будущее

Представьте, что вам нужны значения с плавающей запятой и поддержите все стандартные математические операции над ними, например +,-,/,*. Например. доллар и целое количество центов (как в вашем примере), и вы не можете (или не хотите) использовать плавающие операции. Что делать?

Просто зарезервируйте две младшие цифры в целочисленном значении в виде дробной части. Итак, давайте возьмем пример для цены $ 12:

int price = 1200;    // 00 is reserverd for centes, , price is $12
price += 600;        // add $6, price is $18
price += 44;         // add $0.44, price is $18.55

int dollars = price / 100;   // retrieve total dollars - $18     
int cents = cents % 100;     // retrieve cents less than dollars - 44  

Answer 2

Некоторые полезные ответы здесь.Я также понял, что неправильно понял задание (думая, что мне понадобится общая сумма денег вместо двух отдельных счетчиков для долларов и центов), и поэтому я сделал это сложнее, чем нужно.

Мое решение(который проверен для работы с использованием предоставленного тестового класса) выглядит следующим образом:

public final class Counter {

    private int dollars;
    private int cents;

    public int dollars() {

        return dollars;
    }

    public int cents() {
        return cents;
    }

    public void add(int dollars, int cents) {
        if (dollars >= 0) {
            if (0 <= cents && cents <= 99) {
                this.dollars += dollars;
                this.cents += cents;
            }
            if (this.cents > 100 | cents > 100) {
                this.dollars += this.cents / 100;
                this.cents = this.cents % 100;
            }
            if (this.cents == 100 | cents == 100) {
                this.dollars++;
                this.cents = this.cents - 100;
            }

        }
    }

    public void reset() {
        dollars = 0;
        cents = 0;
    }
}

Answer 3

public class Counter {

    private int dollars = 0;
    private int cents = 0;

    public Counter(int dollars, int cents) {
        this.dollars = dollars;
        this.cents = cents;
    }

    Counter reset() {
        return new Counter(0, 0);
    }

    Counter add(int dollars, int cents) {
        if (dollars < 0 || cents < 0) {
            throw new IllegalArgumentException();
        }

        int dollarsToAdd = this.dollars + dollars;
        int centsToAdd = this.cents + cents;

        if (centsToAdd > 99) {
            dollarsToAdd += centsToAdd / 100;
            centsToAdd = centsToAdd % 100;
        }
        return new Counter(dollarsToAdd, centsToAdd);
    }

    public void print() {
        System.out.println(this.dollars + "." + this.cents);
    }

}

1) Использование двух счетчиков вместо одного.Это облегчает вычисления и позволяет избежать проблем с плавающей точкой.(Проверьте это System.out.println(1.03 - .42);)

2) Проверка входов.Невозможно добавить отрицательные суммы.

3) центов / 100 вернет количество полных долларов.Поскольку деление int на int удаляет десятичные точки.

4) центы% 100 вернут остаток - центы, оставшиеся после конвертации в полные доллары.

5) давайте сделаем его неизменным.Это позволит избежать проблем параллелизма.