SourceFileIterator::getIterator() - Code Metrics - Inspection of "Make test runner able to test directory, close #16" - hanneskod/readme-tester - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 92841c...9121c0 )

by Hannes

created 2017-02-14 21:30 UTC

SourceFileIterator::getIterator() A

↳ Parent: SourceFileIterator

Complexity

Conditions	4
Paths	4

Size

Total Lines	13
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	13
rs	9.2
c	0
b	0
f	0
cc	4
eloc	7
nc	4
nop	0

<?php

namespace hanneskod\readmetester;

/**
 * Iterate over markdown formatted source files
 */
class SourceFileIterator implements \IteratorAggregate
{
    /**
     * @var string Source file or directory name
     */
    private $filename;

    public function __construct(string $filename)
    {
        $this->filename = $filename;
    }

    public function getIterator(): \Traversable
    {
        if (is_file($this->filename)) {
            yield basename($this->filename) => $this->readFile($this->filename);
            return;
        }

        foreach (new \DirectoryIterator($this->filename) as $fileInfo) {
            if (in_array(strtolower($fileInfo->getExtension()), ['md', 'mdown', 'markdown'])) {
                yield $fileInfo->getFilename() => $this->readFile($fileInfo->getRealPath());
            }
        }
    }

    private function readFile(string $filename): string
    {
        if (!is_file($filename) || !is_readable($filename)) {
            throw new \Exception("Not able to read $filename");
        }

        return file_get_contents($filename);
    }
}


1			<?php
2
3			namespace hanneskod\readmetester;
4
5			/**
6			* Iterate over markdown formatted source files
7			*/
8			class SourceFileIterator implements \IteratorAggregate
9			{
10			/**
11			* @var string Source file or directory name
12			*/
13			private $filename;
14
15			public function __construct(string $filename)
16			{
17			$this->filename = $filename;
18			}
19
20			public function getIterator(): \Traversable
21			{
22			if (is_file($this->filename)) {
23			yield basename($this->filename) => $this->readFile($this->filename);
24			return;
25			}
26
27			foreach (new \DirectoryIterator($this->filename) as $fileInfo) {
28			if (in_array(strtolower($fileInfo->getExtension()), ['md', 'mdown', 'markdown'])) {
29			yield $fileInfo->getFilename() => $this->readFile($fileInfo->getRealPath());
30			}
31			}
32			}
33
34			private function readFile(string $filename): string
35			{
36			if (!is_file($filename) \|\| !is_readable($filename)) {
37			throw new \Exception("Not able to read $filename");
38			}
39
40			return file_get_contents($filename);
41			}
42			}
43

hanneskod / readme-tester

Push — master ( 92841c...9121c0 )

SourceFileIterator::getIterator() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like