OutputFile::__construct() - Code Metrics - Inspection of "(WIP)Clean directories when download failure" - hirak/prestissimo - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#24)

by Hiraku

created 2016-01-30 15:32 UTC

OutputFile::__construct() A

↳ Parent: OutputFile

Complexity

Conditions	3
Paths	3

Size

Total Lines	18
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

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

<?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 = true;

    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 setFailure()
    {
        $this->success = false;
    }

    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 = true;
26
27			public function __construct($fileName)
28			{
29			$this->fileName = $fileName;
30			if (is_dir($fileName)) {
31			throw new Downloader\TransportException(
32			"The file could not be written to $fileName. Directory exists."
33			);
34			}
35
36			$this->createDir($fileName);
37
38			$this->fp = fopen($fileName, 'wb');
39			if (!$this->fp) {
40			throw new Downloader\TransportException(
41			"The file could not be written to $fileName."
42			);
43			}
44			}
45
46			public function __destruct()
47			{
48			if ($this->fp) {
49			fclose($this->fp);
50			}
51
52			if (! $this->success) {
53			unlink($this->fileName);
54			foreach ($this->createdDirs as $dir) {
55			rmdir($dir);
56			}
57			}
58			}
59
60			public function getPointer()
61			{
62			return $this->fp;
63			}
64
65			public function setFailure()
66			{
67			$this->success = false;
68			}
69
70			protected function createDir($fileName)
71			{
72			$dir = $fileName;
73			$createdDirs = array();
74			do {
75			$dir = dirname($dir);
76			$createdDirs[] = $dir;
77			} while (! file_exists($dir));
78			array_pop($createdDirs);
79			$this->createdDirs = array_reverse($createdDirs);
80
81			$targetdir = dirname($fileName);
82			if (!file_exists($targetdir)) {
83			$created = mkdir($targetdir, 0766, true);
84			if (!$created) {
85			$this->success = false;
86			throw new Downloader\TransportException(
87			"The file could not be written to $this->fileName."
88			);
89			}
90			}
91			}
92			}
93

hirak / prestissimo

Pull Request — master (#24)

OutputFile::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like