Skip to content

Commit e80bad7

Browse files
committed
Minor reference evaluator refactoring
1 parent d78650e commit e80bad7

File tree

4 files changed

+26
-24
lines changed

4 files changed

+26
-24
lines changed

src/Pointer/Evaluate/LocatorEvaluate.php

Lines changed: 4 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -256,25 +256,12 @@ protected function setupReferenceEvaluateFactory(Reference $reference)
256256
}
257257

258258

259-
/**
260-
* @param Reference $reference
261-
* @return ReferenceEvaluate
262-
*/
263-
protected function createReferenceEvaluate(Reference $reference)
259+
protected function setEvaluateForReference(Reference $reference)
264260
{
265-
return $this
261+
$this->referenceEvaluate = $this
266262
->setupReferenceEvaluateFactory($reference)
267263
->getReferenceEvaluateFactory()
268-
->createReferenceEvaluate();
269-
}
270-
271-
272-
protected function setupReferenceEvaluate(Reference $reference)
273-
{
274-
$this->referenceEvaluate = $this
275-
->createReferenceEvaluate($reference)
276-
->setReference($reference)
277-
->setDataCursor($this->dataCursor);
264+
->createEvaluate();
278265
return $this;
279266
}
280267

@@ -315,7 +302,7 @@ protected function processReference(Reference $reference)
315302
protected function processReferenceEvaluate(Reference $reference)
316303
{
317304
$this
318-
->setupReferenceEvaluate($reference)
305+
->setEvaluateForReference($reference)
319306
->getReferenceEvaluate()
320307
->perform();
321308
$this->dataCursor = &$this

src/Pointer/Evaluate/LocatorWrite.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -70,9 +70,9 @@ protected function createReferenceEvaluateFactory()
7070
}
7171

7272

73-
protected function setupReferenceEvaluate(Reference $reference)
73+
protected function setEvaluateForReference(Reference $reference)
7474
{
75-
$referenceEvaluate = parent::setupReferenceEvaluate($reference)
75+
$referenceEvaluate = parent::setEvaluateForReference($reference)
7676
->getReferenceEvaluate();
7777
if ($referenceEvaluate instanceof ReferenceWrite) {
7878
$referenceEvaluate->setValue($this->getValue());

src/Pointer/Evaluate/ReferenceAdvanceable.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ abstract class ReferenceAdvanceable extends ReferenceEvaluate
88
/**
99
* @var Advancer|null
1010
*/
11-
protected $advanceCursor;
11+
protected $advancer;
1212

1313

1414
/**
@@ -23,13 +23,13 @@ abstract protected function performNonExisting();
2323
*/
2424
protected function getAdvancer()
2525
{
26-
if (null === $this->advanceCursor) {
27-
$this->advanceCursor = $this
26+
if (null === $this->advancer) {
27+
$this->advancer = $this
2828
->createAdvancer()
2929
->setReference($this->getReference())
3030
->setDataCursor($this->getDataCursor());
3131
}
32-
return $this->advanceCursor;
32+
return $this->advancer;
3333
}
3434

3535

src/Pointer/Evaluate/ReferenceEvaluateFactory.php

Lines changed: 16 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,22 @@ public function forbidNonNumericIndices()
9090
}
9191

9292

93-
public function createReferenceEvaluate()
93+
/**
94+
* @return ReferenceEvaluate
95+
*/
96+
public function createEvaluate()
97+
{
98+
return $this
99+
->doCreate()
100+
->setReference($this->getReference())
101+
->setDataCursor($this->getDataCursor());
102+
}
103+
104+
105+
/**
106+
* @return ReferenceEvaluate
107+
*/
108+
protected function doCreate()
94109
{
95110
if (is_object($this->dataCursor)) {
96111
return $this->createProperty();

0 commit comments

Comments
 (0)