Base::__construct() - Code Metrics - Inspection of "Patch 1" - michaeldyrynda/laravel-make-user - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#6)

unknown

created 2018-03-07 06:45 UTC

Base::__construct() A

↳ Parent: Base

Complexity

Conditions	3
Paths	3

Size

Total Lines	18
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	18
rs	9.4285
c	0
b	0
f	0
cc	3
eloc	9
nc	3
nop	1

<?php

namespace Dyrynda\Artisan\BulkImport\Handlers;

use SplFileInfo;
use Dyrynda\Artisan\Exceptions\ImportFileException;
use Dyrynda\Artisan\BulkImport\BulkImportFileHandler;

abstract class Base implements BulkImportFileHandler
{
    protected $file;
    protected $fields;
    protected $filePath;
    protected $fileHandle;

    /**
     * Base constructor.
     * @param $filePath
     *
     * @throws \Dyrynda\Artisan\Exceptions\ImportFileException
     */
    public function __construct(SplFileInfo $file)
    {
        $this->file = $file;

        $this->filePath = $file->getPathname();

        if (! $this->file->getExtension()) {
            throw ImportFileException::noExtension();
        }

        if (! $this->file->isFile()) {
            throw ImportFileException::notExist($this->filePath);
        }

        $this->fileHandle = $this->file->openFile();

        $this->validateSyntax();
    }

    /**
     * Checks file for valid syntax.
     *
     * @return void
     * @throws \Dyrynda\Artisan\Exceptions\ImportFileException
     */
    abstract protected function validateSyntax();

    /**
     * Get the info from the file.
     *
     * @return array
     */
    abstract public function getData();
}


1			<?php
2
3			namespace Dyrynda\Artisan\BulkImport\Handlers;
4
5			use SplFileInfo;
6			use Dyrynda\Artisan\Exceptions\ImportFileException;
7			use Dyrynda\Artisan\BulkImport\BulkImportFileHandler;
8
9			abstract class Base implements BulkImportFileHandler
10			{
11			protected $file;
12			protected $fields;
13			protected $filePath;
14			protected $fileHandle;
15
16			/**
17			* Base constructor.
18			* @param $filePath
19			*
20			* @throws \Dyrynda\Artisan\Exceptions\ImportFileException
21			*/
22			public function __construct(SplFileInfo $file)
23			{
24			$this->file = $file;
25
26			$this->filePath = $file->getPathname();
27
28			if (! $this->file->getExtension()) {
29			throw ImportFileException::noExtension();
30			}
31
32			if (! $this->file->isFile()) {
33			throw ImportFileException::notExist($this->filePath);
34			}
35
36			$this->fileHandle = $this->file->openFile();
37
38			$this->validateSyntax();
39			}
40
41			/**
42			* Checks file for valid syntax.
43			*
44			* @return void
45			* @throws \Dyrynda\Artisan\Exceptions\ImportFileException
46			*/
47			abstract protected function validateSyntax();
48
49			/**
50			* Get the info from the file.
51			*
52			* @return array
53			*/
54			abstract public function getData();
55			}
56

michaeldyrynda / laravel-make-user

Pull Request — master (#6)

Base::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like