AbstractFolderExtrator::getFiles() - Code Metrics - Inspection of "#2 : YamlEntityExtrator" - smartbooster/etl-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#4)

by Nicolas

created 2018-11-19 11:20 UTC

AbstractFolderExtrator::getFiles() A

↳ Parent: AbstractFolderExtrator

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	12
ccs	7
cts	7
cp	1
rs	9.8666
c	0
b	0
f	0
cc	2
nc	2
nop	1
crap	2

<?php

namespace Smart\EtlBundle\Extractor;

use Symfony\Component\Finder\Finder;
use Symfony\Component\Finder\SplFileInfo;

/**
 * Nicolas Bastien <[email protected]>
 */
abstract class AbstractFolderExtrator
{
    /**
     * @var string
     */
    protected $folderToExtract;

    /**
     * @return string
     */
    public function getFolderToExtract()
    {
        return $this->folderToExtract;
    }

    /**
     * @param string $folderToExtract
     */
    public function setFolderToExtract($folderToExtract)
    {
        $this->folderToExtract = $folderToExtract;
    }

    protected function check()
    {
        if (!is_dir($this->folderToExtract)) {
            throw new \BadMethodCallException('Invalid folder to extract');
        }
    }
    
    /**
     * @param  string $extension
     * @return array
     */
    protected function getFiles($extension)
    {
        $finder = new Finder();
        $finder->in($this->folderToExtract);
        $files = [];
        /* @var SplFileInfo $file */
        foreach ($finder->files()->name('*.' . $extension) as $file) {
            $files[] = substr($file->getFilename(), 0, -(strlen($extension)+1));
        }

        return array_combine($files, $files);
    }
}


1		<?php
2
3		namespace Smart\EtlBundle\Extractor;
4
5		use Symfony\Component\Finder\Finder;
6		use Symfony\Component\Finder\SplFileInfo;
7
8		/**
9		* Nicolas Bastien <[email protected]>
10		*/
11		abstract class AbstractFolderExtrator
12		{
13		/**
14		* @var string
15		*/
16		protected $folderToExtract;
17
18		/**
19		* @return string
20		*/
21		public function getFolderToExtract()
22		{
23		return $this->folderToExtract;
24		}
25
26		/**
27		* @param string $folderToExtract
28		*/
29	4	public function setFolderToExtract($folderToExtract)
30		{
31	4	$this->folderToExtract = $folderToExtract;
32	4	}
33
34	3	protected function check()
35		{
36	3	if (!is_dir($this->folderToExtract)) {
37		throw new \BadMethodCallException('Invalid folder to extract');
38		}
39	3	}
40
41		/**
42		* @param string $extension
43		* @return array
44		*/
45	3	protected function getFiles($extension)
46		{
47	3	$finder = new Finder();
48	3	$finder->in($this->folderToExtract);
49	3	$files = [];
50		/* @var SplFileInfo $file */
51	3	foreach ($finder->files()->name('*.' . $extension) as $file) {
52	3	$files[] = substr($file->getFilename(), 0, -(strlen($extension)+1));
53		}
54
55	3	return array_combine($files, $files);
56		}
57		}
58

smartbooster / etl-bundle

Pull Request — master (#4)

AbstractFolderExtrator::getFiles() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like