Repository navigation
Сдача задания - #63
Artem-Grigorash wants to merge 15 commits into
Conversation
| int persons = in.nextInt(); | ||
| while (persons<=1){ | ||
| out.println("Non-correct value for counting, the number of people must be more than one"); | ||
| out.println("Please enter the correct value"); |
There was a problem hiding this comment.
Можно объединить два println в один. Перенос строки можно сделать, вставив в строку "\n"
| out.println("Please enter the correct value"); | ||
| persons = in.nextInt(); | ||
| } | ||
| Positions[] pos = new Positions[100]; |
There was a problem hiding this comment.
Лучше будет использовать не массив, а ArrayList. В большинстве случаев у нас действительно будет не более 100 товаров, но наверняка мы знать не можем.
| out.println("Great! See you later"); | ||
| } | ||
| else | ||
| out.println("Please, enter Yes/No"); |
There was a problem hiding this comment.
В комментарии к заданию я не написал ещё один важный момент, который стоит поправить - сейчас практически весь код написан в методе main класса Main. Это сильно перегружает код, так как разная логика (ввод людей, ввод продуктов, конечный вывод) написаны в одном методе.
Я рекомендую разбить программу на отдельные классы с отдельными методами. Например, можно создать отдельные классы/методы, отвечающие за обработку ввода количества людей, обработку ввода продуктов, конечный вывод, удаление продуктов.
Вообще способов организовать код бесконечность, это будет приходить с опытом. На эту тему могу посоветовать книгу Роберта Мартина "Чистый Код".
| sum+=pos[j].price; | ||
| out.println("Total sum: " + String.format("%.2f", sum) + " rubles"); | ||
| out.println(("Everyone has to pay: ") + String.format("%.2f", sum/persons)+ " rubles"); | ||
| out.println("Do you want to correct the list?"); |
There was a problem hiding this comment.
Также можно объединить эти println в один.
| String nextPosition = in.next(); | ||
| out.println("Enter the price of the dish in the format rub,kop"); | ||
| double nextPrice = in.nextDouble(); | ||
| int i = 0; |
There was a problem hiding this comment.
Лучше использовать такой нейминг, чтобы по названию было понятно, что переменная означает.
2 participants
No description provided.