Repository navigation
Conversation
| public String getItemsAsStr(){ | ||
| String info = "Добавленные товары: \n"; | ||
| for (Item item: items){ | ||
| info = info + item.toString() + "\n"; |
There was a problem hiding this comment.
Можно использовать конструкцию info += item.toString() + "\n";
| import java.lang.Math; | ||
| public class App { | ||
| Scanner sc = new Scanner(System.in); | ||
| Calculator calculator = new Calculator(); |
There was a problem hiding this comment.
Так как калькулятор один на всю программу, можно сделать его, его методы и поля static,чтобы не нужно было создавать его экземпляр. Тогда можно будет просто вызывать методы напрямую: Calculator:.getSum()
| } | ||
| else { | ||
| template = "%.2f рублей"; | ||
| } |
There was a problem hiding this comment.
Для множественных вариантов лучше использовать конструкцию switch
| int peopleCount = sc.nextInt(); | ||
| if (peopleCount <= 1) { | ||
| System.out.println("Колличество человек должно быть больше 1"); | ||
| continue; |
There was a problem hiding this comment.
continue завершает итерацию цикла и начинает заново новый этап цикла. Нет смысла ставить continue в конце блока кода - даже без него итерация кода бы закончилась и начался бы новый цикл. IDE выделяет continue и подсказывает, как отрефакторить код
| Item item = new Item(title,price); | ||
| calculator.addItem(item); | ||
| System.out.println("Товар успешно добавлен"); | ||
| System.out.println("Если это все продукты, то введите 'exit'"); |
There was a problem hiding this comment.
Можно объединить два println в один. Перенос строки делается символом \n
| break; | ||
| } | ||
| else { | ||
| continue; |
There was a problem hiding this comment.
тоже можно обойтись без continue, потому даже без него итерация завершается в этом моменте. IDE подсвечивает continue и предлагает отрефакторить
3 participants
No description provided.