FileSystemWriter::validate() - Code Metrics - Inspection of "Adding more solid principles" - victormech/ditesto - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#13)

by Victor

created 2017-10-24 01:05 UTC

FileSystemWriter::validate() A

↳ Parent: FileSystemWriter

Complexity

Conditions	4
Paths	3

Size

Total Lines	10
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	20

Importance

Changes

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

<?php

namespace LazyEight\DiTesto\FileSystem;

use LazyEight\DiTesto\Exceptions\FileSystemException;
use LazyEight\DiTesto\Interfaces\FileSystem\FileSystemPathInterface;
use LazyEight\DiTesto\Interfaces\FileSystem\FileSystemWriterInterface;
use Test\DiTesto\FileSystem\Exceptions\InvalidPathException;

class FileSystemWriter implements FileSystemWriterInterface
{
    /**
     * @var FileSystemPathInterface
     */
    private $path;

    /**
     * FileSystemWriter constructor.
     * @param FileSystemPathInterface $path
     */
    public function __construct(FileSystemPathInterface $path)
    {
        $this->path = $path;
    }

    /**
     * @inheritDoc
     */
    public function isWritable(): bool
    {
        return is_writable($this->path->rawPath());
    }

    /**
     * @inheritDoc
     */
    public function write(string $content)
    {
        $this->validate();
        file_put_contents($this->path->rawPath(), $content);
    }

    /**
     * @throws FileSystemException
     * @throws InvalidPathException
     */
    public function validate()
    {
        if ($this->path->isDirectory()) {
            throw new InvalidPathException("Error, can't write file content to a directory.");
        }

        if (!is_writable($this->path->pathName()) && !$this->isWritable()) {
            throw new FileSystemException("Error, can't write to the file. The file must be writable.");
        }
    }
}


1			<?php
2
3			namespace LazyEight\DiTesto\FileSystem;
4
5			use LazyEight\DiTesto\Exceptions\FileSystemException;
6			use LazyEight\DiTesto\Interfaces\FileSystem\FileSystemPathInterface;
7			use LazyEight\DiTesto\Interfaces\FileSystem\FileSystemWriterInterface;
8			use Test\DiTesto\FileSystem\Exceptions\InvalidPathException;
9
10			class FileSystemWriter implements FileSystemWriterInterface
11			{
12			/**
13			* @var FileSystemPathInterface
14			*/
15			private $path;
16
17			/**
18			* FileSystemWriter constructor.
19			* @param FileSystemPathInterface $path
20			*/
21			public function __construct(FileSystemPathInterface $path)
22			{
23			$this->path = $path;
24			}
25
26			/**
27			* @inheritDoc
28			*/
29			public function isWritable(): bool
30			{
31			return is_writable($this->path->rawPath());
32			}
33
34			/**
35			* @inheritDoc
36			*/
37			public function write(string $content)
38			{
39			$this->validate();
40			file_put_contents($this->path->rawPath(), $content);
41			}
42
43			/**
44			* @throws FileSystemException
45			* @throws InvalidPathException
46			*/
47			public function validate()
48			{
49			if ($this->path->isDirectory()) {
50			throw new InvalidPathException("Error, can't write file content to a directory.");
51			}
52
53			if (!is_writable($this->path->pathName()) && !$this->isWritable()) {
54			throw new FileSystemException("Error, can't write to the file. The file must be writable.");
55			}
56			}
57			}
58

victormech / ditesto

Pull Request — master (#13)

FileSystemWriter::validate() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like