Проектная работа №1. Версия 2. #68
Conversation
|
|
||
| public static void rublesName(double cost) { | ||
| if (cost % 10 == 1 && cost != 11 && cost % 100 != 11) { | ||
| System.out.println(" рубль."); |
There was a problem hiding this comment.
Можно в этом методе не выводить на консоль слово, а возвращать только его
| public static void rublesName(double cost) { | ||
| if (cost % 10 == 1 && cost != 11 && cost % 100 != 11) { | ||
| System.out.println(" рубль."); | ||
| } else if (cost % 10 >= 2 && cost % 10 <= 4) { |
There was a problem hiding this comment.
cost % 10 повторяется несколько раз, можно посчитать 1 раз, записать в переменную и использовать её.
Ещё в эту ветку попадают числа 12, 13 и 14, а должно быть для них "рублей"
| } | ||
|
|
||
| public static void rublesName(double cost) { | ||
| if (cost % 10 == 1 && cost != 11 && cost % 100 != 11) { |
There was a problem hiding this comment.
cost != 11 и cost % 100 != 11 это одно и тоже, при делении по модулю числа 11 на 100 будет всё равно 11
There was a problem hiding this comment.
Это для чисел больше 100
There was a problem hiding this comment.
Можно оставить только cost % 100 != 11. Для числа 111 cost % 100 = 11, и для числа 11 cost % 100 = 11
| int passPeople = scannerPeople.nextInt(); | ||
| if (passPeople > 1) { | ||
| countPeople = passPeople; | ||
| return; // передает в переменную число людей |
There was a problem hiding this comment.
Можно число людей сделать возвращаемым значением метода howPeople, и избавиться от переменной countPeople
2 participants
Переписал код склонения рубля, сейчас работает с любыми суммами.