-
Notifications
You must be signed in to change notification settings - Fork 220
Comments
Евгений Ким. Консольное приложение No1 v1.0#105
Евгений Ким. Консольное приложение No1 v1.0 #105byBlinque wants to merge 2 commits intoYandex-Practicum:master from
Conversation
src/main/java/Main.java
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Можно не задавать начальное нулевое значение, так как затем всё равно происходит присваивание через scanner.nextInt();
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
можно сделать из этого цикла while цикл do-while и вынести это условие в while()
src/main/java/GoodCalculator.java
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
не хватает ключевого слова break для работы конструкции switch-case
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Конструктор не используется. Его можно убрать.
Евгений Ким. Консольное приложение No1 v1.0