Skip to content

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

Open
struklyts wants to merge 4 commits intomainfrom
dev
Open

Проектная работа №1#1
struklyts wants to merge 4 commits intomainfrom
dev

Conversation

@struklyts
Copy link
Copy Markdown
Owner

No description provided.

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.

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

Copy link
Copy Markdown
Owner Author

@struklyts struklyts Nov 14, 2024

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