0

Доброго времени суток.

Задача: имеется следующий класс

public class Products {

public final Product PRODUCT;
public final double AMOUNT;
public final BigDecimal PRICE;
public final BigDecimal CALORIES;

    public Products(Product product, 
                    double amount, 
                    BigDecimal price, 
                    BigDecimal calories) {
        PRODUCT = product;
        AMOUNT = amount;
        PRICE = price;
        CALORIES = calories;
    }
}

Необходимо переопределить для данного класса метод equals() и hashCode(). Все поля должны участвовать в переопределяемых методах. И самое главное - при сравнении полей PRICE и CALORIES необходимо сравнивать только значения без учета масштаба, т.е. при сравнении двух BigDecimal со значениями 12 и 12.000000 должно возвращаться true.


Как я пытался решить задачу:

@Override
public boolean equals(Object o) {
    if (this == o) return true;
    if (o == null || getClass() != o.getClass()) return false;
    Products that = (Products) o;
    return Double.compare(that.AMOUNT, AMOUNT) == 0 &&
            Objects.equals(PRODUCT, that.PRODUCT) &&
            PRICE.compareTo(that.PRICE) == 0 &&
            CALORIES.compareTo(that.CALORIES) == 0;
}

@Override
public int hashCode() {
    int result = 17;
    result = result * 31 + PRODUCT.hashCode();
    result = result * 31 + Double.hashCode(AMOUNT);
    result = result * 31 + PRICE.hashCode();
    result = result * 31 + CALORIES.hashCode();
    return result;
}




В чем проблема: Метод equals() сравнивает BigDecimal без учета масштаба, а hashCode() использует BigDecimal::hashCode(), который вычисляет значения с учетом масштаба.


Вопрос: как рассчитать hashCode для BigDecimal без учета масштаба?

0

2 ответа 2

1

Я бы пошел от обратного. В реальной жизни цена у вас не может быть с произвольным количеством знаков после запятой. То же самое с калорийностью и количеством (количество я бы тоже хранил в BigDecimal).

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

public class Products {

    public final Product PRODUCT;
    public final BigDecimal AMOUNT;
    public final BigDecimal PRICE;
    public final BigDecimal CALORIES;

    public Products(Product product, 
                    BigDecimal amount, 
                    BigDecimal price, 
                    BigDecimal calories) {
        PRODUCT = product;
        AMOUNT = amount.setScale(3, BigDecimal.ROUND_HALF_UP);  // до грамм
        PRICE = price.setScale(2, BigDecimal.ROUND_HALF_UP);  // до копеек
        CALORIES = calories.setScale(4, BigDecimal.ROUND_HALF_UP);  // до 1/10000
    }
}

Методы equals и hashCode не меняются.

0

можно взять doubleValue у BigDecimal и затем уже брать hash

@Override
public int hashCode() {
    int result = 17;
    result = result * 31 + PRODUCT.hashCode();
    result = result * 31 + Double.hashCode(AMOUNT);
    result = result * 31 + PRICE.hashCode();
    result = result * 31 + Double.hashCode(CALORIES.doubleValue())
    return result;
}

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.