Game::execute() - Code Metrics - Inspection of "Fix list command" - yiisoft/yii-console - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#183)

by Alexander

created 2023-05-28 15:21 UTC

Game::execute() A

↳ Parent: Game

Complexity

Conditions	4
Paths	4

Size

Total Lines	25
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	20

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	4
eloc	16
c	1
b	0
f	0
nc	4
nop	2
dl	0
loc	25
ccs	0
cts	16
cp	0
crap	20
rs	9.7333

<?php

declare(strict_types=1);

namespace Yiisoft\Yii\Console\Command;

use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Question\Question;
use Symfony\Component\Console\Style\SymfonyStyle;

final class Game extends Command
{
    protected static $defaultName = '|yii';
    protected static $defaultDescription = 'A Guessing Game';

    protected function execute(InputInterface $input, OutputInterface $output): int
    {
        $steps = 1;
        $number = random_int(0, 100);

        $io = new SymfonyStyle($input, $output);
        $io->title('Welcome to the Guessing Game!');


        $helper = $this->getHelper('question');
        $question = new Question('Please enter a number between 0 and 100: ');

        while (true) {
            $answer = (int) $helper->ask($input, $output, $question);

            if ($answer === $number) {
                $io->success('You win! You guessed the number in ' . $steps . ' steps.');
                return 0;
            }

            $steps++;
            if ($answer > $number) {
                $io->warning('Too high!');
            } else {
                $io->warning('Too low!');
            }
        }
interface ReturnsInt {
    public function returnsIntHinted(): int;
}

class MyClass implements ReturnsInt {
    public function returnsIntHinted(): int
    {
        if (foo()) {
            return 123;
        }
        // here: null is implicitly returned
    }
}
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Yiisoft\Yii\Console\Command;
6
7			use Symfony\Component\Console\Command\Command;
8			use Symfony\Component\Console\Input\InputInterface;
9			use Symfony\Component\Console\Output\OutputInterface;
10			use Symfony\Component\Console\Question\Question;
11			use Symfony\Component\Console\Style\SymfonyStyle;
12
13			final class Game extends Command
14			{
15			protected static $defaultName = '\|yii';
16			protected static $defaultDescription = 'A Guessing Game';
17
18			protected function execute(InputInterface $input, OutputInterface $output): int
19			{
20			$steps = 1;
21			$number = random_int(0, 100);
22
23			$io = new SymfonyStyle($input, $output);
24			$io->title('Welcome to the Guessing Game!');
25
26
27			$helper = $this->getHelper('question');
28			$question = new Question('Please enter a number between 0 and 100: ');
29
30			while (true) {
31			$answer = (int) $helper->ask($input, $output, $question);
32
33			if ($answer === $number) {
34			$io->success('You win! You guessed the number in ' . $steps . ' steps.');
35			return 0;
36			}
37
38			$steps++;
39			if ($answer > $number) {
40			$io->warning('Too high!');
41			} else {
42			$io->warning('Too low!');
43			}
44			}
			0 ignored issues – show Bug Best Practice introduced 2023-05-28 15:25 UTC by Report Bug Copy Issue Report In this branch, the function will implicitly return `null` which is incompatible with the type-hinted return `integer`. Consider adding a return statement or allowing `null` as return value. For hinted functions/methods where all return statements with the correct type are only reachable via conditions, ?null? gets implicitly returned which may be incompatible with the hinted type. Let?s take a look at an example: interface ReturnsInt { public function returnsIntHinted(): int; } class MyClass implements ReturnsInt { public function returnsIntHinted(): int { if (foo()) { return 123; } // here: null is implicitly returned } } Loading history...
45			}
46			}
47

yiisoft / yii-console

Pull Request — master (#183)

Game::execute() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like