NetworkTrafficCollector::calculateNetworkThroughput()

Complexity

Conditions	9
Paths	22

Size

Total Lines	40
Code Lines	24

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	0

<?php
    function calculateNetworkThroughput($interface): float

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

    {
        $memory = Memory::getInstance();

        $file = new File();

        if (!$file->fileExists('/proc/net/dev')) return 0;

        $lines = $file->getContents('/proc/net/dev', true);
        foreach ($lines as $line) {
            if (strpos($line, $interface . ':') !== false) {
                $parts = preg_split('/\s+/', trim($line));
                $rx = (int)$parts[1];
                $tx = (int)$parts[9];
                $total = $rx + $tx;
                break;
            }
        }

        if (!isset($total)) return 0;

        $history = $memory->getData(self::MEMORY_KEY) ?? [];
        $lastTotal = end($history);

        if (!$lastTotal) $lastTotal = 0;

        if (!empty($history) && $total < $lastTotal) {

The variable $total does not seem to be defined for all execution paths leading up to this point.

            $memory->addData(self::MEMORY_KEY, $total);
            return $total;
        }

        if (!empty($history)) {
            $mb = $total - $lastTotal;
        } else {
            $mb = 0;
        }

        $memory->addData(self::MEMORY_KEY, $total);

        return $mb;
    }
}