DebugArray

Complexity

Total Complexity

7

Size/Duplication

Total Lines	56
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes	1
Bugs	0	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
A	ArrayDebugger::__construct()	0	8	1
A	ArrayDebugger::sendLog()	0	10	1
A	ArrayDebugger::setLogger()	0	3	1
A	ArrayDebugger::offsetGet()	0	5	1
A	ArrayDebugger::offsetSet()	0	5	1
A	ArrayDebugger::offsetExists()	0	5	1

<?php 
namespace PHP;

class ArrayDebugger extends \ArrayObject { 

  const TYPE_GET    = 'GET';
  const TYPE_SET    = 'SET';
  const TYPE_EXISTS = 'EXISTS';
  const TYPE_UNSET  = 'UNSET';

  private $logger;

  public function __construct() {

    $this->logger = function($data_log) {
      return dump($data_log);
    };

    return call_user_func_array('parent::__construct', func_get_args()); 

Constructors do not have meaningful return values, anything that is returned from here is discarded. Are you sure this is correct?

  }

  public function sendLog($type, $key, $value = null, array $backtrace ) {

    $logger           = $this->logger;
    $retorno['type']  = $type;  

$retorno was never initialized. Although not strictly required by PHP, it is generally a good practice to add $retorno = array(); before regardless.

    $retorno['key']   = $key;
    $retorno['value'] = $value;
    $retorno['file']  = $backtrace[0]['file'];
    $retorno['line']  = $backtrace[0]['line'];
    return $logger($retorno);
  }

  public function setLogger(\Closure $logger) {
    $this->logger = $logger;
  }

  public function offsetGet($name) { 

    $this->sendLog(static::TYPE_GET, $name, null, debug_backtrace());
    return call_user_func_array('parent::offsetGet', func_get_args()); 
  } 

  public function offsetSet($name, $value) { 

    $this->sendLog(static::TYPE_SET, $name, $value, debug_backtrace());
    return call_user_func_array('parent::offsetSet', func_get_args()); 
  } 

  public function offsetExists($name) { 

    $this->sendLog(static::TYPE_EXISTS, $name, null, debug_backtrace());
    return call_user_func_array('parent::offsetExists', func_get_args()); 
  } 

  public function offsetUnset($name) { 

    $this->sendLog(static::TYPE_UNSET, $name, null, debug_backtrace());
    return call_user_func_array('parent::offsetUnset', func_get_args()); 
  } 
}