Test evaluator refactored

Author: remorhaz

src/Pointer/Evaluate.php

@@ -58,6 +58,11 @@ abstract class Evaluate
 
     protected $nonNumericIndices = false;
 
+    /**
+     * @var ReferenceEvaluate|null
+     */
+    protected $referenceEvaluate;
+
 
     /**
      * Constructor.
@@ -224,6 +229,28 @@ public function perform()
     abstract protected function createReferenceEvaluate(Reference $reference);
 
 
+    protected function setupReferenceEvaluate(Reference $reference)
+    {
+        $this->referenceEvaluate = $this
+            ->createReferenceEvaluate($reference)
+            ->setReference($reference)
+            ->setData($this->cursor);
+        return $this;
+    }
+
+
+    /**
+     * @return ReferenceEvaluate
+     */
+    protected function getReferenceEvaluate()
+    {
+        if (null === $this->referenceEvaluate) {
+            throw new LogicException("Reference evaluator is not set in locator evaluator");
+        }
+        return $this->referenceEvaluate;
+    }
+
+
     protected function processReference(Reference $reference)
     {
         try {

src/Pointer/Evaluate/ReferenceTestAllowedNonNumericIndex.php

@@ -0,0 +1,35 @@
+<?php
+
+namespace Remorhaz\JSONPointer\Pointer\Evaluate;
+
+class ReferenceTestAllowedNonNumericIndex extends ReferenceEvaluate
+{
+
+
+    protected function doesExist()
+    {
+        return array_key_exists($this->getIndex(), $this->getData());
+    }
+
+
+    protected function performExisting()
+    {
+        $this->data = &$this->data[$this->getIndex()];
+        return $this;
+    }
+
+
+    protected function performNonExisting()
+    {
+        $result = false;
+        return $this->setResult($result);
+    }
+
+
+    protected function getIndex()
+    {
+        return $this
+            ->getReference()
+            ->getValue();
+    }
+}

src/Pointer/Evaluate/ReferenceTestNextIndex.php

@@ -0,0 +1,26 @@
+<?php
+
+namespace Remorhaz\JSONPointer\Pointer\Evaluate;
+
+class ReferenceTestNextIndex extends ReferenceEvaluate
+{
+
+
+    protected function doesExist()
+    {
+        return false;
+    }
+
+
+    protected function performExisting()
+    {
+        throw new LogicException("Next index never exists");
+    }
+
+
+    protected function performNonExisting()
+    {
+        $result = false;
+        return $this->setResult($result);
+    }
+}

src/Pointer/Evaluate/ReferenceTestNotAllowedNonNumericIndex.php

@@ -0,0 +1,36 @@
+<?php
+
+namespace Remorhaz\JSONPointer\Pointer\Evaluate;
+
+class ReferenceTestNotAllowedNonNumericIndex extends ReferenceEvaluate
+{
+
+
+    protected function doesExist()
+    {
+        return false; // We don't allow it to exist :)
+    }
+
+
+    protected function performExisting()
+    {
+        throw new LogicException(
+            "Non-numeric index '{$this->getIndex()}' is not allowed to exist"
+        );
+    }
+
+
+    protected function performNonExisting()
+    {
+        $result = false;
+        return $this->setResult($result);
+    }
+
+
+    protected function getIndex()
+    {
+        return $this
+            ->getReference()
+            ->getValue();
+    }
+}

src/Pointer/Evaluate/ReferenceTestNumericIndex.php

@@ -0,0 +1,35 @@
+<?php
+
+namespace Remorhaz\JSONPointer\Pointer\Evaluate;
+
+class ReferenceTestNumericIndex extends ReferenceEvaluate
+{
+
+
+    protected function doesExist()
+    {
+        return array_key_exists($this->getIndex(), $this->getData());
+    }
+
+
+    protected function performExisting()
+    {
+        $this->data = &$this->data[$this->getIndex()];
+        return $this;
+    }
+
+
+    protected function performNonExisting()
+    {
+        $result = false;
+        return $this->setResult($result);
+    }
+
+
+    protected function getIndex()
+    {
+        return (int) $this
+            ->getReference()
+            ->getValue();
+    }
+}

src/Pointer/Evaluate/ReferenceTestProperty.php

@@ -0,0 +1,35 @@
+<?php
+
+namespace Remorhaz\JSONPointer\Pointer\Evaluate;
+
+class ReferenceTestProperty extends ReferenceEvaluate
+{
+
+
+    protected function doesExist()
+    {
+        return property_exists($this->getData(), $this->getProperty());
+    }
+
+
+    protected function performExisting()
+    {
+        $this->data = &$this->data->{$this->getProperty()};
+        return $this;
+    }
+
+
+    protected function performNonExisting()
+    {
+        $result = false;
+        return $this->setResult($result);
+    }
+
+
+    protected function getProperty()
+    {
+        return $this
+            ->getReference()
+            ->getValue();
+    }
+}

…aluate/ReferenceWriteNonNumericIndex.php …ReferenceWriteAllowedNonNumericIndex.phpsrc/Pointer/Evaluate/ReferenceWriteNonNumericIndex.php renamed to src/Pointer/Evaluate/ReferenceWriteAllowedNonNumericIndex.php src/Pointer/Evaluate/ReferenceWriteNonNumericIndex.php renamed to src/Pointer/Evaluate/ReferenceWriteAllowedNonNumericIndex.php

@@ -2,7 +2,7 @@
 
 namespace Remorhaz\JSONPointer\Pointer\Evaluate;
 
-class ReferenceWriteNonNumericIndex extends ReferenceWrite
+class ReferenceWriteAllowedNonNumericIndex extends ReferenceWrite
 {
 
 

src/Pointer/Evaluate/ReferenceWriteNotAllowedNonNumericIndex.php

@@ -0,0 +1,37 @@
+<?php
+
+namespace Remorhaz\JSONPointer\Pointer\Evaluate;
+
+class ReferenceWriteNotAllowedNonNumericIndex extends ReferenceWrite
+{
+
+
+    protected function doesExist()
+    {
+        return false; // We don't allow it to exist :)
+    }
+
+
+    protected function performExisting()
+    {
+        throw new LogicException(
+            "Non-numeric index '{$this->getIndex()}' is not allowed to exist"
+        );
+    }
+
+
+    protected function performNonExisting()
+    {
+        throw new EvaluateException(
+            "Cannot write to non-numeric index '{$this->getIndex()}' in array"
+        );
+    }
+
+
+    protected function getIndex()
+    {
+        return $this
+            ->getReference()
+            ->getValue();
+    }
+}

src/Pointer/Evaluate/Test.php

@@ -17,34 +17,78 @@ protected function processCursor()
 
     protected function processNextArrayIndex(Reference $reference)
     {
-        $result = false;
-        return $this->setResult($result);
+        throw new LogicException("Deprecated method");
     }
 
 
     protected function processNonExistingObjectProperty(Reference $reference)
     {
-        $result = false;
-        return $this->setResult($result);
+        throw new LogicException("Deprecated method");
     }
 
 
     protected function processNonExistingArrayIndex(Reference $reference)
     {
-        $result = false;
-        return $this->setResult($result);
+        throw new LogicException("Deprecated method");
     }
 
 
     protected function processNotAllowedNonNumericArrayIndex(Reference $reference)
     {
-        $result = false;
-        return $this->setResult($result);
+        throw new LogicException("Deprecated method");
     }
-    
-    
+
+
+    /**
+     * @param Reference $reference
+     * @return ReferenceEvaluate
+     */
     protected function createReferenceEvaluate(Reference $reference)
     {
-        
+        if (is_object($this->cursor)) {
+            return ReferenceTestProperty::factory();
+        }
+        if (is_array($this->cursor)) {
+            switch ($reference->getType()) {
+                case $reference::TYPE_NEXT_INDEX:
+                    return ReferenceTestNextIndex::factory();
+
+                case $reference::TYPE_INDEX:
+                    return ReferenceTestNumericIndex::factory();
+
+                case $reference::TYPE_PROPERTY:
+                    return $this->nonNumericIndices
+                        ? ReferenceTestAllowedNonNumericIndex::factory()
+                        : ReferenceTestNotAllowedNonNumericIndex::factory();
+
+                default:
+                    throw new DomainException(
+                        "Failed to create test evaluator for reference of type {$reference->getType()}"
+                    );
+            }
+        }
+        throw new EvaluateException("Cannot test non-structured data by reference");
+    }
+
+
+    protected function processReferenceEvaluate(Reference $reference)
+    {
+        $this
+            ->setupReferenceEvaluate($reference)
+            ->getReferenceEvaluate()
+            ->perform();
+        $this->cursor = &$this
+            ->getReferenceEvaluate()
+            ->getData();
+        $isReferenceResultSet = $this
+            ->getReferenceEvaluate()
+            ->isResultSet();
+        if ($isReferenceResultSet) {
+            $referenceResult = &$this
+                ->getReferenceEvaluate()
+                ->getResult();
+            $this->setResult($referenceResult);
+        }
+        return $this;
     }
 }