Startwind\Inventorio\Collector\System\General\ConfigurationCollector

Complexity

Total Complexity

13

Size/Duplication

Total Lines	57
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getDiskSize()	0	4	1
A	getCpuCount()	0	9	4
A	collect()	0	6	1
A	getMemorySize()	0	16	6
A	getIdentifier()	0	3	1

<?php

namespace Startwind\Inventorio\Collector\System\General;

use Startwind\Inventorio\Collector\Collector;

class ConfigurationCollector implements Collector
{
    protected const COLLECTION_IDENTIFIER = 'SystemConfigurationCollector';

    /**
     * @inheritDoc
     */
    public function getIdentifier(): string
    {
        return self::COLLECTION_IDENTIFIER;
    }

    /**
     * @inheritDoc
     */
    public function collect(): array
    {
        return [
            'cpu' => $this->getCpuCount(),
            'memory' => $this->getMemorySize(),
            'disk' => $this->getDiskSize()
        ];
    }

    private function getCpuCount(): int
    {
        $os = PHP_OS_FAMILY;
        if ($os === 'Windows') {

The condition $os === 'Windows' is always false.

            return (int)getenv("NUMBER_OF_PROCESSORS");
        } elseif ($os === 'Darwin' || $os === 'Linux') {

The condition $os === 'Linux' is always true.

            return (int)shell_exec("getconf _NPROCESSORS_ONLN");
        }
        return 0;
    }

    private function getMemorySize(): string
    {
        $os = PHP_OS_FAMILY;
        if ($os === 'Windows') {

The condition $os === 'Windows' is always false.

            $output = shell_exec("wmic computersystem get TotalPhysicalMemory");
            preg_match("/\d+/", $output, $matches);
            return isset($matches[0]) ? round($matches[0] / 1024 / 1024, 2) . " MB" : "Nicht verfügbar";
        } elseif ($os === 'Darwin') {

The condition $os === 'Darwin' is always false.

            $memBytes = trim(shell_exec("sysctl -n hw.memsize"));
            return round($memBytes / 1024 / 1024, 2) . " MB";
        } elseif ($os === 'Linux') {

The condition $os === 'Linux' is always true.

            $meminfo = file_get_contents("/proc/meminfo");
            preg_match("/MemTotal:\s+(\d+)\skB/", $meminfo, $matches);
            return isset($matches[1]) ? round($matches[1] / 1024, 2) . " MB" : "Nicht verfügbar";
        }
        return 0;
    }

    function getDiskSize(): string

It is generally recommended to explicitly declare the visibility for methods.

    {
        $bytes = disk_total_space("/");
        return round($bytes / (1024 ** 3), 2) . " GB";
    }

}