Repository navigation
КОНСОЛЬНОЕ ПРИЛОЖЕНИЕ 1 - #75
AlekseyChist wants to merge 2 commits into
Conversation
| System.out.println("Некорректное значение для подсчёта."); | ||
| } | ||
| add.getNumberOfPpl(numberOfPeople);//добавляем кол-во людей | ||
| } while (numberOfPeople < 2); |
There was a problem hiding this comment.
Имеет смысл выделить такие логические части программы (обработка ввода количества людей, ввод товаров) в отдельные методы, чтобы было проще их воспринимать, и здесь их последовательно вызывать
| add.count(); | ||
| add.nameOfCurrency(); | ||
| } | ||
| private static void typeGoods() { |
There was a problem hiding this comment.
Для простого вывода строки можно не создавать отдельный метод
| @@ -0,0 +1,58 @@ | |||
| public class Calculator { | |||
|
|
|||
| private Product[] array = new Product[20]; | |||
There was a problem hiding this comment.
Лучше использовать список из коллекций - например, ArrayList, потому что размер массива ограничен, а теоретически количество товаров неограниченное
| System.out.println("\nСумма для каждого составляет: " + sumForEach + " рубля."); | ||
| } else if (rounded == 0) | ||
| System.out.println("\nСумма для каждого составляет: " + sumForEach + " рублей."); | ||
| } |
There was a problem hiding this comment.
Есть принцип, который называется DRY - Don't repeat yourself. По этому правилу, если видишь, что есть многократно повторяющиеся участки кода, то их можно объединить. Здесь, например, изменяется лишь окончание, поэтому можно сделать строку, которая будет изменяться в зависимости от rounded, и затем присоединять её к предложению
|
Простите я сильно болел гриппом, вот поправился, внес правки. |
2 participants
No description provided.