Repository navigation
Add dev branch - #1
NickFilimonov wants to merge 3 commits into
Conversation
| подсчитывает сумму цен товаров, делит на количество человек | ||
| */ | ||
|
|
||
| public static void CalcBill() { |
There was a problem hiding this comment.
В Java принята конвенция, согласно которой методы лучше называть camelCase'ом, т.е. с маленькой буквы, см. https://www.oracle.com/java/technologies/javase/codeconventions-namingconventions.html
| double productPrise; | ||
| double sum = 0.00; // переменная, которая будет хранить сумму цен товаров | ||
|
|
||
| while (userInput) { |
There was a problem hiding this comment.
Лучше вместо этого сделать условие while(true) и при необходимости выходить из цикла при помощи break или continue (по ситуации)
| передаем в метод переменную everyoneHasToPay - сколько должен оплатить каждый | ||
| определяем целочисленный остаток от деления на 10 и подбираем правильный падеж | ||
| */ | ||
| public static String GetAddition(double everyoneHasToPay) { |
| int preLastDigit = INTeveryoneHasToPay % 100 / 10; | ||
| if (preLastDigit == 1) | ||
| { | ||
| return word = "рублей"; |
There was a problem hiding this comment.
Можно не объявлять переменную word, а просто return "рублей";
| System.out.println("Введено некорректное значение. Попробуйте снова."); | ||
| amountPerson = scanner.nextInt(); | ||
| scanner.nextLine(); // без этой строки программа ломается, насколько я понял из-за нажатия на Enter при тестировании | ||
| } |
There was a problem hiding this comment.
Такие отдельные логические блоки, как этот (ввод и обработка количества людей), а также логический блоки с вводом товаров лучше вынести в отдельные методы и затем вызывать их там, где необходимо. Это позволяет улучшить читаемость кода, т.к. разнородная логика не смешивается в один сплошной код, а также упрощает поддержку кода.
2 participants
Спринт №2. Проектная работа 1. Bill calculator.