Skip to content

Purchase List App#1

Open
vladko-89 wants to merge 4 commits intomainfrom
sprint-2
Open

Purchase List App#1
vladko-89 wants to merge 4 commits intomainfrom
sprint-2

Conversation

@vladko-89
Copy link
Copy Markdown
Owner

No description provided.

} else {
purchaseList = purchaseList + name + ": " + String.format("%.2f",cost) + str + "\n";
System.out.println("Товар " + name + " стоимостью " + String.format("%.2f",cost) + str + " успешно сохранен");
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

У вас есть класс Product, но вы его почему-то не используете. Вы можете сохранять данные в ArrayList purchaseList = new ArrayList();
И при необходимости печати в цикле перебирать элементы листа и подставлять значения в String.format.


}

public static void printPurchaseList() {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

У вас все поля и методы статичные, лучше так не делать, вы можете создать экземпляр класса и с ним работать.
Это делает ваш код более гибким и позволяет легче тестировать и повторно использовать ваш код.

verifyDouble = true;
} else {
System.out.println("Стоимость не может быть меньше 0");
scanner.nextLine();
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

повторно считываешь строку на одну проверку

while (!verifyDouble) {
if (scanner.hasNextDouble()) {
double tmp = scanner.nextDouble();
scanner.nextLine();
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

первый раз считываешь строку


while (!verify) {
if (!scanner.useDelimiter("\n").hasNextInt()) {
System.out.println("Не тот формат ввода. Введите целое число");
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

а тут как раз не хватает получения новой строки (scanner.nextLine();) ты все время проверяешь
в if (!scanner.useDelimiter("\n").hasNextInt()) {
одну и ту же строку, получается вечный цикл

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