Skip to content

Проектная работа №1 - #1

Open
struklyts wants to merge 4 commits into
mainfrom
dev
Open

struklyts wants to merge 4 commits into
mainfrom
dev

Conversation

@struklyts

Copy link
Copy Markdown
Owner

No description provided.

Comment thread src/main/java/main/java/Car.java Outdated
Comment on lines +14 to +27
public Car getWinner() {
if (this.winner != null) {
return this.winner;
}

this.winner = cars.get(0);
for (Car car: cars) {
if (car.speed > winner.speed) {
this.winner = car;
}
}

return this.winner;
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

От хранения массива машин и от этого цикла можно избавиться, запоминая победителя сразу при вводе, тем самым программа будет работать быстрее и требовать меньше памяти для выполнения

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Ты имеешь ввиду перенести расчет победителя в конструктор? Не понимаю как избавиться от цикла: заменить на какой-то имеющийся метод сортировки который отсортирует список по одному из полей?

@struklyts struklyts Nov 14, 2024 •

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

В целом конечно все можно написать короче и проще. Вопросы оптимизации конечно же важны в продакшене, при этом для меня сейчас важно пощупать и разобраться с синтаксисом, с подходами которые используются в java, поиспользовать структуры которые мы изучали и тп

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