extract rules to the warrior builder

Author: mlebkowski

src/Realms/RolePlaying/Character.php

@@ -5,37 +5,14 @@
 namespace App\Realms\RolePlaying;
 
 use App\Realms\RolePlaying\Inventory\Item;
-use App\Realms\RolePlaying\Inventory\ItemType;
-use App\Realms\RolePlaying\Inventory\Problems\CannotWearMultipleArmors;
-use App\Realms\RolePlaying\Inventory\Problems\CannotWearTooManyRings;
-use App\Realms\RolePlaying\Inventory\Problems\InventoryProblem;
-use App\Realms\RolePlaying\Inventory\Problems\NoDualWielding;
-use App\Realms\RolePlaying\Inventory\Problems\WeaponNotEquipped;
 use loophp\collection\Collection;
 use Stringable;
 
 final class Character implements Stringable
 {
-    /**
-     * @throws InventoryProblem
-     * @throws CannotWearTooManyRings
-     * @throws WeaponNotEquipped
-     * @throws CannotWearMultipleArmors
-     * @throws NoDualWielding
-     */
     public static function of(string $name, int $hitPoints, Item ...$items): self
     {
         $items = Collection::fromIterable($items);
-
-        $countByType = static fn (ItemType $type) => $items
-            ->filter(static fn (Item $item) => $item->type->is($type))
-            ->count();
-
-        WeaponNotEquipped::whenNoWeapon($countByType(ItemType::Weapon));
-        NoDualWielding::whenMoreThanOneWeapon($countByType(ItemType::Weapon));
-        CannotWearMultipleArmors::whenMoreThanOneArmor($countByType(ItemType::Armor));
-        CannotWearTooManyRings::whenMoreThanTwoRings($countByType(ItemType::Ring));
-
         $armor = $items->reduce(static fn (int $sum, Item $item) => $sum + $item->armor, 0);
         $attack = $items->reduce(static fn (int $sum, Item $item) => $sum + $item->damage, 0);
 

src/Realms/RolePlaying/WarriorBuilder.php

@@ -0,0 +1,65 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Realms\RolePlaying;
+
+use App\Realms\RolePlaying\Inventory\Item;
+use App\Realms\RolePlaying\Inventory\ItemType;
+use App\Realms\RolePlaying\Inventory\Problems\CannotWearMultipleArmors;
+use App\Realms\RolePlaying\Inventory\Problems\CannotWearTooManyRings;
+use App\Realms\RolePlaying\Inventory\Problems\NoDualWielding;
+use App\Realms\RolePlaying\Inventory\Problems\WeaponNotEquipped;
+use loophp\collection\Collection;
+
+final class WarriorBuilder
+{
+    private string $name = "player";
+    private int $hitPoints = 100;
+    private array $items = [];
+
+    public static function start(): self
+    {
+        return new self();
+    }
+
+    public function withHitPoints(int $hitPoints): self
+    {
+        $this->hitPoints = $hitPoints;
+        return $this;
+    }
+
+    public function withName(string $name): self
+    {
+        $this->name = $name;
+        return $this;
+    }
+
+    public function withItems(Item ...$items): self
+    {
+        $this->items = array_merge($this->items, $items);
+        return $this;
+    }
+
+    /**
+     * @throws WeaponNotEquipped
+     * @throws NoDualWielding
+     * @throws CannotWearMultipleArmors
+     * @throws CannotWearTooManyRings
+     */
+    public function build(): Character
+    {
+        $items = Collection::fromIterable($this->items);
+
+        $countByType = static fn (ItemType $type) => $items
+            ->filter(static fn (Item $item) => $item->type->is($type))
+            ->count();
+
+        WeaponNotEquipped::whenNoWeapon($countByType(ItemType::Weapon));
+        NoDualWielding::whenMoreThanOneWeapon($countByType(ItemType::Weapon));
+        CannotWearMultipleArmors::whenMoreThanOneArmor($countByType(ItemType::Armor));
+        CannotWearTooManyRings::whenMoreThanTwoRings($countByType(ItemType::Ring));
+
+        return Character::of($this->name, $this->hitPoints, ...$this->items);
+    }
+}

src/Solutions/Y2015/D21/RolePlayingGameSimulator.php

@@ -8,9 +8,9 @@
 use App\Aoc\Progress\Progress;
 use App\Aoc\Runner\RunMode;
 use App\Aoc\Solution;
-use App\Realms\RolePlaying\Character;
 use App\Realms\RolePlaying\Inventory\Item;
 use App\Realms\RolePlaying\Inventory\ItemType;
+use App\Realms\RolePlaying\WarriorBuilder;
 use App\Solutions\Y2015\D21\Strategy\InventoriesSatisfyingRequirements;
 use App\Solutions\Y2015\D21\Strategy\ShoppingCart;
 use loophp\collection\Collection;
@@ -25,12 +25,13 @@ public function challenges(): iterable
 
     public function solve(Challenge $challenge, mixed $input, RunMode $runMode): mixed
     {
-        $boss = Character::of(
-            'boss',
-            $input->hitPoints,
-            Item::of('boss weapon', damage: $input->damage, armor: 0, type: ItemType::Weapon),
-            Item::of('boss armor', damage: 0, armor: $input->armor, type: ItemType::Armor),
-        );
+        $boss = WarriorBuilder::start()
+            ->withName('boss')
+            ->withHitPoints($input->hitPoints)
+            ->withItems(
+                Item::of('boss item', $input->damage, $input->armor, ItemType::Weapon),
+            )
+            ->build();
 
         $positivePointsRequired = $boss->attack + $boss->armor;
 

src/Solutions/Y2015/D22/Readme.md

@@ -0,0 +1,3 @@
+# Oh boy!
+
+https://ericlippert.com/2015/05/11/wizards-and-warriors-part-five/

tests/Realms/RolePlaying/CharacterMother.php

@@ -13,18 +13,19 @@ final class CharacterMother
 {
     public static function withItems(string $name, int $hitPoints, string ...$names): Character
     {
-        $items = Collection::fromIterable(ItemsFactory::all())
+        $stock = Collection::fromIterable(ItemsFactory::all())
             ->map(static fn (ItemPrice $itemPrice) => [$itemPrice->item->name, $itemPrice->item])
             ->unpack()
             ->all(false);
 
-        return Character::of(
-            $name,
-            $hitPoints,
-            ...
-            Collection::fromIterable($names)
-                ->map(static fn (string $name) => $items[$name] ?? NoItemByThatName::ofName($name))
-                ->all(),
-        );
+        $items = Collection::fromIterable($names)
+            ->map(static fn (string $name) => $stock[$name] ?? NoItemByThatName::ofName($name))
+            ->all();
+
+        return WarriorBuilder::start()
+            ->withName($name)
+            ->withHitPoints($hitPoints)
+            ->withItems(...$items)
+            ->build();
     }
 }