Fyuze\Debug\Timer

Complexity

Total Complexity

7

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getTimers()	0	3	1
A	stop()	0	13	4
A	start()	0	9	1
A	transaction()	0	7	1

<?php
namespace Fyuze\Debug;

class Timer
{
    /**
     * @var
     */
    protected $timers;

    protected $current;

    /**
     * @param string $name
     * @return float
     */
    public function start($name = 'default')
    {
        $this->current = $name;

        $start = microtime(true);

        $this->timers[$name]['start'] = $start;

        return $start;

The expression return $start also could return the type string which is incompatible with the documented return type double.

    }

    /**
     * @param null $name

Are you sure the doc-type for parameter $name is correct as it would always require null to be passed?

     * @return array
     */
    public function stop($name = null)
    {
        $key = ($name !== null || ($name && array_key_exists($name, $this->timers))) ? $name : 'default';

$name is of type null, thus it always evaluated to false.

$name of type void is incompatible with the type integer|string expected by parameter $key of array_key_exists().

        $this->timers[$key]['stop'] = microtime(true);

        /** @var $start float */
        /** @var $stop float */
        extract($this->timers[$key]);

        $this->timers[$key]['duration'] = $stop - $start;

        return $this->timers[$key];
    }

    /**
     *
     */
    public function getTimers()
    {
        return $this->timers;
    }

    /**
     * @param \Closure $closure
     * @return array
     */
    public function transaction(\Closure $closure)
    {
        $this->start();

        $closure();

        return $this->stop();
    }
}