Directory::__construct() - Code Metrics - Inspection of "init commit" - tigitz/php-spellchecker - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Push — master ( 01d6ae...9ca6d9 )

by Philippe

created 2019-02-04 00:17 UTC

Directory::__construct() A

↳ Parent: Directory

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	1

Importance

Changes

Metric	Value
cc	1
eloc	2
nc	1
nop	2
dl	0
loc	4
rs	10
c	0
b	0
f	0
ccs	3
cts	3
cp	1
crap	1

<?php

declare(strict_types=1);

namespace PhpSpellcheck\Source;

use PhpSpellcheck\Text;

class Directory implements SourceInterface
{
    /**
     * @var string
     */
    private $dirPath;

    /**
     * @var null|string
     */
    private $pattern;

    public function __construct(string $dirPath, ?string $pattern = null)
    {
        $this->dirPath = $dirPath;
        $this->pattern = $pattern;
    }

    /**
     * @return Text[]
     */
    public function getContents(): iterable
    {
        $filesInDir = new \RecursiveIteratorIterator(
            new \RecursiveDirectoryIterator(
                $this->dirPath,
                \FilesystemIterator::SKIP_DOTS | \FilesystemIterator::CURRENT_AS_PATHNAME
            ),
            \RecursiveIteratorIterator::SELF_FIRST
        );

        if ($this->pattern) {
            $filesInDir = new \RegexIterator($filesInDir, $this->pattern, \RegexIterator::GET_MATCH);
        }

        /** @var \SplFileInfo|string|array $file */
        foreach ($filesInDir as $file) {
            if (is_string($file)) {
                $file = new \SplFileInfo($file);
            } elseif (is_array($file)) {
                // When regex pattern is used, an array containing the file path in its first element is returned
                $file = new \SplFileInfo(current($file));
            }

            if (!$file->isDir()) {
                if ($file->getRealPath() !== false) {
                    yield current((new File($file->getRealPath()))->toTexts());

                }
            }
        }
    }

    public function toTexts(array $context): iterable
    {
        foreach ($this->getContents() as $text) {
            yield new Text(
                $text->getContent(),
                $text->getEncoding(),
                array_merge($text->getContext(), $context)
            );
        }
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace PhpSpellcheck\Source;
6
7		use PhpSpellcheck\Text;
8
9		class Directory implements SourceInterface
10		{
11		/**
12		* @var string
13		*/
14		private $dirPath;
15
16		/**
17		* @var null\|string
18		*/
19		private $pattern;
20
21	2	public function __construct(string $dirPath, ?string $pattern = null)
22		{
23	2	$this->dirPath = $dirPath;
24	2	$this->pattern = $pattern;
25	2	}
26
27		/**
28		* @return Text[]
29		*/
30	2	public function getContents(): iterable
31		{
32	2	$filesInDir = new \RecursiveIteratorIterator(
33	2	new \RecursiveDirectoryIterator(
34	2	$this->dirPath,
35	2	\FilesystemIterator::SKIP_DOTS \| \FilesystemIterator::CURRENT_AS_PATHNAME
36		),
37		\RecursiveIteratorIterator::SELF_FIRST
38	2	);
39	1
40		if ($this->pattern) {
41		$filesInDir = new \RegexIterator($filesInDir, $this->pattern, \RegexIterator::GET_MATCH);
42		}
43	2
44	2	/** @var \SplFileInfo\|string\|array $file */
45	1	foreach ($filesInDir as $file) {
46	1	if (is_string($file)) {
47		$file = new \SplFileInfo($file);
48	1	} elseif (is_array($file)) {
49		// When regex pattern is used, an array containing the file path in its first element is returned
50		$file = new \SplFileInfo(current($file));
51	2	}
52	2
53		if (!$file->isDir()) {
54		if ($file->getRealPath() !== false) {
55	2	yield current((new File($file->getRealPath()))->toTexts());
		0 ignored issues – show Bug Best Practice introduced 2019-02-04 09:12 UTC by Report Bug Copy Issue Report The expression `yield current(new PhpSpe...RealPath())->toTexts())` returns the type `Generator` which is incompatible with the documented return type `PhpSpellcheck\Text[]`. Loading history...
56		}
57	2	}
58		}
59	2	}
60	2
61	2	public function toTexts(array $context): iterable
62	2	{
63	2	foreach ($this->getContents() as $text) {
64		yield new Text(
65		$text->getContent(),
66	2	$text->getEncoding(),
67		array_merge($text->getContext(), $context)
68		);
69		}
70		}
71		}
72

tigitz / php-spellchecker

Push — master ( 01d6ae...9ca6d9 )

Directory::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like