MohsenAbrishami\Stethoscope\Commands\MonitorCommand

Complexity

Total Complexity

13

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Importance

Changes	3
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	1
C	handle()	0	29	12

<?php

namespace MohsenAbrishami\Stethoscope\Commands;

use Illuminate\Console\Command;
use MohsenAbrishami\Stethoscope\Events\TroubleOccurred;
use MohsenAbrishami\Stethoscope\LogRecord\Facades\Record;
use MohsenAbrishami\Stethoscope\Services\Cpu;
use MohsenAbrishami\Stethoscope\Services\HardDisk;
use MohsenAbrishami\Stethoscope\Services\Memory;
use MohsenAbrishami\Stethoscope\Services\Network;
use MohsenAbrishami\Stethoscope\Services\WebServer;

class MonitorCommand extends Command
{
    public function __construct(Cpu $cpu, Memory $memory, Network $network, WebServer $webServer, HardDisk $hardDisk)
    {
        parent::__construct();

        $this->cpu = $cpu;

The property cpu does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->memory = $memory;

The property memory does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->network = $network;

The property network does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->webServer = $webServer;

The property webServer does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->hardDisk = $hardDisk;

The property hardDisk does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }


    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'stethoscope:monitor';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'monitor memory usage, cpu usage, network connection and nginx status';

    /**
     * Execute the console command.
     *
     * @return int
     */
    public function handle()
    {
        $resourceReports = [];

        $cpuUsage = $this->cpu->check();
        $memoryUsage = $this->memory->check();
        $networkStatus = $this->network->check();
        $hardDiskFreeSpace = $this->hardDisk->check();
        $webServerStatuses = $this->webServer->check();

        if (config('stethoscope.monitorable_resources.cpu') && $cpuUsage > config(('stethoscope.thresholds.cpu')))
            $resourceReports['cpu'] = $cpuUsage;

        if ($memoryUsage > config(('stethoscope.thresholds.memory')) && config('stethoscope.monitorable_resources.memory'))
            $resourceReports['memory'] = $memoryUsage;

        if ($networkStatus == 'false' && config('stethoscope.monitorable_resources.network'))
            $resourceReports['network'] = $networkStatus;

        if ($hardDiskFreeSpace < config(('stethoscope.thresholds.hard_disk')) && config('stethoscope.monitorable_resources.hard_disk'))
            $resourceReports['hardDisk'] = $hardDiskFreeSpace;

        if ($webServerStatuses != 'active' && config('stethoscope.monitorable_resources.web_server'))
            $resourceReports['webServer'] = $webServerStatuses;

        Record::record($resourceReports);

The call to MohsenAbrishami\Stethoscope\LogRecord\Facades\Record::record() has too few arguments starting with memoryUsage.

$resourceReports of type array is incompatible with the type integer expected by parameter $cpuUsage of MohsenAbrishami\Stethoscope\LogRecord\Facades\Record::record().

        if (!empty($resourceReports))
            TroubleOccurred::dispatch($resourceReports);
    }
}