ContainerAccessors

Complexity

Total Complexity

15

Size/Duplication

Total Lines	115
Duplicated Lines	9.57 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	0	Features	0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	setAccessors()	0	3	1
A	setDirect()	0	3	1
A	getDirect()	0	3	1
A	unsetDirect()	0	3	1
A	assignAccessor()	11	11	3
A	performMagic()	0	11	3
A	__set()	0	7	2
A	__get()	0	3	1
A	__unset()	0	3	1
A	__isset()	0	6	1

<?php

use \Common\ContainerMagic;

/**
 * Class ContainerAccessors
 *
 * Support accessors for properties: getter, setter, unsetter
 *
 * Below $that - is a shortcut for container object which will be passed to accessor
 *
 * Getter is a callable like
 *    function ($this) {}
 *
 * Setter is a callable like
 *    function ($that, $value)  {}
 *
 * Unsetter is a callable like
 *    function ($that) {}
 *
 * Use setDirect() and getDirect() methods to access same variable for setter/getter function!
 * If setter works with other object properties it needs an unsetter to handle clearProperties() method
 *
 */
class ContainerAccessors extends ContainerMagic {
  /**
   * Array of accessors - getters/setters/etc
   *
   * @var callable[][]
   */
  protected $accessors = array();

  /**
   * @param array $array
   */
  public function setAccessors($array) {
    $this->accessors = $array;
  }

  /**
   * Direct access to parent class setter
   *
   * @param string $name
   * @param mixed  $value
   *
   * @return mixed
   */
  public function setDirect($name, $value) {
    ContainerMagic::__set($name, $value);
  }

  /**
   * Direct access to parent class getter
   *
   * @param string $name
   *
   * @return mixed
   */
  public function getDirect($name) {
    return ContainerMagic::__get($name);
  }

  /**
   * Direct access to parent class unsetter
   *
   * @param string $name
   */
  public function unsetDirect($name) {
    ContainerMagic::__unset($name);
  }

  /**
   * Assign accessor to a named variable
   *
   * Different accessors have different signatures - you should look carefully before assigning accessor
   *
   * @param string   $varName
   * @param string   $type - getter/setter/importer/exporter/etc
   * @param callable $callable
   *
   * @throws Exception
   */
  public function assignAccessor($varName, $type, $callable) {

This method seems to be duplicated in your project.

    if (empty($callable)) {
      return;
    }

    if (is_callable($callable)) {
      $this->accessors[$varName][$type] = $callable;
    } else {
      throw new Exception('Error assigning callable in ' . get_called_class() . '! Callable typed [' . $type . '] is not a callable or not accessible in the scope');
    }
  }

  /**
   * Performs $processor operation on property with specified name
   *
   * @param string     $processor
   * @param string     $name
   * @param null|mixed $value
   *
   * @return mixed
   */
  protected function performMagic($processor, $name, $value = null) {
    if (
      !empty($this->accessors[$name][$processor])
      &&
      is_callable($this->accessors[$name][$processor])
    ) {
      return call_user_func($this->accessors[$name][$processor], $this, $value);

    } else {
      return parent::$processor($name, $value);
    }
  }

  public function __set($name, $value) {
    if (is_callable($value)) {
      $this->accessors[$name][P_CONTAINER_GET] = $value;
    } else {
      $this->performMagic(P_CONTAINER_SET, $name, $value);
    }
  }

  public function __get($name) {
    return $this->performMagic(P_CONTAINER_GET, $name, null);
  }

  public function __unset($name) {
    $this->performMagic(P_CONTAINER_UNSET, $name, null);
  }

  public function __isset($name) {
    // TODO - or here already can isset($this->name) ????

42% of this comment could be valid code. Did you maybe forget this after debugging?

    $value = $this->$name;

    return isset($value);
  }

}