Точность с плавающей точкой с Math.Ceil (Java)

Question

Точность с плавающей точкой с Math.Ceil (Java)

1

Я использую бизнес-правило для расчета процентного прироста на уровне запасов:

Stock level | Percentage Increase | Expected output
    100     |           93        |        193

Поскольку уровни десятичного запаса не определены правильно, правило состоит в округлении вывода:

public int calculateStockLevel(int rawStockLevel, double percentageIncrease) {
    return Math.ceil(rawStockLevel * (1 + (percentageIncrease / 100));
}

Фактический выход для ситуации выше: 194 (Тест не прошел)

Это похоже на ошибку точности с плавающей запятой, какой элегантный и читаемый способ пройти этот тест?

Edd 10 фев. 2015, в 10:44

Источник

3

Использование плавающих точек с денежными данными не очень хорошая идея, именно из-за проблем с точностью; вы хотите использовать BigDecimal ; также есть Joda Money (о последнем, я знаю, что он существует, я никогда не использовал его на самом деле)
fge 10 фев. 2015, в 09:46
0

почему ты делаешь 1+ ??
Prashant 10 фев. 2015, в 09:49
1

@Prashant Это процентное увеличение, так что при процентном увеличении на 93% вы эффективно умножаетесь на 193%.
Edd 10 фев. 2015, в 09:59
0

@Prashant То же самое, что rawStockLevel + rawStockLevel * (increase /100)
1Darco1 10 фев. 2015, в 10:02
0

@fge Это не деньги. Это единицы чего-то. Можно ожидать, что все проблемы с двоичной плавающей точкой будут возникать с десятичной плавающей точкой.
Pascal Cuoq 10 фев. 2015, в 10:05
0

если я делаю Math.ceil(100 * (100+93.0) / 100) its giving 193.0
Prashant 10 фев. 2015, в 10:22
0

в основном из-за 1 + (93.0 / 100) только вывод получает большую точность, что приводит к 1.9300000000000002, поэтому ceil () дает 194.0
Prashant 10 фев. 2015, в 10:45
0

@fge Это не деньги, продукты на складе представляют собой количество рекламы, которую мы предлагаем компании на нашем сайте. BigDecimal решает проблему, но это не очень приятное решение. Это пока подойдет :)
Edd 10 фев. 2015, в 10:50
0

return Math.ceil(rawStockLevel * (100 + percentageIncrease) / 100);
phuclv 10 фев. 2015, в 12:28
0

@LưuVĩnhPhúc LưuVĩnhPhúc Это работает, но почему?
Edd 10 фев. 2015, в 14:56
0

Это просто более рекомендуемый способ, который не всегда дает правильный результат, потому что двоичные типы с плавающей точкой просто не могут представлять все десятичные рациональные значения.
phuclv 10 фев. 2015, в 15:05
1

Нет, ответ LuuVinhPhuc работает (если только ваши числа не являются действительно большими), потому что вы работаете с целочисленными значениями до окончательного деления на 100. Таким образом, до деления округления нет. Деление правильно округлено, что подразумевает, в частности, что вы не попадете на неправильную сторону целого числа из-за округления, полученного из деления.
tmyklebu 10 фев. 2015, в 15:55

Показать ещё 10 комментариев

Теги:

java

floating-point

2 ответа

0

Двоичные типы с плавающей запятой не могут представлять каждое значение в десятичном значении. Таким образом, percentageIncrease/100 получит самое близкое представление в двоичном формате. В вашем случае точное значение 0,93 с двойной точностью составляет 0,930000000000000048849813083507, что немного больше, чем истинное десятичное значение. Поэтому после ceil он будет округлен до 0,94

Если вам нужен точный десятичный вывод, вы должны использовать десятичную арифметику, например BigDecimal. Но в вашем случае вы можете просто сделать это в виде простого целого числа, используя ract, чтобы дробная часть результата была больше 0, если остаток от деления не равен нулю.

int temp   = rawStockLevel * (100 + percentageIncrease);
int result = temp/100;
if (temp % 100 != 0)
    result++; // round up if not divisible by 100
return result;

phuclv 10 фев. 2015, в 12:34

Ещё вопросы

Использование плавающих точек с денежными данными не очень хорошая идея, именно из-за проблем с точностью; вы хотите использовать BigDecimal ; также есть Joda Money (о последнем, я знаю, что он существует, я никогда не использовал его на самом деле)
@Prashant Это процентное увеличение, так что при процентном увеличении на 93% вы эффективно умножаетесь на 193%.
@Prashant То же самое, что rawStockLevel + rawStockLevel * (increase /100)
@fge Это не деньги. Это единицы чего-то. Можно ожидать, что все проблемы с двоичной плавающей точкой будут возникать с десятичной плавающей точкой.
если я делаю Math.ceil(100 * (100+93.0) / 100) its giving 193.0
в основном из-за 1 + (93.0 / 100) только вывод получает большую точность, что приводит к 1.9300000000000002, поэтому ceil () дает 194.0
@fge Это не деньги, продукты на складе представляют собой количество рекламы, которую мы предлагаем компании на нашем сайте. BigDecimal решает проблему, но это не очень приятное решение. Это пока подойдет :)
return Math.ceil(rawStockLevel * (100 + percentageIncrease) / 100);
@LưuVĩnhPhúc LưuVĩnhPhúc Это работает, но почему?
Это просто более рекомендуемый способ, который не всегда дает правильный результат, потому что двоичные типы с плавающей точкой просто не могут представлять все десятичные рациональные значения.
Нет, ответ LuuVinhPhuc работает (если только ваши числа не являются действительно большими), потому что вы работаете с целочисленными значениями до окончательного деления на 100. Таким образом, до деления округления нет. Деление правильно округлено, что подразумевает, в частности, что вы не попадете на неправильную сторону целого числа из-за округления, полученного из деления.

Andrei Vajna II · Accepted Answer · 2015-02-10T10-10-00.000Z

2

Лучший ответ

Вы должны использовать BigDecimal, чтобы указать, какую точность вы хотите, но для вашего случая вы могли бы сделать что-то более простое: просто разделите на 100 в конце.

public int calculateStockLevel(int rawStockLevel, double percentageIncrease) {
    return Math.ceil(rawStockLevel * (100 + percentageIncrease) / 100);
}

Andrei Vajna II 10 фев. 2015, в 10:10

0

Я попробовал и проверил это с целочисленными процентами, и это, кажется, работает, но почему это отличается от моего решения в вопросе? Это математически идентично ....
Edd 10 фев. 2015, в 14:37
0

@ Ed0906 Было бы математически идентично, если бы арифметика с плавающей запятой была арифметикой действительных чисел или даже рациональной арифметикой. Это другая арифметика со своими собственными правилами, предназначенными для ограничения результатов до конечного набора значений двоичной дроби путем округления. Поскольку результат с плавающей запятой может быть немного больше или чуть меньше результата арифметики с рациональным числом, такие операции, как потолок и пол, могут получить другой ответ.
Patricia Shanahan 10 фев. 2015, в 15:12
0

Из-за ограничений с плавающей запятой. Он не может представлять десятичные числа с точностью 100%, даже для, казалось бы, простых случаев, таких как 1 + 0,93. (что даст 1.9300000000000002) Однако, оно может представлять целые числа со 100% точностью. (если они не слишком велики, что здесь не так) И поскольку во втором случае нам нужно иметь дело только с целыми числами, результат будет точным.
Andrei Vajna II 10 фев. 2015, в 15:22

Показать ещё 1 комментарий