AlreadyExtract\Utils\ExtractorCommandDecoder\File

Complexity

Total Complexity

6

Size/Duplication

Total Lines	64
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	isDir()	0	4	1
A	isFile()	0	4	1
A	getPath()	0	4	1
A	setPath()	0	5	1
A	getUnpackedSize()	0	4	1
A	setUnpackedSize()	0	5	1

<?php

namespace AlreadyExtract\Utils\ExtractorCommandDecoder;

/**
 * Created by PhpStorm.
 * User: mhor
 * Date: 02/08/17
 * Time: 23:33
 */
class File
{
    /**
     * @var string
     */
    private $path;

    /**
     * @var string
     */
    private $unpackedSize;

    /**
     * @return bool
     */
    public function isDir()
    {
        return $this->unpackedSize === 0;

The strict comparison === seems to always evaluate to false as the types of $this->unpackedSize (string) and 0 (integer) can never be identical. Maybe you want to use a loose comparison == instead?

    }

    /**
     * @return bool
     */
    public function isFile()
    {
        return $this->unpackedSize > 0;
    }

    /**
     * @return mixed
     */
    public function getPath()
    {
        return $this->path;
    }

    /**
     * @param mixed $path
     * @return File
     */
    public function setPath($path)
    {
        $this->path = $path;
        return $this;
    }

    /**
     * @return mixed
     */
    public function getUnpackedSize()
    {
        return $this->unpackedSize;
    }

    /**
     * @param mixed $unpackedSize
     * @return File
     */
    public function setUnpackedSize($unpackedSize)
    {
        $this->unpackedSize = $unpackedSize;
        return $this;
    }
}