Skip to content

Даааа это жоска - #79

Open
3x-1 wants to merge 4 commits into
Yandex-Practicum:devfrom
3x-1:dev
Open

3x-1 wants to merge 4 commits into
Yandex-Practicum:devfrom
3x-1:dev

Conversation

@3x-1

@3x-1 3x-1 commented Jun 2, 2023

Copy link
Copy Markdown

Ээх, щас бы второй раз проект писать, потому что папку снес xD

Comment thread src/main/java/Calculator.java Outdated
stuff.price = scanner.nextDouble();
totalCOst += stuff.price;
stuff.name = scanner.nextLine();
ArrayList<String> allNames = new ArrayList<>();

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

стоит переименовать переменную allNames, к примеру itemsNamesList

Comment thread src/main/java/Calculator.java Outdated

System.out.println("Введите стоимость товара");
System.out.println("Введите название товара");
Stuff stuff = new Stuff();

@nadzyathedude nadzyathedude Jun 3, 2023 •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

чаще подобные перечисления именуют просто items

Comment thread src/main/java/Stuff.java Outdated
Scanner scanner = new Scanner(System.in);
String name = scanner.nextLine();
Double price = scanner.nextDouble();
String keyWOrd1 = "завершить";

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

стоит переименовать, например finishKeyword

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