Transphporm\Parser\ValueResult

Complexity

Total Complexity

20

Size/Duplication

Total Lines	89
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	2
Bugs	0	Features	0

15 Methods

Rating	Name	Duplication	Size	Complexity
A	arg()	0	3	1
A	not()	0	3	1
A	setMode()	0	3	1
A	getMode()	0	3	1
A	getResult()	0	3	1
A	pop()	0	3	1
A	processValue()	0	17	4
A	concat()	0	3	1
A	equals()	0	3	1
A	add()	0	3	1
A	sub()	0	3	1
A	mult()	0	3	1
A	div()	0	3	1
A	write()	0	5	3
A	clear()	0	3	1

<?php
/* @description     Transformation Style Sheets - Revolutionising PHP templating    *
 * @author          Tom Butler tom@r.je                                             *
 * @copyright       2015 Tom Butler <tom@r.je> | https://r.je/                      *
 * @license         http://www.opensource.org/licenses/bsd-license.php  BSD License *
 * @version         1.0                                                             */
namespace Transphporm\Parser;
class ValueResult {
	private $result = [];
	/*
		The next operation to perform. Will be one of the following:
			ARG - A new value e.g,  "a","b"  becomes ["a", "b"]
			CONCAT - Concat onto the current arg e.g "a" + "b" becomes ["ab"]
			NOT - Boolean operation "a" != "b" becomes [true]
			EQUALS - Boolean operation "a" = "b" becomes [false]
	*/
	private $mode = Tokenizer::ARG;

	//Processes $newValue using $mode. Either concats to the current argument, adds a new argument
	//Or usess the two arguments for a boolean comparison
	public function processValue($newValue) {
		$funcs = [
			Tokenizer::ARG => 'arg',
			Tokenizer::CONCAT => 'concat',
			Tokenizer::EQUALS => 'equals',
			Tokenizer::NOT => 'not',
			Tokenizer::SUBTRACT => 'sub',
			Tokenizer::MULTIPLY => 'mult',
			Tokenizer::DIVIDE => 'div'
		];

		if ($funcs[$this->mode] === 'concat' && is_numeric($newValue)
			&& is_numeric($this->result[count($this->result)-1]))
			$this->add($newValue);
		else
			$this->{$funcs[$this->mode]}($newValue);
	}

	public function arg($value) {
		$this->result[] = $value;
	}

	public function concat($value) {
		$this->result[count($this->result)-1] .= $value;
	}

	public function not($value) {
		$this->result[count($this->result)-1] = $this->result[count($this->result)-1] != $value;
	}

	public function equals($value) {
		$this->result[count($this->result)-1] = $this->result[count($this->result)-1] == $value;
	}

	public function add($value) {
		$this->result[count($this->result)-1] += $value;
	}

	public function sub($value) {
		$this->result[count($this->result)-1] -= $value;
	}

	public function mult($value) {
		$this->result[count($this->result)-1] *= $value;
	}

	public function div($value) {
		$this->result[count($this->result)-1] /= $value;
	}

	public function setMode($mode) {
		$this->mode = $mode;
	}

	public function getMode() {
		return $this->mode;
	}

	public function getResult() {
		return $this->result;
	}

	public function pop() {
		return array_pop($this->result);
	}

	public function write($index, $value, $allowNull = false) {
		if ($value !== null || $allowNull == true) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

			$this->result[$index] = $value;
		}
	}

	public function clear() {
		$this->result = [];
	}
}