RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-26 09:37:45 +00:00
Code Issues Activity Actions

Hearts: Avoid reallocations for Vectors when possible

Browse source
This commit is contained in:
Gunnar Beutner 2021-07-24 00:55:26 +02:00 • committed by Andreas Kling
parent 5201c17d9b
commit d76b42599e
1 changed files with 2 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

2
Userland/Games/Hearts/Game.cpp
View file

@ -203,6 +203,7 @@ void Game::setup(String player_name, int hand_number)
} }
NonnullRefPtrVector<Card> deck; NonnullRefPtrVector<Card> deck;
deck.ensure_capacity(Card::card_count * 4);
for (int i = 0; i < Card::card_count; ++i) { for (int i = 0; i < Card::card_count; ++i) {
deck.append(Card::construct(Card::Type::Clubs, i)); deck.append(Card::construct(Card::Type::Clubs, i));
@ -212,6 +213,7 @@ void Game::setup(String player_name, int hand_number)
} }
for (auto& player : m_players) { for (auto& player : m_players) {
player.hand.ensure_capacity(Card::card_count);
for (uint8_t i = 0; i < Card::card_count; ++i) { for (uint8_t i = 0; i < Card::card_count; ++i) {
auto card = deck.take(get_random_uniform(deck.size())); auto card = deck.take(get_random_uniform(deck.size()));
if constexpr (!HEARTS_DEBUG) { if constexpr (!HEARTS_DEBUG) {