Repository navigation
Conversation
| System.out.println("Сколько счетов у Вас будет?"); | ||
| int howPeople = 0; | ||
| while (true){ | ||
| howPeople = scanner.nextInt(); |
There was a problem hiding this comment.
⚠ Нужно еще учесть, что пользователь может ввести строку и тогда приложение упадёт. Лучше такого не допускать, а обрабатывать. Можно использовать методы scanner.hasNextInt()/hasNextFloat() или try-catch
| String RUB; | ||
| String RUB2; |
There was a problem hiding this comment.
🍏 Переменные лучше именовать используя lowerCamelCase-стиль
| if (endingRubles % 100 >= 11 && endingRubles % 100 <= 14) { | ||
| RUB = "рублей"; | ||
| } else | ||
|
|
||
| switch (endingRubles %10){ | ||
| case 1: | ||
| RUB = "рубль"; | ||
| break; | ||
| case 2: | ||
| case 3: | ||
| case 4: | ||
| RUB = "рубля"; | ||
| break; | ||
| default: | ||
| RUB = "рублей"; | ||
| break; | ||
| } | ||
| if (endingRubles2 % 100 >= 11 && endingRubles2 % 100 <= 14) { | ||
| RUB2 = "рублей"; | ||
| } else | ||
| switch (endingRubles2 % 10){ | ||
| case 1: | ||
| RUB2 = "рублю"; | ||
| break; | ||
| case 2: | ||
| case 3: | ||
| case 4: | ||
| RUB2 = "рубля"; | ||
| break; | ||
| default: | ||
| RUB2 = "рублей"; | ||
| break; | ||
| } |
There was a problem hiding this comment.
⚠ Здесь дублируется код из-за двух переменных. Лучше не допускать повторений кода. Для избежания этого, можно вычисление окончания слова "рубль", вынести в метод, который на вход может принимать числовое значение и возвращать вычисленную строку. Следовательно, можно этот метод переиспользовать там, где это необходимо.
String formatRub(double price) {
////
}
| String product = scanner.next(); | ||
| listProducts += product + "\n"; | ||
| System.out.println("Введите цену товара в формате ХХ,ХХ"); | ||
| double productPrice = scanner.nextDouble(); |
There was a problem hiding this comment.
⚠ Здесь тоже есть необходимость добавить обработку некорректного ввода в виде строки
| if (productPrice <= 0) { | ||
| System.out.println("Введена неверная цена, цена должна быть больше 0.00"); | ||
| } | ||
| if (productPrice > 0) { | ||
| } |
There was a problem hiding this comment.
⚠ Вне зависимости от того, что производится проверка ввода отрицательного числа, значение все равно будет сохранено в finalProductPrice
У тебя уже есть проверка, в которой можно сохранять положительные значения
if (productPrice > 0) {
}
| @@ -1 +1,2 @@ | |||
| # Пустой репозиторий для работы с Java кодом в Android Studio | |||
| И пусть все получиться! No newline at end of file | |||
|
Добрый день, что-то в почте только смотрел, только увидел, что проверено еще 13 числа. Не очень понятно, все замечания надо исправить, или это для дальнейшего. Короче я не очень понимаю как понять принята работа или нет, где это можно увидеть? |
Посмотреть статус проверки, наверное, можно на платформе, через которую ты отправлял ссылку. Если что, на этот счет, ты можешь уточнить у наставника или наставницы. Там же я оставил замечания, которые нужно исправить. А здесь я их продублировал и отметил ⚠. Замечания с 🍏 носят рекомендательный характер |
|
Спасибо, увидел где! |
2 participants
Это мой первый Pull Request, надеюсь теперь получиться!