perf(CopyTransformer): Move identity check to the beginning.

rekalogika · Jan 15, 2024 · 0555d64 · 0555d64
1 parent 58c0485
commit 0555d64
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -9,6 +9,7 @@
 * dx(`TypeCheck`): Now accept `MixedType` as an argument.
 * test: Mapping test first version.
 * refactor: rename `RefuseToHandleException` to `RefuseToTransformException`.
+* perf(`CopyTransformer`): Move identity check to the beginning.
 
 ## 0.5.10
 

diff --git a/src/Transformer/CopyTransformer.php b/src/Transformer/CopyTransformer.php
@@ -30,6 +30,10 @@ public function transform(
         ?Type $targetType,
         Context $context
     ): mixed {
+        if ($targetType !== null && !TypeCheck::isVariableInstanceOf($source, $targetType)) {
+            throw new RefuseToTransformException();
+        }
+
         if (!is_object($source)) {
             return $source;
         }
@@ -42,10 +46,6 @@ public function transform(
 
         $result = clone $source;
 
-        if ($targetType !== null && !TypeCheck::isVariableInstanceOf($result, $targetType)) {
-            throw new RefuseToTransformException();
-        }
-
         return $result;
     }