Repository navigation
Conversation
| class nameProd { | ||
| public static void nameProde(int countUser) { |
There was a problem hiding this comment.
🍏 Обычно принято именовать классы начиная с заглавной буквы, используя CamelCase стиль, т.е. лучше будет - NameProd
| if (num % 100 > 4 && num % 100 < 21) { | ||
| rubl = "рублей"; | ||
| } else if (num % 10 == 1) { | ||
| rubl = "рубль"; | ||
| } else if (num % 10 > 1 && num % 10 < 5) { | ||
| rubl = "рубля"; | ||
| } |
There was a problem hiding this comment.
🍏 Не критично конечно, но чтобы не загромождать функцию, часть функциональности, как вычисление правильного окончания слова "рубль", лучше выносить в отдельную функцию и вызывать её там, где это необходимо.
Это важно для читаемости.
В идеале функции должны выполнять одну работу и именоваться соответствующим образом.
| if (scanner.hasNextDouble()) { | ||
| if (sale < 0) { | ||
| System.out.println("Это некорректное значение для подсчёта"); | ||
| } else if (sale >= 0) { | ||
| sale = scanner.nextDouble(); |
There was a problem hiding this comment.
⚠ Тут лучше внести исправление небольшое. Если ввести стоимость товара в виде отрицательного числа, то программа добавляет товар, хотя стоимость не может быть отрицательной.
| System.out.println("Товар " + line + " за " + sale + " успешно добавлен. Введите следующий товар"); | ||
| break; |
There was a problem hiding this comment.
🍏 К этому сообщению неплохо было бы добавить информацию, что если необходимо закончить ввод данных, то надо ввести "Завершить"
| public static String rubb(int num) { | ||
| String rubl = null; | ||
| // if( num % 100 > 4 && num % 100 < 21) { | ||
| // rubl = "рублей"; | ||
| // } | ||
| if(num %10==1) { | ||
| rubl = "рубль"; | ||
| } | ||
| else if(num % 10 > 1 && num % 10 < 5) | ||
| { | ||
| rubl = "рубля"; | ||
| } | ||
| else {//if( num % 100 > 25 && num % 100 < 21) { | ||
| rubl = "рублей"; | ||
| } | ||
| return rubl; | ||
| } |
There was a problem hiding this comment.
⚠ До этого функция работала нормально. Сейчас имеется некорректность. Если, например, цена на каждого будет 114, то функция возвращает "рубля". Надо либо сделать как было, или изменить алгоритм.
There was a problem hiding this comment.
День добрый. Не одно, так второе) Попытка №3. Внёс изменения.
2 participants
No description provided.