ListenCommand::handle() - Code Metrics - Inspection of "Style/rename hard disk to storage" - MohsenAbrishami/stethoscope - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — main (#145)

by mohsen

created 2024-04-16 07:09 UTC

ListenCommand::handle() B

↳ Parent: Project

Complexity

Conditions	11
Paths	32

Size

Total Lines	37
Code Lines	19

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	1	Features	0

Metric	Value
cc	11
eloc	19
c	2
b	1
f	0
nc	32
nop	0
dl	0
loc	37
rs	7.3166

How to fix Complexity

<?php

namespace MohsenAbrishami\Stethoscope\Commands;

use Illuminate\Console\Command;
use Illuminate\Support\Facades\Storage;
use MohsenAbrishami\Stethoscope\Services\Cpu;
use MohsenAbrishami\Stethoscope\Services\storage;

use MohsenAbrishami\Stethoscope\Services\Memory;
use MohsenAbrishami\Stethoscope\Services\Network;
use MohsenAbrishami\Stethoscope\Services\WebServer;
use MohsenAbrishami\Stethoscope\Traits\MessageCreatorTrait;

class ListenCommand extends Command
{
    use MessageCreatorTrait;

    public function __construct(Cpu $cpu, Memory $memory, Network $network, WebServer $webServer, storage $storage)
    {
        parent::__construct();

        $this->cpu = $cpu;
        $this->memory = $memory;
        $this->network = $network;
        $this->webServer = $webServer;
        $this->storage = $storage;

        $this->storage = Storage::disk(config('stethoscope.storage.driver'));
    }

    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'stethoscope:listen {resources?*}';

    /**
     * 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()
    {
        $resources = collect($this->argument('resources'));

        $resourcesIsEmpty = $resources->isEmpty();

        $this->line(
            $this->timeMessage()
        );

        if ($resources->contains('cpu') || $resourcesIsEmpty) {
            $this->info(
                $this->cpuMessage($this->cpu->check())
            );
        }

        if ($resources->contains('memory') || $resourcesIsEmpty) {
            $this->info(
                $this->memoryMessage($this->memory->check())
            );
        }

        if ($resources->contains('network') || $resourcesIsEmpty) {
            $this->info(
                $this->networkMessage($this->network->check())
            );
        }

        if ($resources->contains('web-server') || $resourcesIsEmpty) {
            $this->info(
                $this->webServerMessage($this->webServer->check())
            );
        }

        if ($resources->contains('hdd') || $resourcesIsEmpty) {
            $this->info(
                $this->storageMessage($this->storage->check())
            );
        }
    }
}


1			<?php
2
3			namespace MohsenAbrishami\Stethoscope\Commands;
4
5			use Illuminate\Console\Command;
6			use Illuminate\Support\Facades\Storage;
7			use MohsenAbrishami\Stethoscope\Services\Cpu;
8			use MohsenAbrishami\Stethoscope\Services\storage;
			0 ignored issues – show Bug introduced 2024-04-16 07:27 UTC by Report Bug Copy Issue Report A parse error occurred: Cannot use MohsenAbrishami\Stethoscope\Services\storage as storage because the name is already in use Loading history...
9			use MohsenAbrishami\Stethoscope\Services\Memory;
10			use MohsenAbrishami\Stethoscope\Services\Network;
11			use MohsenAbrishami\Stethoscope\Services\WebServer;
12			use MohsenAbrishami\Stethoscope\Traits\MessageCreatorTrait;
13
14			class ListenCommand extends Command
15			{
16			use MessageCreatorTrait;
17
18			public function __construct(Cpu $cpu, Memory $memory, Network $network, WebServer $webServer, storage $storage)
19			{
20			parent::__construct();
21
22			$this->cpu = $cpu;
23			$this->memory = $memory;
24			$this->network = $network;
25			$this->webServer = $webServer;
26			$this->storage = $storage;
27
28			$this->storage = Storage::disk(config('stethoscope.storage.driver'));
29			}
30
31			/**
32			* The name and signature of the console command.
33			*
34			* @var string
35			*/
36			protected $signature = 'stethoscope:listen {resources?*}';
37
38			/**
39			* The console command description.
40			*
41			* @var string
42			*/
43			protected $description = 'monitor memory usage, cpu usage, network connection and nginx status';
44
45			/**
46			* Execute the console command.
47			*
48			* @return int
49			*/
50			public function handle()
51			{
52			$resources = collect($this->argument('resources'));
53
54			$resourcesIsEmpty = $resources->isEmpty();
55
56			$this->line(
57			$this->timeMessage()
58			);
59
60			if ($resources->contains('cpu') \|\| $resourcesIsEmpty) {
61			$this->info(
62			$this->cpuMessage($this->cpu->check())
63			);
64			}
65
66			if ($resources->contains('memory') \|\| $resourcesIsEmpty) {
67			$this->info(
68			$this->memoryMessage($this->memory->check())
69			);
70			}
71
72			if ($resources->contains('network') \|\| $resourcesIsEmpty) {
73			$this->info(
74			$this->networkMessage($this->network->check())
75			);
76			}
77
78			if ($resources->contains('web-server') \|\| $resourcesIsEmpty) {
79			$this->info(
80			$this->webServerMessage($this->webServer->check())
81			);
82			}
83
84			if ($resources->contains('hdd') \|\| $resourcesIsEmpty) {
85			$this->info(
86			$this->storageMessage($this->storage->check())
87			);
88			}
89			}
90			}
91

MohsenAbrishami / stethoscope

Pull Request — main (#145)

ListenCommand::handle() B

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like