Conversation
| Scanner scanner_humans = new Scanner(System.in); | ||
| Scanner scanner_products = new Scanner(System.in); | ||
| Scanner scanner_prices = new Scanner(System.in); | ||
|
|
There was a problem hiding this comment.
🍏 Для всего взаимодействия с пользователем достаточно открыть один сканнер
| float summ = 0; | ||
|
|
||
| // Сканеры | ||
| Scanner scanner_humans = new Scanner(System.in); |
There was a problem hiding this comment.
🍏 В Java принято переменные писать в camelCase, а названия классов CamelCase.
SNAKE_CASE используется для именования констант, а snake_case - для именования xml файлов.
https://www.oracle.com/java/technologies/javase/codeconventions-namingconventions.html
|
|
||
| // Получаем количество людей | ||
| while (true) { | ||
| if (scanner_humans.hasNextInt()) { |
There was a problem hiding this comment.
👏 Круто, что делаешь проверку на наличие числа
| float forPerson = summ / humans; | ||
| int rubles = (int) forPerson; | ||
|
|
||
| System.out.println("Сумма на каждого человека " + String.format("%.2f", forPerson) + " " + grammar(rubles)); |
There was a problem hiding this comment.
🍏 При печати в памяти будут создаваться промежуточные строки из-за множественного использования оператора "+". Можно использовать StringBuilder, который сформирует только результирующую строку, или String.format()
|
🍏 Классно было бы подсказать пользователю, как выйти из сценария |
Изменение ревьюера Co-authored-by: kgornostaeva <[email protected]>
Изменения ревьюера Co-authored-by: kgornostaeva <[email protected]>
Hi, this is my first PR project