Skip to content

First try of my homework - #1

Open
Ild0re wants to merge 5 commits into
mainfrom
dev
Open

Ild0re wants to merge 5 commits into
mainfrom
dev

Conversation

@Ild0re

@Ild0re Ild0re commented Dec 25, 2023

Copy link
Copy Markdown
Owner

No description provided.

Comment thread src/main/java/Calculator.java Outdated
Product newProduct = new Product(item, itemCost);
System.out.println("Товар успешно добавлен.");
sum += newProduct.cost;
items += "\n" + newProduct.name + " " + newProduct.cost;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

я бы советовал сделать ArrayList вместо String items; и там хранить созданные объекты. Строки не изменяемый тип и при конкатенации строк, вы создаете новый экземпляр строки в памяти(пуле строк). Советую в свободное время об этом почитать, про это часто спрашивают на собеседованиях) И использовать функцию с циклом для создания строки, когда нужно ее вывести на экран.

Comment thread src/main/java/Product.java Outdated
}

public String toString() {
return name + " " + cost ;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

тут бы хорошо смотрелся String.format(), можешь если время есть почитать про него и регулярные выражения, темы интересные и полезные. Заучивать их не нужно, но ознакомиться рекомендую)

@Ild0re

Ild0re commented Dec 29, 2023

Copy link
Copy Markdown
Owner Author

Постарался добавить в выводах String.format(), поправил логику.

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