Shell::getContents() - Code Metrics - Inspection of "Improve getContents function" - Pleets/dronephp - Measure and Improve Code Quality continuously with Scrutinizer

Passed
Push — master ( b176c5...1503d9 )

by Darío
created 2018-12-28 20:16 UTC
Shell::getContents() C

↳ Parent: Shell
Complexity

Conditions	13
Paths	14
Size

Total Lines	45
Code Lines	23
Duplication

Lines	0
Ratio	0 %
Importance

Changes
Metric	Value
cc	13
eloc	23
nc	14
nop	5
dl	0
loc	45
rs	6.6166
c	0
b	0
f	0
How to fix Complexity

<?php
/**
 * DronePHP (http://www.dronephp.com)
 *
 * @link      http://github.com/Pleets/DronePHP
 * @copyright Copyright (c) 2016-2018 Pleets. (http://www.pleets.org)
 * @license   http://www.dronephp.com/license
 * @author    Darío Rivera <[email protected]>
 */

namespace Drone\FileSystem;

use Drone\Error\Errno;

/**
 * Shell class
 *
 * This class represents a system terminal
 */
class Shell implements ShellInterface
{
    use \Drone\Error\ErrorTrait;

    /**
     * Home directory (~)
     *
     * @var string
     */
    private $home;

    /**
     * Result of last command (optional)
     *
     * @var string
     */
    private $buffer = null;

    /**
     * Returns the home attribute
     *
     * @return string
     */
    public function getHome()
    {
        return $this->home;
    }

    /**
     * Returns the buffer attribute
     *
     * @return mixed
     */
    public function getBuffer()
    {
        return $this->buffer;
    }

    /**
     * Constructor
     *
     * @param string $home
     */
    public function __construct($home = null)
    {
        $this->home = (is_null($home) || empty($home)) ? $this->pwd() : $home;
class Id
{
    public $id;

    public function __construct($id)
    {
        $this->id = $id;
    }

}

class Account
{
    /** @var  Id $id */
    public $id;
}

$account_id = false;

if (starsAreRight()) {
    $account_id = new Id(42);
}

$account = new Account();
if ($account instanceof Id)
{
    $account->id = $account_id;
}
        $this->cd($this->home);
        $this->cd(/** @scrutinizer ignore-type */ $this->home);
    }

    /**
     * Iterative function for directories and files
     *
     * @param string   $directory
     * @param callable $fileCallback
     * @param callable $dirCallback
     * @param callable $callback
     * @param boolean  $showParentPath
     *
     * @return array
     */
    public function getContents($directory, $fileCallback, $dirCallback, $callback = null, $showParentPath = false)
    {
        $contents = [];

        if (is_dir($directory))
        {
            foreach ($this->ls($directory) as $item)
            {
                if ($item != '.' && $item != '..')
                    $contents[] = $item;
            }

            //$allContents = $contents;

            if (count($contents) > 0)
            {
                foreach ($contents as $i)
                {
                    if (is_file($directory.'/'.$i))
                    {
                        $allContents[] = ($showParentPath) ? $directory.'/'.$i : $i;

                        $this->buffer = $directory.'/'.$i;
                        call_user_func($fileCallback, $this);
                    }
                    else if (is_dir($directory.'/'.$i))
                    {
                        $allContents[] = ($showParentPath) ? $directory.'/'.$i : $i;
                        $allContents[] = $this->getContents($directory.'/'.$i, $fileCallback, $dirCallback, null, false);

                        $this->buffer = $directory.'/'.$i;
                        call_user_func($dirCallback, $this);
                    }
                }
            }
        }
        else if (is_file($directory))
            throw new \InvalidArgumentException("'$directory' is actually a file");
        else
            throw new \InvalidArgumentException("The directory '$directory' does not exists");

        if (!is_null($callback))
            call_user_func($callback, $this);

        return $allContents;

    }

    /**
     * Returns the curent directory
     *
     * @return string|boolean
     */
    public function pwd()
    {
        if (getcwd())
            $this->buffer = getcwd();
        else
            return false;

        return $this->buffer;
    }

    /**
     * Returns a list with directory contents
     *
     * @param string|null $path
     * @param boolean     $recursive
     *
     * @return array
     */
    public function ls($path = null, $recursive = false)
    {
        $filesToReturn = [];

        $path = (is_null($path) || empty($path)) ? '.' : $path;

        if (is_file($path))
            $filesToReturn = array($path);
        else if (is_dir($path))
        {
            $pathIns = dir($path);

            if ($recursive)
            {
                $dirs = $files = [];

                $filesToReturn = $this->getContents($path,
                    # file's callback
                    function($event) use (&$files)
                    {
                        $files[] = $event->getBuffer();
                    },
                    # folder's callback
                    function($event) use (&$dirs)
                    {
                        $dirs[] = $event->getBuffer();
                    }
                );

                /*foreach ($dirs as $item)
                    $filesToReturn[] = $item;

                foreach ($files as $item)
                    $filesToReturn[] = $item;*/
            }
            else
            {
                while (($item = $pathIns->read()) !== false)
                {
                    if ($item != '.' && $item != '..')
                        $filesToReturn[] = $item;
                }

                $pathIns->close();
            }
        }
        else {

            $contents = $this->ls();

            foreach ($contents as $item)
            {
                if (!empty($path))
                    if (!strlen(stristr($item, $path)) > 0)
                        continue;
                if (strstr($item,'~') === false && $item != '.' && $item != '..')
                    $filesToReturn[] = $item;
            }
        }

        return $filesToReturn;
    }

    /**
     * Changes the current directory
     *
     * @param string|null $path
     *
     * @return boolean
     */
    public function cd($path = null)
    {
        $path = (is_null($path) || empty($path)) ? $this->home : $path;

        if (is_dir($path))
        {
            if (chdir($path))
                return true;
        }

        return false;
    }

    /**
     * Changes the file's date
     *
     * @param string
     *
     * @return boolean
     */
    public function touch($file)
    {
        $contents = file_exists($file) ? file_get_contents($file) : "";

        $hd = @fopen($file, "w+");

        if (!$hd || @fwrite($hd, $contents) === false)

        {
            $this->error(Errno::FILE_PERMISSION_DENIED, $file);
            return false;
        }

        @fclose($hd);
        /** @scrutinizer ignore-unhandled */ @fclose($hd);

        return true;
    }

    /**
     * Deletes one or more files
     *
     * @param string       $file
     * @param boolean|null $recursive
     *
     * @return boolean
     */
    public function rm($file, $recursive = null)
    {
        $recursive = is_null($recursive) ? false : $recursive;

        if (is_null($file))

            throw new \InvalidArgumentException("Missing first parameter");

        if (file_exists($file) && !$recursive)
            unlink($file);
        elseif (is_dir($file) && $recursive)
        {
            $that = $this;

            $this->getContents($file, function() use ($that) {
                unlink($that->getBuffer());
            }, function() use ($that) {
                rmdir($that->getBuffer());
            }, function() use ($file) {
                @rmdir($file);
                /** @scrutinizer ignore-unhandled */ @rmdir($file);
            });
        }

        return true;
    }

    /**
     * Copies one or more files or directories
     *
     * @param string       $file
     * @param string       $dest
     * @param boolean|null $recursive
     *
     * @return boolean
     */
    public function cp($file, $dest, $recursive = null)
    {
        $recursive = (is_null($recursive)) ? false : $recursive;

        if (empty($file) || empty($dest))
            throw new \InvalidArgumentException("Missing parameters");

        if (is_dir($file) && !$recursive)
            throw new \RuntimeException("Ommiting directory <<$foo>>");


        if (is_dir($file) && (is_dir($dest) || !file_exists($dest)))
        {
            $files = [
                "files"   => [],
                "folders" => []
            ];

            if (is_dir($dest))
                $files["folders"][] = basename($file);

            if (!file_exists($dest))
                mkdir($dest);

            $that = $this;

            $this->getContents($file,
                # file's callback
                function() use($that, &$files)
                {
                    $files["files"][] = $that->getBuffer();
                },
                # folder's callback
                function() use($that, &$files)
                {
                    $files["folders"][] = $that->getBuffer();
                },
                # final callback
                function() use (&$files, $dest)
                {
                    if (count($files["folders"]))
                    {
                        foreach ($files["folders"] as $folder)
                        {
                            if (!file_exists($dest.'/'.$folder))
                                mkdir("$dest/$folder", 0777, true);
                        }
                    }

                    if (count($files["files"]))
                    {
                        foreach ($files["files"] as $item)
                        {
                            if (!file_exists("$dest/$files"))
                                copy($item, $dest.'/'.$item);
                        }
                    }
                }
            );
        }

        if (is_dir($dest))
        {
            if (!$recursive)
                copy($file, $dest.'/'.$file);
            else {

                $files = array();
                $files[0] = array();
                $files[1] = array();

                $_SESSION["BUFFER"]["EXO"]["cp"][2] = $dest;

                $that = $this;

                $this->getContents($file, function() use($that, &$files) {
                    $files[0][] = $that->getBuffer();
                }, function() use($that, &$files) {
                    $files[1][] = $that->getBuffer();
                }, function() use ($files, $dest) {

                    foreach ($files[1] as $item)
                    {
                        if (!file_exists($dest.'/'.$item))
                            mkdir("$dest/$item", 0777, true);
                    }

                    foreach ($files[0] as $item)
                    {
                        if (!file_exists("$dest/$files"))
                            copy($item, $dest.'/'.$item);
                    }
                });
            }
        }
        else
            copy($file, $dest);

        return true;
    }

    /**
     * Moves or renames files
     *
     * @param string $oldfile
     * @param string $newfile
     *
     * @return boolean
     */
    public function mv($oldfile, $newfile)
    {
        if (empty($oldfile))
            throw new \InvalidArgumentException("Missing first parameter");

        if (is_dir($newfile))
                $newfile .= '/'.basename($oldfile);

        if ($oldfile == $newfile)
            throw new \Exception("'$oldfile' and '$newfile' are the same file");

        if(!rename($oldfile, $newfile))
            return false;

        return true;
    }

    /**
     * Creates a directory
     *
     * @param string       $dir
     * @param string|null  $dest
     * @param boolean|null $recursive
     *
     * @return boolean
     */
    public function mkdir($dir, $dest = null, $recursive = null)
    {
        if (empty($dir))
            throw new \InvalidArgumentException("Missing first parameter");

        if (empty($dest))
            $dest = '.';

        $recursive = (is_null($recursive)) ? false : $recursive;

        if ($recursive)
            mkdir("$dest/$dir", 0777, true);
        else {
            if (!is_dir($dir))
            {
                if(!mkdir("$dir", 0777))
                    return false;
            }
        }
        return true;
    }

    /**
     * Deletes a directory
     *
     * @param string $dir
     *
     * @return boolean
     */
    public function rmdir($dir)
    {
        if (is_null($dir) || empty($dir))
            throw new \RuntimeException("Missing first parameter");

        if (rmdir($dir))
            return true;
        else
            return false;
    }
}
Pleets / dronephp

Push — master ( b176c5...1503d9 )

Shell::getContents() C

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like
