Skip to content

Проектная работа №1 «Калькулятор счёта» (ver 1.0) - #7

Open
el-pirata-bueno wants to merge 3 commits into
Yandex-Practicum:masterfrom
el-pirata-bueno:master
Open

el-pirata-bueno wants to merge 3 commits into
Yandex-Practicum:masterfrom
el-pirata-bueno:master

Conversation

@el-pirata-bueno

Copy link
Copy Markdown

Проектная работы №1 «Калькулятор счёта» (ver 1.0)
В этот раз PR сделал между ветками master моего репозитория и репозитория ЯП.

В прошлый раз делал PR между ветками master и main своего репозитория, все изменения были доступны тут: el-pirata-bueno@74923d6

@el-pirata-bueno

el-pirata-bueno commented Aug 19, 2022 •

Copy link
Copy Markdown
Author

Проектная работа №1 «Калькулятор счёта» (ver 1.5)
Во втором коммите учёл все замечания в ревью + создал отдельный метод для обработки количества человек в счёте.

@el-pirata-bueno el-pirata-bueno changed the title Проектная работы №1 «Калькулятор счёта» (ver 1.0) Проектная работа №1 «Калькулятор счёта» (ver 1.0) Aug 19, 2022
Comment thread src/main/java/Main.java Outdated
/* Ввод пользователем вручную количества человек в счёте. Допустимое значение - от 2 и больше */
while (true) {
peoplesInCheck = scanPeoplesInCheck();
if (peoplesInCheck != -1) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

На будущее, здесь лучше сравнивать с константой INPUT_ERROR, так код будет понятнее

Comment thread src/main/java/Main.java Outdated
itemName = scanItemName();
itemPrice = scanItemPrice();

if (itemPrice != -1.0) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

На будущее, здесь лучше сравнивать с константой INPUT_ERROR, так код будет понятнее

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants