Skip to content

Приложение номер 1. Сдача ДЗ № 1 - #1

Open
nataliakozhanova wants to merge 3 commits into
mainfrom
dev
Open

nataliakozhanova wants to merge 3 commits into
mainfrom
dev

Conversation

@nataliakozhanova

Copy link
Copy Markdown
Owner

No description provided.

Comment thread src/main/java/Main.java
} else {
System.out.println("Ошибка ввода. Введите целое число.");
scanner.nextLine();
}

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
} else if ((1 < round && round < 5) || (round > 21 && ((round % 10 == 2 || round % 10 == 3 || round % 10 == 4) && (round % 100 != 12 && round % 100 != 13 && round % 100 != 14)))){
System.out.println("Сумма к оплате (на каждого человека): " + sumToPrint + " рубля.");
} else {
System.out.println("Сумма к оплате (на каждого человека): " + sumToPrint + " рублей.");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

System.out.println("Сумма к оплате (на каждого человека): " + sumToPrint повторяется три раза, можно его вызвать один раз в конце и прибавить окончание в зависимости от числа

Comment thread src/main/java/Main.java Outdated
public static class Calculator { // расчет суммы для оплаты
public static double countSumToPay(int number, double sum) {
double sumToPay = sum / number;
return sumToPay;

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 sum / number;

Comment thread src/main/java/Main.java Outdated
int userInputNumber = 0;
boolean valid = false;

while (!valid) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Это условие здесь не нужно. Можно просто while (true)

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