Repository navigation
Conversation
| } | ||
|
|
||
|
|
||
| System.out.println("Сумма на каждого: " + payment + " " + rub + "\n****"); |
There was a problem hiding this comment.
Такие отдельные логические блоки, как этот (обработка окончаний), а также логические блоки с с вводом людей, вводом товаров и выводом результатов лучше выносить в отдельные сущности (методы и классы) и затем вызывать их там, где необходимо. Это позволяет улучшить читаемость кода, т.к. разнородная логика не смешивается в один сплошной код, а также упрощает поддержку кода.
| на скольких человек необходимо разделить счёт. | ||
| */ | ||
|
|
||
| int k; |
There was a problem hiding this comment.
Лучше делать наименования более содержательными, эта тема хорошо раскрывается во 2 главе книги Роберта Мартина "Чистый код". Также на хабре есть краткий конспект книги: https://habr.com/ru/post/485118/
| pr = scanner.nextDouble(); | ||
| while (pr < 0) { | ||
| System.out.println("!! Цена должна быть положительным числом. Укажите новую цену."); | ||
| pr = 0.0; |
There was a problem hiding this comment.
pr = 0.0 - лишнее. В следующей строчке переприсваиваем
| pr = 0.0; | ||
| pr = scanner.nextDouble(); | ||
| } | ||
| sum = sum + pr; |
There was a problem hiding this comment.
Есть короткая конструкция sum += pr;
| } else { | ||
| System.out.println("!! Укажите цену цифрами, для разделения используй запятую.\nНазвание товара укажи заново.\n"); | ||
| } | ||
| Sku sku = new Sku(text, pr); |
There was a problem hiding this comment.
Переменная объявляется и не используется
|
|
||
|
|
||
| System.out.println("\n****\nСписок всех товаров:" + list.substring(0, list.length()) + "\n"); | ||
| System.out.println("Общая сумма: " + sum + " Количество человек: " + k + "\n"); |
There was a problem hiding this comment.
Такие подряд идущие println можно объединить в один.
| Sku sku = new Sku(text, pr); | ||
| } | ||
|
|
||
| if (list.equals("")) { |
2 participants
Консольное приложение