Petrica\StatsdSystem\Gauge\CpuAverageGauge

Complexity

Total Complexity

6

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes	3
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getSamplingPeriod()	0	4	1
A	getPath()	0	4	1
A	getValue()	0	11	3
A	getLoadAverage()	0	4	1

<?php
/**
 * Created by PhpStorm.
 * User: Petrica
 * Date: 3/22/2016
 * Time: 23:03
 */
namespace Petrica\StatsdSystem\Gauge;

class CpuAverageGauge implements GaugeInterface
{
    /**
     * Report every 60 seconds
     * Load average is calculated every 1 minute
     *
     * @return int
     */
    public function getSamplingPeriod()
    {
        return 60;
    }

    /**
     * CPU average load path reported to statsd
     *
     * @return string
     */
    public function getPath()
    {
        return 'cpu.load.average';
    }

    /**
     * {@inheritdoc}
     */
    public function getValue()
    {
        $load = $this->getLoadAverage();

        $value = null;
        if ($load && isset($load[0])) {

The expression $load of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $value = $load[0];
        }

        return $value;
    }

    /**
     * Return CPU load average
     *
     * @return array
     */
    protected function getLoadAverage()
    {
        return sys_getloadavg();
    }
}