Bar::suffixMulti() - Code Metrics - Inspection of "refactor Cli/Runner" - webservco/framework - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( bf5028...3a1f66 )

by Radu

created 2018-11-09 10:56 UTC

Bar::suffixMulti() A

↳ Parent: Bar

Complexity

Conditions	1
Paths	1

Size

Total Lines	6
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php
namespace WebServCo\Framework\Cli\Progress;

//https://gist.github.com/mayconbordin/2860547
final class Bar
{
    protected $type;

    protected $width;
    protected $padding;
    protected $total;
    protected $item;

    protected $outBar;
    protected $outPad;
    protected $outMessage;

    public function __construct($width = 20)
    {
        $this->type = 'single_line';
        $this->width = $width;
        $this->padding = 25;
        $this->total = 100;
        $this->item = 1;
    }

    public function start($total = 100)
    {
        $this->total = $total;
    }

    public function advanceTo($item)
    {
        $this->item = $item;
    }

    public function setType($type)
    {
        if (in_array($type, ['single_line', 'multi_line'])) {
            $this->type = $type;
        }
    }

    public function prefix($message = '')
    {
        switch ($this->type) {
            case 'single_line':
            case 'multi_line':
                return $this->prefixProgress($message);
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
        }
    }

    public function suffix($result = true)
    {
        switch ($this->type) {
            case 'single_line':
                return $this->suffixSingle($result);
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            case 'multi_line':
                return $this->suffixMulti($result);
                break;
        }
    }

    protected function prefixProgress($message)
    {
        $percent = round($this->item * 100 / $this->total);
        $bar = (int) round($this->width * $percent / 100);
        $this->outBar = sprintf(
            "%s%% [%s>%s] %s",
            $percent,
            str_repeat('=', $bar),
            str_repeat(' ', (int) round($this->width-$bar)),
            $this->item . '/' . $this->total
        );
        $this->outMessage = $message;

        $padLen = ($this->width + $this->padding) - strlen($this->outBar);
        $this->outPad = (0 < $padLen) ? str_repeat(' ', (int) $padLen) : null;
        return $this->outBar.$this->outPad.$this->outMessage;
    }

    protected function suffixSingle($result, $overwrite = false)
    {
        $totalLen = strlen($this->outBar.$this->outPad.$this->outMessage);
        $output = null;

        if ($overwrite) {
            //overwrite current line
            $output .= "\033[" . $totalLen . 'D';
            $output .= str_repeat(' ', $this->width + $this->padding);
            $output .= $this->outMessage;
        }

        $padLen = 74 - $totalLen;
        if (0 < $padLen) {
            $output .= str_repeat(' ', $padLen);
        }
        $output .= '[';
        $output .= $result ? "\e[32mOK" : "\e[31mKO";
        $output .= "\e[0m" . ']';
        $output .= "\r";

        return $output;
    }

    protected function suffixMulti($result, $overwrite = true)
    {
        $output = $this->suffixSingle($result, $overwrite);
        $output .= PHP_EOL;

        return $output;
    }

    public function finish()
    {
        return "\033[" . 0 . 'D' . str_repeat(' ', 74) . "\r";
    }
}


1			<?php
2			namespace WebServCo\Framework\Cli\Progress;
3
4			//https://gist.github.com/mayconbordin/2860547
5			final class Bar
6			{
7			protected $type;
8
9			protected $width;
10			protected $padding;
11			protected $total;
12			protected $item;
13
14			protected $outBar;
15			protected $outPad;
16			protected $outMessage;
17
18			public function __construct($width = 20)
19			{
20			$this->type = 'single_line';
21			$this->width = $width;
22			$this->padding = 25;
23			$this->total = 100;
24			$this->item = 1;
25			}
26
27			public function start($total = 100)
28			{
29			$this->total = $total;
30			}
31
32			public function advanceTo($item)
33			{
34			$this->item = $item;
35			}
36
37			public function setType($type)
38			{
39			if (in_array($type, ['single_line', 'multi_line'])) {
40			$this->type = $type;
41			}
42			}
43
44			public function prefix($message = '')
45			{
46			switch ($this->type) {
47			case 'single_line':
48			case 'multi_line':
49			return $this->prefixProgress($message);
50			break;
			0 ignored issues – show Unused Code introduced 2018-03-20 17:17 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
51			}
52			}
53
54			public function suffix($result = true)
55			{
56			switch ($this->type) {
57			case 'single_line':
58			return $this->suffixSingle($result);
59			break;
			0 ignored issues – show Unused Code introduced 2018-03-20 17:17 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
60			case 'multi_line':
61			return $this->suffixMulti($result);
62			break;
63			}
64			}
65
66			protected function prefixProgress($message)
67			{
68			$percent = round($this->item * 100 / $this->total);
69			$bar = (int) round($this->width * $percent / 100);
70			$this->outBar = sprintf(
71			"%s%% [%s>%s] %s",
72			$percent,
73			str_repeat('=', $bar),
74			str_repeat(' ', (int) round($this->width-$bar)),
75			$this->item . '/' . $this->total
76			);
77			$this->outMessage = $message;
78
79			$padLen = ($this->width + $this->padding) - strlen($this->outBar);
80			$this->outPad = (0 < $padLen) ? str_repeat(' ', (int) $padLen) : null;
81			return $this->outBar.$this->outPad.$this->outMessage;
82			}
83
84			protected function suffixSingle($result, $overwrite = false)
85			{
86			$totalLen = strlen($this->outBar.$this->outPad.$this->outMessage);
87			$output = null;
88
89			if ($overwrite) {
90			//overwrite current line
91			$output .= "\033[" . $totalLen . 'D';
92			$output .= str_repeat(' ', $this->width + $this->padding);
93			$output .= $this->outMessage;
94			}
95
96			$padLen = 74 - $totalLen;
97			if (0 < $padLen) {
98			$output .= str_repeat(' ', $padLen);
99			}
100			$output .= '[';
101			$output .= $result ? "\e[32mOK" : "\e[31mKO";
102			$output .= "\e[0m" . ']';
103			$output .= "\r";
104
105			return $output;
106			}
107
108			protected function suffixMulti($result, $overwrite = true)
109			{
110			$output = $this->suffixSingle($result, $overwrite);
111			$output .= PHP_EOL;
112
113			return $output;
114			}
115
116			public function finish()
117			{
118			return "\033[" . 0 . 'D' . str_repeat(' ', 74) . "\r";
119			}
120			}
121

webservco / framework

Push — master ( bf5028...3a1f66 )

Bar::suffixMulti() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like