Classes\Update\ProgressBar

Complexity

Total Complexity

16

Size/Duplication

Total Lines	92
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	0

Importance

Changes	1
Bugs	0	Features	0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	1	1
A	getInstance()	0	9	2
A	setMaxByte()	0	6	1
A	setProgress()	0	7	1
A	getProgress()	0	4	1
A	calcule()	0	8	2
A	finish()	0	9	3
A	render()	0	17	4
A	clear()	0	7	1

<?php

namespace Classes\Update;

class ProgressBar
{

    private static $finished = 0;
    private static $progressBar;

    private $max = 0;

    private $progress = 0;

    private $progresslength = 0;

    private function __construct (){ }

    /**
     * @return \Classes\Update\ProgressBar
     */
    public static function getInstance ()
    {
        if ( is_null ( self::$progressBar ) )
        {
            self::$progressBar = new ProgressBar();
        }

        return self::$progressBar;
    }

    public function setMaxByte ( $bytesMax )
    {
        $this->max = $bytesMax;

        return $this;
    }

    public function setProgress ( $progress )
    {
        $this->progress = $progress;
        $this->calcule ();

        return $this;
    }

    public function getProgress ()
    {
        return $this->progresslength;
    }

    public function calcule ()
    {
        if ( $this->progress > 0 )
        {
            $this->progresslength = round( $this->progress * 100 / $this->max );

The property $progresslength was declared of type integer, but round($this->progress * 100 / $this->max) is of type double. Maybe add a type cast?

            //$this->progresslength = (int) ( ( $this->progress / $this->max ) * 100 );

44% of this comment could be valid code. Did you maybe forget this after debugging?

        }
    }

    public function finish ()
    {
        if ( $this->getProgress () >= 99 && !self::$finished)
        {
            echo "\n\033[1;32mDone!\033[0m\n";
            self::$finished = 1;
        }

    }

    public function render ()
    {
        if ( $this->progress > 0 && !self::$finished)
        {
            if ( ! isset( $this->max ) )
            {
                printf ( "\rUnknown filesize.. %2d kb done.." , $this->progress / 1024 );
            } else
            {
                $length = $this->getProgress ();
                printf ( "\r[%-100s] %d%%" , str_repeat ( "=" , $length )
                                             . ">" , $length );
            }
        }

        return $this;
    }

    public function clear ()
    {
        $this->max = 0;
        $this->progress = 0;

        return $this;
    }
}