Пишем консольное приложение#118
Conversation
| if (Math.floor(price) >= 11 && Math.floor(price) <= 14) { | ||
| return roubleCase = " рублей"; | ||
| } else { | ||
| remainder = Math.floor(price) % 10; |
There was a problem hiding this comment.
Math.floor(price) считается 3 раза, можно посчитать 1 раз и записать в переменную
| @@ -0,0 +1,20 @@ | |||
| public class CaseEnding { | |||
|
|
|||
| static String roubleCase = " рублей"; | |||
There was a problem hiding this comment.
Переменную можно убрать, и ниже при возврате результата писать просто return " <нужное слово>";. Или оставить переменную, но убрать здесь инициализацию, в каждом if убрать return, оставить только присваивание переменной нужного слова, а после всех if-else поставить return roubleCase;
| static double remainder; | ||
|
|
||
| static String setEnding(double price) { | ||
| if (Math.floor(price) >= 11 && Math.floor(price) <= 14) { |
There was a problem hiding this comment.
Если здесь добавить деление по модулю на 100 Math.floor(price) % 100, то для 111, 211 и т.д. будет работать тоже
|
|
||
| public class Item { | ||
|
|
||
| static String name; |
There was a problem hiding this comment.
Все модификаторы static можно в этом классе убрать, а при вызове метода calculate в Main сначала создавать объект класса Item. Подробнее про static будет в следующих уроках, пока в нем сильной необходимости нет
| static String values = ""; | ||
| static double sum = 0; | ||
|
|
||
| static void calculate() { |
There was a problem hiding this comment.
Можно результатом этого метода возвращать посчитанную sum, тогда не надо будет её использовать в Main, код будет выглядеть логичнее
|
|
||
| while (true) { | ||
|
|
||
| printMenu(); |
There was a problem hiding this comment.
Можно этот вывод вынести до while, тогда он будет печататься один раз, в принципе этого достаточно для пользователя
2 participants
Добрый день! Задание по спринту 2.