Innmind\Server\Status\Servers\Decorator\CacheMemory

Complexity

Total Complexity

9

Size/Duplication

Total Lines	64
Duplicated Lines	0 %

Test Coverage

Coverage

91.67%

Importance

Changes	2
Bugs	0	Features	2

7 Methods

Rating	Name	Duplication	Size	Complexity
A	cpu()	0	3	1
A	tmp()	0	3	1
A	memory()	0	18	3
A	loadAverage()	0	3	1
A	disk()	0	3	1
A	__construct()	0	8	1
A	processes()	0	3	1

<?php
declare(strict_types = 1);

namespace Innmind\Server\Status\Servers\Decorator;

use Innmind\Server\Status\{
    Server,
    Server\Cpu,
    Server\Memory,
    Server\Processes,
    Server\LoadAverage,
    Server\Disk,
};
use Innmind\TimeContinuum\{
    Clock,
    ElapsedPeriod,
    PointInTime,
};
use Innmind\Url\Path;

final class CacheMemory implements Server
{
    private Server $server;
    private Clock $clock;
    private ElapsedPeriod $threshold;
    private ?PointInTime $cachedAt = null;
    private ?Memory $data = null;

    public function __construct(
        Server $server,
        Clock $clock,
        ElapsedPeriod $threshold
    ) {
        $this->server = $server;
        $this->clock = $clock;
        $this->threshold = $threshold;
    }

    public function cpu(): Cpu
    {
        return $this->server->cpu();
    }

    /**
     * @psalm-suppress InvalidNullableReturnType
     */
    public function memory(): Memory
    {
        $now = $this->clock->now();

        if (
            $this->cachedAt &&
            $this->threshold->longerThan(
                $now->elapsedSince($this->cachedAt)
            )
        ) {
            /** @psalm-suppress NullableReturnStatement */
            return $this->data;

The expression return $this->data could return the type null which is incompatible with the type-hinted return Innmind\Server\Status\Server\Memory. Consider adding an additional type-check to rule them out.

        }

        $this->data = $this->server->memory();
        $this->cachedAt = $now;

        return $this->data;
    }

    public function processes(): Processes
    {
        return $this->server->processes();
    }

    public function loadAverage(): LoadAverage
    {
        return $this->server->loadAverage();
    }

    public function disk(): Disk
    {
        return $this->server->disk();
    }

    public function tmp(): Path
    {
        return $this->server->tmp();
    }
}