Skip to content

Первая проектная работа#1

Open
Rai-gou wants to merge 7 commits intomainfrom
dev
Open

Первая проектная работа#1
Rai-gou wants to merge 7 commits intomainfrom
dev

Conversation

@Rai-gou
Copy link
Copy Markdown
Owner

@Rai-gou Rai-gou commented Jan 18, 2023

Счетчик товаров

Copy link
Copy Markdown

@SeveNChaK SeveNChaK left a comment

Choose a reason for hiding this comment

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

В файлах код прыгает, отступы не одинаковые - рекомендую использовать встроенные форматтер. Если ты программируешь в IDE от JetBrains, то можешь пройтись по всем файлам и нажать комбинацию (Windows: Shift + Ctrl + Alt + L, MacOS: Shift + Option + Command + L), затем выбрать, какую часть кода хочешь изменить, и выполнить команду. Среда разработки автоматически сделает нужные отступы и переносы (как она это делает можно изменить в настройках, но дефолтные значения вполне нормальные).


while (true) {

spisokTovar += tovar + "\n";
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 является неизменяемым объектом, то при каждом сложении строк создаются новые объекты, что расходует память. Для таких случаев лучше использовать StringBuilder


rubl = Math.floor(stoimSumma / endHum);
calc.getRubl(rubl);
Calc.getRubl(rubl);
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ После применения Math.floor теряется дробная часть, если сумма была не целыи числом -> результат считается некорректно. Данный метод надо использовать, чтобы получить целую часть, чтобы потом определить склонение. Но результат итоговый надо счет без применения округления

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