Skip to content

Консольное приложение №1#1

Open
ValentinPside wants to merge 9 commits into
devfrom
main
Open

Консольное приложение №1#1
ValentinPside wants to merge 9 commits into
devfrom
main

Conversation

@ValentinPside
Copy link
Copy Markdown
Owner

Вопрос ао ПР: при пуше изменения идут в ветку main, а не в dev. Как это исправить?

Comment thread src/main/java/Main.java
System.out.println("Счёт будет разделён на " + personCount + " персоны");
return personCount;
}
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread src/main/java/Dish.java
if(!line.equals(line.replaceAll("[^0-9.]", ""))){
System.out.println("Нужно число без букв. Попробуйте снова");
continue;
}
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(ending.equals("5") || ending.equals("6") || ending.equals("7") || ending.equals("8") || ending.equals("9") ||ending.equals("0")){
ending = z;
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Эти проверки дублируются тут и в totalPriceForPerson

}
if(ending.equals("5") || ending.equals("6") || ending.equals("7") || ending.equals("8") || ending.equals("9") ||ending.equals("0")){
ending = z;
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

ending = x;
}
if(ending.equals("2") || ending.equals("3") || ending.equals("4")){
ending = x;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Тут опечатался, должно быть y

@gusar-off
Copy link
Copy Markdown

Вопрос ао ПР: при пуше изменения идут в ветку main, а не в dev. Как это исправить?

Это значит, что ты находишься в ветке main, чтобы сменить ветку у консольного приложения git есть команда checkout
git checkout dev сменит твою текущую ветку и позволит пушить в дев.

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