Psecio\Invoke\MatchInstance

Complexity

Total Complexity

9

Size/Duplication

Total Lines	76
Duplicated Lines	10.53 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	setConfig()	0	4	1
A	getConfig()	8	8	3
A	getError()	0	11	4
	evaluate()	0	1	?

<?php

namespace Psecio\Invoke;

abstract class MatchInstance
{
	/**
	 * Current configuration details
	 * @var array
	 */
	protected $config = array();

	/**
	 * Flag to negate the check
	 * @var boolean
	 */
	protected $negate = false;

	/**
	 * Init the object with the given confgiuration and negation flag
	 *
	 * @param array $config Configuration information
	 * @param boolean $negate Negation flag, default false [optional]
	 */
	public function __construct(array $config = array(), $negate = false)

The parameter $negate is not used and could be removed.

	{
		$this->setConfig($config);
	}

	/**
	 * Set the current configuration values
	 *
	 * @param array $config Configuration information
	 */
	public function setConfig(array $config)
	{
		$this->config = $config;
	}

	/**
	 * Get the current configuration, or if a key is provided
	 * 	try to locate that one value
	 *
	 * @param string $key Key to locate [optional]
	 * @return mixed Either entire config or key is found
	 */
	public function getConfig($key = null)

This method seems to be duplicated in your project.

	{
		if ($key !== null) {
			return (isset($this->config[$key])) ? $this->config[$key] : null;
		} else {
			return $this->config;
		}
	}

	/**
	 * Get the current formatted error message
	 *
	 * @return string Error message contents
	 */
	public function getError()
	{
		$config = $this->getConfig()['data'];
		$message = (isset($this->error)) ? $this->error : 'There was an error on match for '.__CLASS__;

The property error does not exist. Did you maybe forget to declare it?

		if (strpos($message, ':data') !== false) {
			$data = '';

$data is not used, you could remove the assignment.

			$data = (is_array($config)) ? '['.implode(',', array_values($config)).']' : $config;
			$message = str_replace(':data', $data, $message);
		}
		return $message;
	}

	/**
	 * Evaluate the match
	 *
	 * @param \Psecio\Invoke\Data $data Object instance
	 * @return boolean Result of evaluation
	 */
	abstract public function evaluate($data);
}