Conversation
| while (true) { | ||
|
|
||
| System.out.println("Введите название товара."); | ||
| String name = sc.next(); // ? |
There was a problem hiding this comment.
Используя функцию scanner.nextLine() вместо scanner.next(), ты сможешь считывать не только одно слово, но и целые предложения в названии продукта.
There was a problem hiding this comment.
Изначально именно ее и использовал, если здесь заменить на .nextLine() строка перестанет считываться, не смог понять из-за чего это происходит. При этом с .next() все работает как надо.
|
|
||
| // Добавление товаров в список | ||
| static void goodsInput() { | ||
| Scanner sc = new Scanner(System.in); |
There was a problem hiding this comment.
Всегда старайся выбирать понятные названия для переменных и констант. Хорошие названия выполняют роль документации и значительно облегчают процесс чтения кода.
| System.out.println("==================="); | ||
| } | ||
|
|
||
| // Расчет итоговой суммы |
There was a problem hiding this comment.
закомментированный код стоит удалить, т.к визуально загрязняется код
|
|
||
| // Обрабатываем случай, если вводят слово ЗАВЕРШИТЬ. | ||
| static boolean end() { | ||
| Scanner sc = new Scanner(System.in); |
There was a problem hiding this comment.
после использования сканера хорошей практикой будет закрыть его с помощью функции scanner.close(), это позволит избежать возможных утечек памяти
There was a problem hiding this comment.
Да, это я знаю и в некоторых местах можно найти применения метода .close(). В тех местах, где его в конечном итоге не поставил, IDE подчеркивала строчку "sc.close()" красным. Если есть объяснения для такого явления, буду рад их услышать.
| } | ||
|
|
||
| // Расчет итоговой суммы | ||
| static void totalAmount(int people) { |
There was a problem hiding this comment.
Имена функций должны содержать в себе действие. Функция делает что-либо. Хорошо помогает практика переводить имена функций дословно, если после перевода понятно чем занимается функция, то название хорошее
Нулевой трай.