Feature/Optimizations (#38)

* Cache attributes in memory to optimize performance
* Keep cached string keys list in memory cache to optimize performance
* Keep cached invariants list in memory cache to optimize performance

Co-authored-by: Dmytro Zasiadko <dmytro@2amigos.us>

Author: segoddnja

src/Domain/Traits/HasAttributes.php

@@ -14,6 +14,18 @@
  */
 trait HasAttributes
 {
+    /**
+     * Static property to keep cached attributes list to optimize performance.
+     * @var array<array<string, mixed>>
+     */
+    private static array $_attributesCache = [];
+
+    /**
+     * Static property to keep cached string keys to optimize performance
+     * @var array<string, string>
+     */
+    private static array $_stringKeysCache = [];
+
     /**
      * Return the list of attributes of the current class.
      * Properties starting with "_" will be considered as internal use only.
@@ -22,10 +34,13 @@ trait HasAttributes
      */
     final public static function attributes(): array
     {
-        return array_filter(
-            array_keys(get_class_vars(static::class)),
-            fn(string $item) => strpos($item, '_') !== 0
-        );
+        if (empty(static::$_attributesCache[static::class])) {
+            static::$_attributesCache[static::class] = array_filter(
+                array_keys(get_class_vars(static::class)),
+                fn(string $item) => strpos($item, '_') !== 0
+            );
+        }
+        return static::$_attributesCache[static::class];
     }
 
     /**
@@ -108,12 +123,16 @@ final protected function set(string $attribute, $value): void
      */
     protected function getStringKey(string $id, string $prefix = '', string $suffix = ''): string
     {
-        return sprintf(
-            '%s%s%s',
-            $prefix,
-            implode('', map(fn(string $chunk) => ucfirst($chunk), explode('_', $id))),
-            $suffix
-        );
+        $cacheKey = implode('-', [$id, $prefix, $suffix]);
+        if (empty(static::$_stringKeysCache[$cacheKey])) {
+            static::$_stringKeysCache[$cacheKey] = sprintf(
+                '%s%s%s',
+                $prefix,
+                implode('', map(fn(string $chunk) => ucfirst($chunk), explode('_', $id))),
+                $suffix
+            );
+        }
+        return static::$_stringKeysCache[$cacheKey];
     }
 
     /**

src/Domain/Traits/HasInvariants.php

@@ -18,6 +18,12 @@
  */
 trait HasInvariants
 {
+    /**
+     * Static property to keep cached invariants list to optimize performance.
+     * @var array<string, <string, mixed>>
+     */
+    private static $_invariantsCache = [];
+
     /**
      * Invariant configuration.
      *
@@ -40,20 +46,25 @@ trait HasInvariants
      */
     final public static function invariants(): array
     {
-        $invariants = [];
-        foreach (get_class_methods(static::class) as $invariant) {
-            if (strpos($invariant, 'invariant') === 0 && $invariant !== 'invariants') {
-                $invariants[$invariant] = str_replace(
-                    'invariant ',
-                    '',
-                    strtolower(
-                        preg_replace('/[A-Z]([A-Z](?![a-z]))*/', ' $0', $invariant)
-                    )
-                );
+        if (empty(static::$_invariantsCache[static::class])) {
+            $invariants = [];
+
+            foreach (get_class_methods(static::class) as $invariant) {
+                if (strpos($invariant, 'invariant') === 0 && $invariant !== 'invariants') {
+                    $invariants[$invariant] = str_replace(
+                        'invariant ',
+                        '',
+                        strtolower(
+                            preg_replace('/[A-Z]([A-Z](?![a-z]))*/', ' $0', $invariant)
+                        )
+                    );
+                }
             }
+
+            static::$_invariantsCache[static::class] = $invariants;
         }
 
-        return $invariants;
+        return static::$_invariantsCache[static::class];
     }
 
     /**