Skip to content

Work проверка №2 - #2

Merged
Vizor777 merged 3 commits into
devfrom
work
May 14, 2024
Merged

Vizor777 merged 3 commits into
devfrom
work

Conversation

@Vizor777

Copy link
Copy Markdown
Owner

No description provided.

@AleksandrIlinskii

Copy link
Copy Markdown

Исправлять код нужно в том же пулл реквесте, в котором тебе его оставили.

@AleksandrIlinskii

Copy link
Copy Markdown

У тебя есть функция, которая ничего не делает
image

Comment thread src/main/java/Calc.java Outdated
Comment on lines +21 to +24
for (Product product : products
) {
System.out.println(product.name);
}

@AleksandrIlinskii AleksandrIlinskii May 13, 2024 •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
for (Product product : products
) {
System.out.println(product.name);
}
for (Product product : products) {
System.out.println(product.name);
}

Comment thread src/main/java/Main.java Outdated
System.out.println("Введите название товара.\n");
name = scanner.nextLine();
name = name.trim();
if (name == null || name.trim().isEmpty() || name.isBlank()) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Проверка на null излишня
image

Comment thread src/main/java/Main.java Outdated
}
} else System.out.println("Неверное количество человек, повторите ввод.\n");
}
if (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.

Проверка излишня
image

@Vizor777
Vizor777 merged commit 5d2f506 into dev May 14, 2024
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