Skippable::shouldProcess() - Code Metrics - Inspection of "More refactoring." - gnutix/videos-downloader - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( b580f7...9f1ea2 )

by Dorian

created 2018-05-17 12:18 UTC

Skippable::shouldProcess() B

↳ Parent: Skippable

Complexity

Conditions	5
Paths	3

Size

Total Lines	18
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	5
eloc	9
nc	3
nop	4
dl	0
loc	18
rs	8.8571
c	0
b	0
f	0

<?php declare(strict_types=1);

namespace App\UI;

use App\Collection\Collection;

trait Skippable
{
    /**
     * @param \App\UI\UserInterface $ui
     *
     * @return bool
     */
    protected function skip(UserInterface $ui): bool
    {
        if ($ui->isDryRun()) {
            $ui->writeln('<info>[DRY-RUN]</info> Not doing anything...'.PHP_EOL);

            return true;
        }

        return false;
    }

    /**
     * @param \App\UI\UserInterface $ui
     * @param \App\Collection\Collection $collection
     * @param string $message
     * @param string $action
     *
     * @return bool
     */
    private function shouldProcess(UserInterface $ui, Collection $collection, string $message, string $action): bool
    {
        if ($collection->isEmpty()) {
            $ui->writeln(sprintf($ui->indent().'<comment>Nothing to %s.</comment>'.PHP_EOL, $action));

            return false;
        }

        $ui->writeln($message);

        $ui->write($ui->indent());
        if ($this->skip($ui) || !$ui->confirm()) {
            $ui->writeln(($ui->isDryRun() ? '' : PHP_EOL).'<info>Done.</info>'.PHP_EOL);

            return false;
        }

        return true;
    }
}


1			<?php declare(strict_types=1);
2
3			namespace App\UI;
4
5			use App\Collection\Collection;
6
7			trait Skippable
8			{
9			/**
10			* @param \App\UI\UserInterface $ui
11			*
12			* @return bool
13			*/
14			protected function skip(UserInterface $ui): bool
15			{
16			if ($ui->isDryRun()) {
17			$ui->writeln('<info>[DRY-RUN]</info> Not doing anything...'.PHP_EOL);
18
19			return true;
20			}
21
22			return false;
23			}
24
25			/**
26			* @param \App\UI\UserInterface $ui
27			* @param \App\Collection\Collection $collection
28			* @param string $message
29			* @param string $action
30			*
31			* @return bool
32			*/
33			private function shouldProcess(UserInterface $ui, Collection $collection, string $message, string $action): bool
34			{
35			if ($collection->isEmpty()) {
36			$ui->writeln(sprintf($ui->indent().'<comment>Nothing to %s.</comment>'.PHP_EOL, $action));
37
38			return false;
39			}
40
41			$ui->writeln($message);
42
43			$ui->write($ui->indent());
44			if ($this->skip($ui) \|\| !$ui->confirm()) {
45			$ui->writeln(($ui->isDryRun() ? '' : PHP_EOL).'<info>Done.</info>'.PHP_EOL);
46
47			return false;
48			}
49
50			return true;
51			}
52			}
53

gnutix / videos-downloader

Push — master ( b580f7...9f1ea2 )

Skippable::shouldProcess() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like