fix: minor fixes

src/defender/defender.cpp

@@ -52,11 +52,8 @@ std::optional<size_t> Defender::get_nearest_attacker_index(
           if (b.is_ability_active())
             return true;
 
-          auto d1 = this->get_position().distance_to(a.get_position());
-
-          auto d2 = this->get_position().distance_to(b.get_position());
-
-          return d1 < d2;
+          return this->get_position().distance_to(a.get_position()) <
+                 this->get_position().distance_to(b.get_position());
         });
     return std::distance(attackers.begin(), nearest_attacker);
   }

src/game/game.cpp

@@ -55,13 +55,13 @@ Game Game::simulate(
         }
       });
 
-  for (Game::attacker_id id : ability_activations) {
+  ranges::for_each(ability_activations, [&](Game::attacker_id id) {
     auto attacker_index = this->get_attacker_index_by_id(id);
     unsigned ability_activation_cost =
         attackers[*attacker_index].get_ability_activation_cost();
 
     if (coins_left < ability_activation_cost) {
-      break;
+      return;
     }
     coins_left -= ability_activation_cost;
     // In case the ability is already active,
@@ -70,7 +70,7 @@ Game Game::simulate(
     if (attacker_index.has_value()) {
       attackers[*attacker_index].activate_ability(turn);
     }
-  }
+  });
 
   // Attacker Loop
   ranges::for_each(attackers, [&](Attacker &attacker) mutable {