Improve preprocessor

Author: SerafimArts

src/ContextInterface.php

@@ -9,18 +9,10 @@
 
 declare(strict_types=1);
 
-namespace FFI\Preprocessor\Directives\Directive;
-
-use FFI\Preprocessor\Directives\DirectiveInterface;
-use FFI\Preprocessor\Directives\RepositoryProviderInterface;
+namespace FFI\Preprocessor\Directive;
 
 abstract class Directive implements DirectiveInterface
 {
-    /**
-     * @var string
-     */
-    public const DEFAULT_VALUE = RepositoryProviderInterface::DEFAULT_VALUE;
-
     /**
      * @var string
      */
@@ -87,12 +79,12 @@ public function getMinArgumentsCount(): int
      * @param mixed $result
      * @return string
      */
-    protected function toString(mixed $result): string
+    public static function render(mixed $result): string
     {
         return match(true) {
             $result === true => '1',
             \is_null($result), $result === false => '0',
-            \is_string($result) => \var_export($result, true),
+            \is_string($result) => '"' . \addcslashes($result, '"') . '"',
             default => (string)$result,
         };
     }

src/Directives/Directive/Directive.php renamed to src/Directive/Directive.php

@@ -9,13 +9,18 @@
 
 declare(strict_types=1);
 
-namespace FFI\Preprocessor\Directives;
+namespace FFI\Preprocessor\Directive;
 
 /**
  * @mixin callable
  */
 interface DirectiveInterface
 {
+    /**
+     * @var string
+     */
+    public const DEFAULT_VALUE = '';
+
     /**
      * @param string ...$args
      * @return string

src/Directives/DirectiveInterface.php renamed to src/Directive/DirectiveInterface.php

@@ -9,7 +9,7 @@
 
 declare(strict_types=1);
 
-namespace FFI\Preprocessor\Directives\Directive;
+namespace FFI\Preprocessor\Directive;
 
 final class FunctionDirective extends Directive
 {
@@ -38,6 +38,6 @@ public function __construct(callable $cb)
      */
     public function __invoke(string ...$args): string
     {
-        return $this->toString(($this->callback)(...$args));
+        return $this->render(($this->callback)(...$args));
     }
 }

src/Directives/Directive/FunctionDirective.php renamed to src/Directive/FunctionDirective.php

@@ -9,9 +9,9 @@
 
 declare(strict_types=1);
 
-namespace FFI\Preprocessor\Directives\Directive;
+namespace FFI\Preprocessor\Directive;
 
-use FFI\Preprocessor\Directives\Compiler;
+use FFI\Preprocessor\Directive\FunctionLikeDirective\Compiler;
 
 final class FunctionLikeDirective extends Directive
 {
@@ -64,6 +64,6 @@ public function __invoke(string ...$args): string
             $this->compiled = Compiler::compile($this->body, $this->args);
         }
 
-        return $this->toString(($this->compiled)(...$args));
+        return $this->render(($this->compiled)(...$args));
     }
 }

src/Directives/Directive/FunctionLikeDirective.php renamed to src/Directive/FunctionLikeDirective.php

@@ -9,9 +9,9 @@
 
 declare(strict_types=1);
 
-namespace FFI\Preprocessor\Directives;
+namespace FFI\Preprocessor\Directive\FunctionLikeDirective;
 
-use FFI\Preprocessor\Directives\Directive\FunctionLikeDirective;
+use FFI\Preprocessor\Directive\FunctionLikeDirective;
 
 /**
  * @internal Refers to {@see FunctionLikeDirective}

src/Directives/Compiler.php renamed to src/Directive/FunctionLikeDirective/Compiler.php

@@ -9,7 +9,7 @@
 
 declare(strict_types=1);
 
-namespace FFI\Preprocessor\Directives\Directive;
+namespace FFI\Preprocessor\Directive;
 
 final class ObjectLikeDirective extends Directive
 {

src/Directives/Directive/ObjectLikeDirective.php renamed to src/Directive/ObjectLikeDirective.php

@@ -9,12 +9,12 @@
 
 declare(strict_types=1);
 
-namespace FFI\Preprocessor\Environment;
+namespace FFI\Preprocessor\Directive;
 
-interface EnvironmentProviderInterface
+interface ProviderInterface
 {
     /**
-     * @param EnvironmentInterface $env
+     * @return RepositoryInterface
      */
-    public function load(EnvironmentInterface $env): void;
+    public function getDirectives(): RepositoryInterface;
 }

src/Environment/EnvironmentProviderInterface.php renamed to src/Directive/ProviderInterface.php

@@ -0,0 +1,30 @@
+<?php
+
+/**
+ * This file is part of FFI package.
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
+namespace FFI\Preprocessor\Directive;
+
+use FFI\Preprocessor\Exception\DirectiveDefinitionException;
+
+interface RegistrarInterface
+{
+    /**
+     * @param string $directive
+     * @param mixed $value
+     * @throws DirectiveDefinitionException
+     */
+    public function define(string $directive, mixed $value = DirectiveInterface::DEFAULT_VALUE): void;
+
+    /**
+     * @param string $directive
+     * @return bool
+     */
+    public function undef(string $directive): bool;
+}

src/Directive/RegistrarInterface.php

@@ -0,0 +1,110 @@
+<?php
+
+/**
+ * This file is part of FFI package.
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
+namespace FFI\Preprocessor\Directive;
+
+use FFI\Preprocessor\Exception\DirectiveDefinitionException;
+
+final class Repository implements RepositoryInterface, RegistrarInterface
+{
+    /**
+     * @var array<string, DirectiveInterface>
+     */
+    private array $directives = [];
+
+    /**
+     * @param iterable<string, DirectiveInterface> $directives
+     * @throws \ReflectionException
+     */
+    public function __construct(iterable $directives = [])
+    {
+        foreach ($directives as $directive => $definition) {
+            $this->define($directive, $definition);
+        }
+    }
+
+    /**
+     * @param string|callable|DirectiveInterface $directive
+     * @return DirectiveInterface
+     * @throws \ReflectionException
+     */
+    private function cast(mixed $directive): DirectiveInterface
+    {
+        return match (true) {
+            $directive instanceof DirectiveInterface => $directive,
+            \is_callable($directive) => new FunctionDirective($directive),
+            \is_scalar($directive), $directive instanceof \Stringable => new ObjectLikeDirective((string)$directive),
+            default => $directive,
+        };
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    public function define(string $directive, mixed $value = DirectiveInterface::DEFAULT_VALUE): void
+    {
+        try {
+            $expr = $this->cast($value);
+        } catch (\Throwable $e) {
+            throw new DirectiveDefinitionException($e->getMessage(), (int)$e->getCode(), $e);
+        }
+
+        if ($expr instanceof ObjectLikeDirective) {
+            $this->directives = \array_merge([$directive => $expr], $this->directives);
+        } else {
+            $this->directives[$directive] = $expr;
+        }
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    public function undef(string $directive): bool
+    {
+        $exists = $this->defined($directive);
+
+        \unset($this->directives[$directive]);
+
+        return $exists;
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    public function defined(string $directive): bool
+    {
+        return isset($this->directives[$directive]);
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    public function find(string $directive): ?DirectiveInterface
+    {
+        return $this->directives[$directive] ?? null;
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    public function getIterator(): \Traversable
+    {
+        return new \ArrayIterator($this->directives);
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    public function count(): int
+    {
+        return \count($this->directives);
+    }
+}