Skip to content

Pull request созданный из Android studio - #1

Open
ZakharVP wants to merge 7 commits into
devfrom
One
Open

ZakharVP wants to merge 7 commits into
devfrom
One

Conversation

@ZakharVP

Copy link
Copy Markdown
Owner

Первый Pull request который создан через меню IDE

Comment thread Main.java
Scanner in = new Scanner(System.in);
System.out.println("На скольких человек необходимо разделить счет?");

boolean proverka = true;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Лучше не использовать нейминг русских слов в транслите, это в целом ни на что не влияет, но тогда надо и все остальные переменные и функции также называть
можно назвать "check"

Comment thread Main.java
boolean checkCost = true;
boolean checkEnter = true;
float cost = 0;
// int count = 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.

Если не используется, лучше удалить

Comment thread Main.java

//Функция второй части
//Что было заказано и по какой стоимости, с точностью до копеек.
static Map secondInput(){

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Можно более осознанно назвать, например, ввод товаров, ввод количества человек и т.п.

Comment thread Main.java
}

//Стоимость
System.out.println("Введите стоимость до копеек(.00).");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Тут можно добавить, в каком формате надо вводить копейки (через запятую или точку)

Comment thread Main.java
System.out.println("Если вы хотите завершить ввод - наберите \"Завершить\" или введите новый товар.");
checkCost = true;

// count += 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.

Можно удалить если не нужно

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