Startwind\Inventorio\Metrics\Memory\Memory

Complexity

Total Complexity

14

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	getInstance()	0	7	2
A	__construct()	0	2	1
A	addData()	0	11	3
A	getDataSet()	0	3	1
A	getData()	0	3	1
A	addDataSet()	0	4	2
A	getNumberOfGreaterThan()	0	11	4

<?php

namespace Startwind\Inventorio\Metrics\Memory;
/**
 * @property array $data
 */
class Memory
{
    private const MEMORY_SIZE = 12; // one hour if every 5 minutes taken

    private static ?Memory $instance = null;

    private array $data = [];

    static public function getInstance(): Memory
    {
        if (self::$instance === null) {
            self::$instance = new self();
        }

        return self::$instance;

The expression return self::instance could return the type null which is incompatible with the type-hinted return Startwind\Inventorio\Metrics\Memory\Memory. Consider adding an additional type-check to rule them out.

    }

    private function __construct()
    {
    }

    public function addData(string $key, float $value): void
    {
        if (!array_key_exists($key, $this->data)) {
            $this->data[$key] = [];
        }

        if (count($this->data[$key]) >= self::MEMORY_SIZE) {
            array_shift($this->data[$key]);
        }

        $this->data[$key][] = $value;
    }

    public function getData(string $key): float
    {
        return $this->data[$key] ?? -1;
    }

    public function addDataSet(array $dataset): void
    {
        foreach ($dataset as $key => $value) {
            $this->addData($key, $value);
        }
    }

    public function getNumberOfGreaterThan(string $key, int $threshold): int
    {
        if (!array_key_exists($key, $this->data)) return 0;

        $count = 0;

        foreach ($this->data[$key] as $value) {
            if ($value >= $threshold) $count++;
        }

        return $count;
    }

    public function getDataSet(): array
    {
        return $this->data;
    }
}