Repository navigation
Conversation
| // ваш код начнется здесь | ||
| // вы не должны ограничиваться только классом Main и можете создавать свои классы по необходимости | ||
| System.out.println("Привет Мир"); | ||
| int count=0; |
There was a problem hiding this comment.
По умолчанию int инициализируется 0, так что здесь инициализировать не обязательно
| count = countOfPeople(); | ||
| System.out.println (count); | ||
| Scanner scanner = new Scanner(System.in); | ||
| boolean trigger = true; |
There was a problem hiding this comment.
Это инициализированное значение не используется (в while будет проверяться то, что будет получено от !chek.equalsIgnoreCase("complete");), так что можно не инициализировать
| if (bill <2 && bill>=1){ | ||
| System.out.println(String.format(finalPhrase1, bill)); | ||
| } else{ | ||
| System.out.println(String.format(finalPhrase2, bill)); |
There was a problem hiding this comment.
Строка System.out.println(String.format(<окончание слова рубль>, bill)); повторяется, лучше её вынести за пределы if-else, а в нем оставить только определение окончания, записать в переменную и использовать после оператора
| String finalPhrase1 = "%.2f рубль"; | ||
| String finalPhrase2 = "%.2f рубля"; | ||
| String finalPhrase3 = "%.2f рублей"; | ||
| int ostatok = (int) (Math.floor(bill)%10.0); |
There was a problem hiding this comment.
Хорошо, что используешь переменные для подсчета только 1 раз
Вместо 10.0 можно использовать просто 10
| if (ostatok == 1) { | ||
| System.out.println(String.format(finalPhrase1, bill)); | ||
| } else{ | ||
| System.out.println(String.format(finalPhrase2, bill)); |
There was a problem hiding this comment.
Код System.out.println(String.format(finalPhrase2, bill)); повторяется несколько раз, лучше его вынести за if-else, а в нем определять только в переменной, каким будет слово рубль
2 participants
Использовал английский язык в выводе, надеюсь это не критично.