Repository navigation
Проектная работа 1 - #1
KozlovskiyD wants to merge 3 commits into
Conversation
| } | ||
| } else { | ||
| System.out.println("Неверное количество друзей. Значение должно быть болье единицы, давайте попробуем еще раз."); | ||
| System.out.println("Введено некоректное значение.\nВедите новое значени:"); |
There was a problem hiding this comment.
Такие отдельные логические блоки, как этот (ввод и обработка количества людей), а также логические блоки с вводом товаров и выводом результатов лучше выносить в отдельные методы и затем вызывать их. Это позволяет улучшить читаемость кода.
| System.out.println("Введите стоимость товара:"); | ||
| calcut.zapTovar(newTovar); | ||
| System.out.println("Товар успешно добавлен\nДобавить еще товар или завершить?"); | ||
| System.out.println("Для добавления ещё товара введите любое значени\nЕсли все товары добавлены введите 'завершить'"); |
There was a problem hiding this comment.
Такие подряд идущие println можно объединить в один. Некритично, но небольшая экономия производительности - один вызов метода дешевле, чем два.
|
|
||
| System.out.println(item.name + " в корзине"); | ||
| public class Calculator { | ||
| String nameTovar; |
| System.out.println("Введено некоректное значени соимости\nВведите новое значение"); | ||
| } | ||
| } | ||
| summa = summa + cena; |
There was a problem hiding this comment.
Можно также использовать сокращенную конструкцию summa += cena;
| double divideSum() { | ||
| return totalPrice / friendsCount; | ||
| String rub; | ||
| float sumNaTovar; |
| String rub; | ||
| float sumNaTovar; | ||
| int ost; | ||
| void padeji (float sumNaTovar) { |
There was a problem hiding this comment.
В Java принята конвенция, согласно которой методы лучше называть глаголами
См. https://www.oracle.com/java/technologies/javase/codeconventions-namingconventions.html
3 participants
No description provided.