Проектная работа №1 попытка №1 Чурилов Алексей.#60
Проектная работа №1 попытка №1 Чурилов Алексей.#60bigcapus wants to merge 3 commits intoYandex-Practicum:masterfrom
Conversation
| // вы не должны ограничиваться только классом Main и можете создавать свои классы по необходимости | ||
| System.out.println("Привет Мир"); | ||
| Scanner scanner = new Scanner(System.in); | ||
| int person = 0; |
There was a problem hiding this comment.
Поскольку по умолчанию неинициализированная переменная типа int будет равна 0, можно не писать = 0
| Calculator calculator = new Calculator(); | ||
| float price = 0; | ||
| String product = ""; | ||
| String command = ""; |
There was a problem hiding this comment.
Здесь тоже все переменные такими и будут по умолчанию, поэтому этими значениями можно не инициализировать
| } | ||
| } | ||
| System.out.println("Итоговый счет:\n" + calculator.items); | ||
| int list = (int) Math.floor(calculator.sum / (double) person); |
There was a problem hiding this comment.
Лучше назвать переменную более очевидно
| System.out.println("Итоговый счет:\n" + calculator.items); | ||
| int list = (int) Math.floor(calculator.sum / (double) person); | ||
| if (list == 1) { | ||
| System.out.println(String.format("Сумма на одного человека:%.2f рубль", calculator.sum / (double) person)); |
There was a problem hiding this comment.
Код calculator.sum / (double) person повторяется много раз, лучше посчитать один раз и использовать переменную
| if (list == 1) { | ||
| System.out.println(String.format("Сумма на одного человека:%.2f рубль", calculator.sum / (double) person)); | ||
| } else if (2 <= list && list <= 4) { | ||
| System.out.println(String.format("Сумма на одного человека:%.2f рубля", calculator.sum / (double) person)); |
There was a problem hiding this comment.
Вся строчка System.out.println(String.format("Сумма на одного человека:%.2f рубля", calculator.sum / (double) person));, за исключением самого слова "рубля" повторяется много раз, лучше вынести её за пределы if-else, а в нем определять только само слово
| } else if (5 <= list && list <= 20) { | ||
| System.out.println(String.format("Сумма на одного человека:%.2f рублей", calculator.sum / (double) person)); | ||
| } else { | ||
| int digit = list % 10; |
There was a problem hiding this comment.
Можно было бы оставить только switch, без верхнего if-else. Для случаев, когда list <= 10 и когда list >= 20, нам нужна только последняя цифра для определения падежа, для этого можно использовать digit = list % 10 (если будет меньше 10, результат будет равен самому list), и в одном switch проверять цифру. А до него ещё с помощью одного if проверять только, что list % 100 больше 10 и меньше 20
| case 8: | ||
| case 7: | ||
| case 5: | ||
| System.out.println(String.format("Сумма на одного человека:%.2f рублей", calculator.sum / (double) person)); |
There was a problem hiding this comment.
Поскольку здесь много case, можно их объединить, используя default в конце
Спринт 2 , приложение "Калькулятор".