Skip to content

Проектная работа №1 - #1

Open
cerj275 wants to merge 4 commits into
mainfrom
dev
Open

cerj275 wants to merge 4 commits into
mainfrom
dev

Conversation

@cerj275

@cerj275 cerj275 commented Mar 8, 2023

Copy link
Copy Markdown
Owner

No description provided.

Comment thread src/main/java/Calculator.java Outdated
if (productPrice < 0) {
System.out.println("Введите положительное число");
} else {
productsList += "\n" + productName + " стоимостью " + productPrice + "р";

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⏫ Так как String является неизменяемым объектом, то при каждом сложении строк создаются новые объекты, что расходует память. Для таких случаев лучше использовать StringBuilder

Comment thread src/main/java/Formatter.java Outdated
public void reform(double sum) {
String caseOfRub = "";

if (sum == 1 || sum % 10 == 1) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⏫ Дробные числа не рекомендуется сравнивать обычным сравнением, так как эти числа имеют погрешность. Рекомендуется использовать специальные методы у соответствующих типов для double это Double.compare(d1, d2). Возвращает 0, если числа равны, < 0, если первое меньше второго, и > 0, если первое больше второго.

Comment thread src/main/java/Formatter.java Outdated

if (sum == 1 || sum % 10 == 1) {
caseOfRub = "рубль";
} else if (sum > 1 && sum < 5 || sum % 10 > 1 && sum % 10 < 5) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Есть специфичный случай, когда число оканчивается на 11-14. Тогда должно быть "рублей", а сейчас получается либо "рубль", либо "рубля" в зависимости от числа.

⏫ И я бы вынес в переменную это вычислениеsum % 10 и использовал бы уже ее в првоерки условий, чтобы каждый раз не писать полное вычисление

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants