RecursiveFileListing::isFileOk() - Code Metrics - Inspection of "upload complete directory" - CSFCloud/php-sdk - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 32cffc...62a894 )

by Bence

created 2019-02-01 21:33 UTC

RecursiveFileListing::isFileOk() A

↳ Parent: RecursiveFileListing

Complexity

Conditions	4
Paths	4

Size

Total Lines	12
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	20

Importance

Changes

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

<?php

namespace CSFCloud\Files;

use Exception;

class RecursiveFileListing {

    private $root_dir;
    private $files;
    private $filters = [];

    public function __construct (string $dir) {
        if (!is_dir($dir)) {
            throw new Exception("Directory required");
        }

        $this->root_dir = $dir;
    }

    public function addFilter (string $regex) {
        $this->filters[] = $regex;
    }

    public function clearFilters () {
        $this->filters = [];
    }

    public function scan () : array {
        $this->files = [];
        $this->recursiveScan($this->root_dir);
        sort($this->files);
        return $this->files;
    }

    private function recursiveScan (string $dir) {
        $files = scandir($dir);
        foreach ($files as $filename) {
            if ($filename !== "." && $filename !== "..") {
                $file_path = $dir . "/" . $filename;
                if (is_file($file_path) && $this->isFileOk($file_path)) {
                    $this->files[] = $file_path;
                } else if (is_dir($file_path)) {
                    $this->recursiveScan($file_path);
                }
            }
        }
    }

    private function isFileOk (string $path) : bool {
        if (count($this->filters) == 0) {
            return true;
        }

        foreach ($this->filters as $filter) {
            if (preg_match($filter, $path)) {
                return true;
            }
        }

        return false;
    }

}

1			<?php
2
3			namespace CSFCloud\Files;
4
5			use Exception;
6
7			class RecursiveFileListing {
8
9			private $root_dir;
10			private $files;
11			private $filters = [];
12
13			public function __construct (string $dir) {
14			if (!is_dir($dir)) {
15			throw new Exception("Directory required");
16			}
17
18			$this->root_dir = $dir;
19			}
20
21			public function addFilter (string $regex) {
22			$this->filters[] = $regex;
23			}
24
25			public function clearFilters () {
26			$this->filters = [];
27			}
28
29			public function scan () : array {
30			$this->files = [];
31			$this->recursiveScan($this->root_dir);
32			sort($this->files);
33			return $this->files;
34			}
35
36			private function recursiveScan (string $dir) {
37			$files = scandir($dir);
38			foreach ($files as $filename) {
39			if ($filename !== "." && $filename !== "..") {
40			$file_path = $dir . "/" . $filename;
41			if (is_file($file_path) && $this->isFileOk($file_path)) {
42			$this->files[] = $file_path;
43			} else if (is_dir($file_path)) {
44			$this->recursiveScan($file_path);
45			}
46			}
47			}
48			}
49
50			private function isFileOk (string $path) : bool {
51			if (count($this->filters) == 0) {
52			return true;
53			}
54
55			foreach ($this->filters as $filter) {
56			if (preg_match($filter, $path)) {
57			return true;
58			}
59			}
60
61			return false;
62			}
63
64			}

CSFCloud / php-sdk

Push — master ( 32cffc...62a894 )

RecursiveFileListing::isFileOk() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like