refactor: minor clean up

Author: innocenzi

packages/mapper/src/CasterFactory.php

@@ -18,7 +18,7 @@ final class CasterFactory
     /**
      * @var array<string, array{string|Closure, class-string<\Tempest\Mapper\Caster>|Closure, int}[]>
      */
-    private array $contextCasters = [];
+    private array $casters = [];
 
     public function __construct(
         private readonly Container $container,
@@ -29,13 +29,10 @@ public function __construct(
      */
     public function addCaster(string|array|Closure $for, string|Closure $casterClass, string $context = Context::DEFAULT, int $priority = 0): self
     {
-        if (! isset($this->contextCasters[$context])) {
-            $this->contextCasters[$context] = [];
-        }
-
-        $this->contextCasters[$context][] = [$for, $casterClass, $priority];
+        $this->casters[$context] ??= [];
+        $this->casters[$context][] = [$for, $casterClass, $priority];
 
-        usort($this->contextCasters[$context], static fn (array $a, array $b) => $a[2] <=> $b[2]);
+        usort($this->casters[$context], static fn (array $a, array $b) => $a[2] <=> $b[2]);
 
         return $this;
     }
@@ -62,8 +59,8 @@ public function forProperty(PropertyReflector $property, string $context = Conte
         }
 
         $casters = [
-            ...($this->contextCasters[$context] ?? []),
-            ...($this->contextCasters[Context::DEFAULT] ?? []),
+            ...($this->casters[$context] ?? []),
+            ...($this->casters[Context::DEFAULT] ?? []),
         ];
 
         foreach ($casters as [$for, $casterClass]) {

packages/mapper/src/SerializerFactory.php

@@ -19,7 +19,7 @@ final class SerializerFactory
     /**
      * @var array<string,array{string|Closure,class-string<\Tempest\Mapper\Serializer>|Closure,int}[]>
      */
-    private(set) array $contextSerializers = [];
+    private(set) array $serializers = [];
 
     public function __construct(
         private readonly Container $container,
@@ -30,13 +30,10 @@ public function __construct(
      */
     public function addSerializer(string|array|Closure $for, string|Closure $serializerClass, string $context = Context::DEFAULT, int $priority = 0): self
     {
-        if (! isset($this->contextSerializers[$context])) {
-            $this->contextSerializers[$context] = [];
-        }
-
-        $this->contextSerializers[$context][] = [$for, $serializerClass, $priority];
+        $this->serializers[$context] ??= [];
+        $this->serializers[$context][] = [$for, $serializerClass, $priority];
 
-        usort($this->contextSerializers[$context], static fn (array $a, array $b) => $a[2] <=> $b[2]);
+        usort($this->serializers[$context], static fn (array $a, array $b) => $a[2] <=> $b[2]);
 
         return $this;
     }
@@ -63,8 +60,8 @@ public function forProperty(PropertyReflector $property, string $context = Conte
         }
 
         $serializers = [
-            ...($this->contextSerializers[$context] ?? []),
-            ...($this->contextSerializers[Context::DEFAULT] ?? []),
+            ...($this->serializers[$context] ?? []),
+            ...($this->serializers[Context::DEFAULT] ?? []),
         ];
 
         foreach ($serializers as [$for, $serializerClass]) {
@@ -95,8 +92,8 @@ public function forValue(mixed $value, string $context = Context::DEFAULT): ?Ser
         }
 
         $serializers = [
-            ...($this->contextSerializers[$context] ?? []),
-            ...($this->contextSerializers[Context::DEFAULT] ?? []),
+            ...($this->serializers[$context] ?? []),
+            ...($this->serializers[Context::DEFAULT] ?? []),
         ];
 
         foreach ($serializers as [$for, $serializerClass]) {