OutputFile::createDir() - Code Metrics - Inspection of "(WIP)Fixed dist-mirrors" - hirak/prestissimo - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#42)

by Hiraku

created 2016-02-06 06:08 UTC

OutputFile::createDir() B

↳ Parent: OutputFile

Complexity

Conditions	4
Paths	3

Size

Total Lines	22
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	18
CRAP Score	4

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
c	1
b	0
f	1
dl	0
loc	22
ccs	18
cts	18
cp	1
rs	8.9197
cc	4
eloc	16
nc	3
nop	1
crap	4

<?php
/*
 * @author Hiraku NAKANO
 * @license MIT https://github.com/hirak/prestissimo
 */
namespace Hirak\Prestissimo;

use Composer\Downloader;

/**
 * file pointer wrapper with auto clean
 */
class OutputFile
{
    /** @var resource<file>|null */
    protected $fp;

    /** @var string */
    protected $fileName;

    /** @var string[] */
    protected $createdDirs = array();

    /** @var bool */
    private $success = false;

    public function __construct($fileName)
    {
        $this->fileName = $fileName;
        if (is_dir($fileName)) {
            throw new Downloader\TransportException(
                "The file could not be written to $fileName. Directory exists."
            );
        }

        $this->createDir($fileName);

        $this->fp = fopen($fileName, 'wb');
        if (!$this->fp) {
            throw new Downloader\TransportException(
                "The file could not be written to $fileName."
            );
        }
    }

    public function __destruct()
    {
        if ($this->fp) {
            fclose($this->fp);
        }

        if (!$this->success) {
            unlink($this->fileName);
            foreach ($this->createdDirs as $dir) {
                rmdir($dir);
            }
        }
    }

    public function getPointer()
    {
        return $this->fp;
    }

    public function setSuccess()
    {
        $this->success = true;
    }

    protected function createDir($fileName)
    {
        $dir = $fileName;
        $createdDirs = array();
        do {
            $dir = dirname($dir);
            $createdDirs[] = $dir;
        } while (!file_exists($dir));
        array_pop($createdDirs);
        $this->createdDirs = array_reverse($createdDirs);

        $targetdir = dirname($fileName);
        if (!file_exists($targetdir)) {
            $created = mkdir($targetdir, 0766, true);
            if (!$created) {
                $this->success = false;
                throw new Downloader\TransportException(
                    "The file could not be written to $this->fileName."
                );
            }
        }
    }
}


1		<?php
2		/*
3		* @author Hiraku NAKANO
4		* @license MIT https://github.com/hirak/prestissimo
5		*/
6		namespace Hirak\Prestissimo;
7
8		use Composer\Downloader;
9
10		/**
11		* file pointer wrapper with auto clean
12		*/
13		class OutputFile
14		{
15		/** @var resource<file>\|null */
16		protected $fp;
17
18		/** @var string */
19		protected $fileName;
20
21		/** @var string[] */
22		protected $createdDirs = array();
23
24		/** @var bool */
25		private $success = false;
26
27	8	public function __construct($fileName)
28		{
29	8	$this->fileName = $fileName;
30	8	if (is_dir($fileName)) {
31	1	throw new Downloader\TransportException(
32	1	"The file could not be written to $fileName. Directory exists."
33	1	);
34		}
35
36	7	$this->createDir($fileName);
37
38	6	$this->fp = fopen($fileName, 'wb');
39	6	if (!$this->fp) {
40	1	throw new Downloader\TransportException(
41	1	"The file could not be written to $fileName."
42	1	);
43		}
44	5	}
45
46	5	public function __destruct()
47		{
48	5	if ($this->fp) {
49	5	fclose($this->fp);
50	5	}
51
52	5	if (!$this->success) {
53	4	unlink($this->fileName);
54	4	foreach ($this->createdDirs as $dir) {
55	4	rmdir($dir);
56	4	}
57	4	}
58	5	}
59
60	5	public function getPointer()
61		{
62	5	return $this->fp;
63		}
64
65	1	public function setSuccess()
66		{
67	1	$this->success = true;
68	1	}
69
70	7	protected function createDir($fileName)
71		{
72	7	$dir = $fileName;
73	7	$createdDirs = array();
74		do {
75	7	$dir = dirname($dir);
76	7	$createdDirs[] = $dir;
77	7	} while (!file_exists($dir));
78	7	array_pop($createdDirs);
79	7	$this->createdDirs = array_reverse($createdDirs);
80
81	7	$targetdir = dirname($fileName);
82	7	if (!file_exists($targetdir)) {
83	6	$created = mkdir($targetdir, 0766, true);
84	6	if (!$created) {
85	1	$this->success = false;
86	1	throw new Downloader\TransportException(
87	1	"The file could not be written to $this->fileName."
88	1	);
89		}
90	5	}
91	6	}
92		}
93

hirak / prestissimo

Pull Request — master (#42)

OutputFile::createDir() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like