Conversation
src/main/java/Calc.java
Outdated
|
|
||
| public void finCountDown(int guestNumber) { | ||
| System.out.println("Добавлено:\n" + productNameC); | ||
| /*System.out.println(String.format(productNameC,System.lineSeparator()));*///хз нашел в интернетах,не разобрался |
There was a problem hiding this comment.
Если есть вопросы - всегда можно спросить у куратора.
Но комментарии лучше не пушить
There was a problem hiding this comment.
Это для себя было , на случай если где то поломается.Спасибо!
src/main/java/FormRub.java
Outdated
| if ((int)(Calc.finalCost % 100 / 10) == 1){ | ||
| return " рублей."; | ||
| } | ||
| switch ((int) (Calc.finalCost % 10)){ |
There was a problem hiding this comment.
Совет на будущее: Если поставить курсор на switch, то можно увидеть фонарик желтый. Он подскажет улучшения кода
There was a problem hiding this comment.
Тут два момента :
- Хотелось\показалось что Swich тут подходит.
- при изменении выдает ошибки которые я не совсем понимаю как править.
src/main/java/Gests.java
Outdated
|
|
||
|
|
||
|
|
||
|
|
There was a problem hiding this comment.
Слишком много пустых строк, лучше не стоит так делать
There was a problem hiding this comment.
Принято убираю , в будущем постараюсь не оставлять.
| @@ -0,0 +1,30 @@ | |||
| import java.util.Scanner; | |||
|
|
|||
| public class Gests { | |||
There was a problem hiding this comment.
Если предполагалось, что название класса переводится как "Гость", то тут опечатка
There was a problem hiding this comment.
Да нет , я пока писал неслолько раз создавал похожие классы.
Крутил разные куски кода , в итоге что бы не путаться в блокноте получилось так.
Понимаю, буду внимательнее.
src/main/java/Gests.java
Outdated
|
|
||
| public class Gests { | ||
| public static int guestNumber; | ||
| public static int Gest() { |
There was a problem hiding this comment.
И тут опечатка (см коммент выше)
src/main/java/Gests.java
Outdated
|
|
||
| public class Gests { | ||
| public static int guestNumber; | ||
| public static int Gest() { |
There was a problem hiding this comment.
Зачем делать возвращаемый тип у метода int, если ты его не используешь? Для таких целей есть void
src/main/java/Main.java
Outdated
|
|
||
|
|
||
|
|
||
|
|
| @@ -0,0 +1,24 @@ | |||
| public class Calc { | |||
| String productNameC = ""; | |||
There was a problem hiding this comment.
Названия productName вполне достаточно для понимания зачем оно нужно
src/main/java/FormRub.java
Outdated
| case 4: | ||
| return " рубля."; | ||
| default: | ||
| return " рублей."; |
There was a problem hiding this comment.
Критическое замечание
Неверно выводится окончание слова рубль
… "лампочки" (больше желтым не подсвечивает). Незнаю можно ли делать commit после отправки ссылки на Практикум. Риск дело благородное.))
| } | ||
| if ((int) (Calc.finalCost % 10) == 1) { | ||
| return " рубль."; | ||
| } else if ((int) (Calc.finalCost % 10) == 2 || (int) (Calc.finalCost % 10) == 3 || (int) (Calc.finalCost % 10) == 4) { |
There was a problem hiding this comment.
Как совет, ты можешь парой строчек выше завести переменную типа
int lastDigit = (int) (Calc.finalCost % 10) и ее подставлять во все проверки твоих условий, а то сейчас получается, что ты считаешь одно и то же 4 раза.
Это называется boilerplate code (код, который повторяется в нескольких местах). Очень легко так ошибиться
4 дня ужаса
безнадеги
5-6 удалений всего и вся
и этот "монстр франкенштейна" ~ "готов".
Сдаюсь.
С богом.)))
Забыл в FormRub сделать 2ой метод на склонение слова Рубль.