Skip to content

consoleProject №1_ZaharinVN - #34

Closed
ZaharinVN wants to merge 7 commits into
Yandex-Practicum:mainfrom
ZaharinVN:dev
Closed

ZaharinVN wants to merge 7 commits into
Yandex-Practicum:mainfrom
ZaharinVN:dev

Conversation

@ZaharinVN

Copy link
Copy Markdown

No description provided.

Comment thread src/main/java/Main.java Outdated
Scanner sc = new Scanner(System.in);
while (true) {
System.out.println("На какое количество человек хотите разделить счёт? (цифрой)");
guestsNum = sc.nextInt();

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Если ввести значение не типа int в консоль, данный метод выбросит исключение. У Scanner есть удобный метод hasNextInt, можно сразу узнать без дополнительных проверок возможно ли интерпретировать строку как int

Comment thread src/main/java/Main.java Outdated
allDishName += dishName + "\n";
}
System.out.println("Введите стоимость блюда цифрами");
dishPrice = sc.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.

Тут тоже самое только hasNextDouble

Comment thread src/main/java/Price.java Outdated
@@ -0,0 +1,2 @@
public class Price {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Так не прокатит) Код нужно логически поделить на части (классы)

Comment thread src/main/java/Main.java Outdated
String str2 = "Количество гостей - " + guestsNum;
System.out.println(str2 + " чел.");
String str3 ="Сумма для каждого гостя: " + allDishPrice/guestsNum;
System.out.println(str3 + " рублей");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Некорректно рассчитывается окончание. Правильный алгоритм звучит так - Проверить лежит ли остаток от деления частного на 100 в интервале от 11 до 14 включительно, если да, то возвращаем рублей, если нет, то берем остаток от деления на 10 и проверяем - если 1, то рубль, 2-4 то рубля, в остальных случаях - рублей. То есть нужно во-первых проверять именно остаток от деления, во-вторых учесть, что числа заканчивающиеся на 11-12-13-14 и заканчивающиеся на 1-2-3-4, будут иметь разные окончания

@ZaharinVN

ZaharinVN commented Feb 12, 2023 via email

Copy link
Copy Markdown
Author

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.

3 participants