Repository navigation
Conversation
| if (productPrice < 0) { | ||
| System.out.println("Введите положительное число"); | ||
| } else { | ||
| productsList += "\n" + productName + " стоимостью " + productPrice + "р"; |
There was a problem hiding this comment.
⏫ Так как String является неизменяемым объектом, то при каждом сложении строк создаются новые объекты, что расходует память. Для таких случаев лучше использовать StringBuilder
| public void reform(double sum) { | ||
| String caseOfRub = ""; | ||
|
|
||
| if (sum == 1 || sum % 10 == 1) { |
There was a problem hiding this comment.
⏫ Дробные числа не рекомендуется сравнивать обычным сравнением, так как эти числа имеют погрешность. Рекомендуется использовать специальные методы у соответствующих типов для double это Double.compare(d1, d2). Возвращает 0, если числа равны, < 0, если первое меньше второго, и > 0, если первое больше второго.
|
|
||
| if (sum == 1 || sum % 10 == 1) { | ||
| caseOfRub = "рубль"; | ||
| } else if (sum > 1 && sum < 5 || sum % 10 > 1 && sum % 10 < 5) { |
There was a problem hiding this comment.
⏫ И я бы вынес в переменную это вычислениеsum % 10 и использовал бы уже ее в првоерки условий, чтобы каждый раз не писать полное вычисление
No description provided.