Skip to content

CProject - #34

Open
KTDem wants to merge 4 commits into
Yandex-Practicum:masterfrom
KTDem:master
Open

KTDem wants to merge 4 commits into
Yandex-Practicum:masterfrom
KTDem:master

Conversation

@KTDem

@KTDem KTDem commented Sep 12, 2022

Copy link
Copy Markdown

Calculator

System.out.println("Ошибка, введите корректное число.");
}
while (true) {
double i = scanner.nextDouble();

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Лучше называть переменные так, чтобы при взгляде на них было сразу понятно, что они означают. Например, тут использовать имя price

break;
}
}
System.out.println("Успех! \nХотите ли добавить еще продукт?");

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 result = summary / guests;
String rubleTitle = rubleCase(result);
System.out.println(dishes);
System.out.println(String.format("%.2f", result) + " " + rubleTitle);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Можно добавить больше информации в вывод, например "Каждый человек должен заплатить ..."

return "рубля";
}
else {
return "рублей ";

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Этот if-else можно переписать на switch

Comment thread src/main/java/Main.java
// ваш код начнется здесь
// вы не должны ограничиваться только классом Main и можете создавать свои классы по необходимости
System.out.println("Привет Мир");
int i;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Лучше называть переменные так, чтобы при взгляде на них было сразу понятно, что они означают. Например, тут использовать имя person

Comment thread src/main/java/Main.java
System.out.println("Введите корректное количество гостей.");
}

if (i > 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.

Это условие можно перенести в виде else к предыдущему if

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