StaticFactory :: use match instead of if else

2025-08-04 22:17:25 +02:00 · 2022-06-30 00:59:46 +03:00
parent b2d6414f81
commit 912d60b437
1 changed files with 5 additions and 7 deletions
--- a/Creational/StaticFactory/StaticFactory.php
+++ b/Creational/StaticFactory/StaticFactory.php
@@ -14,12 +14,10 @@ final class StaticFactory
 {
    public static function factory(string $type): Formatter
    {
-        if ($type == 'number') {
-            return new FormatNumber();
-        } elseif ($type == 'string') {
-            return new FormatString();
-        }
-
-        throw new InvalidArgumentException('Unknown format given');
+        return match($type) {
+            'number' => new FormatNumber(),
+            'string' => new FormatString(),
+            default => throw new InvalidArgumentException('Unknown format given'),
+        };
    }
 }