Refactoring and naming strategies draft

Author: ivan.babenko

src/Configurable.php

@@ -0,0 +1,18 @@
+<?php
+
+namespace BabenkoIvan\FluentArray;
+
+interface Configurable
+{
+    /**
+     * @param FluentArray|null $defaultConfig
+     * @return mixed
+     */
+    public static function defaultConfig(FluentArray $defaultConfig = null);
+
+    /**
+     * @param FluentArray|null $config
+     * @return mixed
+     */
+    public function config(FluentArray $config = null);
+}

src/FluentArray.php

@@ -2,53 +2,21 @@
 
 namespace BabenkoIvan\FluentArray;
 
-class FluentArray
+class FluentArray implements Configurable
 {
+    use HasConfiguration;
+
     /**
      * @var array
      */
     private $storage = [];
 
-    /**
-     * @var FluentArray
-     */
-    private static $defaultConfig;
-
-    /**
-     * @var FluentArray
-     */
-    private $config;
-
     /**
      * @param FluentArray|null $config
      */
     public function __construct(FluentArray $config = null)
     {
-        $this->config = $config;
-    }
-
-    /**
-     * @return FluentArray
-     */
-    public static function defaultConfig()
-    {
-        if (!isset(static::$defaultConfig)) {
-            static::$defaultConfig = new FluentArray();
-        }
-
-        return static::$defaultConfig;
-    }
-
-    /**
-     * @return FluentArray
-     */
-    public function config()
-    {
-        if (!isset($this->config)) {
-            $this->config = clone static::defaultConfig();
-        }
-
-        return $this->config;
+        $this->config($config);
     }
 
     /**

src/HasConfiguration.php

@@ -0,0 +1,51 @@
+<?php
+
+namespace BabenkoIvan\FluentArray;
+
+trait HasConfiguration
+{
+    /**
+     * @var FluentArray
+     */
+    private static $defaultConfig;
+
+    /**
+     * @var FluentArray
+     */
+    private $config;
+
+    /**
+     * @param FluentArray|null $defaultConfig
+     * @return mixed
+     */
+    public static function defaultConfig(FluentArray $defaultConfig = null)
+    {
+        if (isset($defaultConfig)) {
+            static::$defaultConfig = $defaultConfig;
+        } else {
+            if (!isset(static::$defaultConfig)) {
+                static::$defaultConfig = new FluentArray();
+            }
+
+            return static::$defaultConfig;
+        }
+    }
+
+    /**
+     * @param FluentArray|null $config
+     * @return mixed
+     */
+    public function config(FluentArray $config = null)
+    {
+        if (isset($config)) {
+            $this->config = $config;
+            return $this;
+        } else {
+            if (!isset($this->config)) {
+                $this->config = clone static::defaultConfig();
+            }
+
+            return $this->config;
+        }
+    }
+}

src/NamingStrategies/CamelCaseStrategy.php

@@ -0,0 +1,15 @@
+<?php
+
+namespace BabenkoIvan\FluentArray\NamingStrategies;
+
+class CamelCaseStrategy implements NamingStrategy
+{
+    /**
+     * @param string $key
+     * @return string
+     */
+    public function transform(string $key): string
+    {
+        // TODO: Implement transform() method.
+    }
+}

src/NamingStrategies/NamingStrategy.php

@@ -0,0 +1,12 @@
+<?php
+
+namespace BabenkoIvan\FluentArray\NamingStrategies;
+
+interface NamingStrategy
+{
+    /**
+     * @param string $key
+     * @return string
+     */
+    public function transform(string $key): string;
+}

src/NamingStrategies/NullStrategy.php

@@ -0,0 +1,15 @@
+<?php
+
+namespace BabenkoIvan\FluentArray\NamingStrategies;
+
+class NullStrategy implements NamingStrategy
+{
+    /**
+     * @param string $key
+     * @return string
+     */
+    public function transform(string $key): string
+    {
+        return $key;
+    }
+}

src/NamingStrategies/UnderscoreStrategy.php

@@ -0,0 +1,15 @@
+<?php
+
+namespace BabenkoIvan\FluentArray\NamingStrategies;
+
+class UnderscoreStrategy implements NamingStrategy
+{
+    /**
+     * @param string $key
+     * @return string
+     */
+    public function transform(string $key): string
+    {
+        // TODO: Implement transform() method.
+    }
+}