SyncProgress::__construct() - Code Metrics - Inspection of "Refactor library" - hansott/lastify - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( aed180...cec8e0 )

by Hans

created 2016-03-06 11:59 UTC

SyncProgress::__construct() A

↳ Parent: SyncProgress

Complexity

Conditions	2
Paths	2

Size

Total Lines	10
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	10
rs	9.4285
ccs	8
cts	8
cp	1
cc	2
eloc	6
nc	2
nop	3
crap	2

<?php

namespace HansOtt\Lastify;

use InvalidArgumentException;

final class SyncProgress
{
    private $current;

    private $total;

    private $callback = null;

    public function __construct($initial = 0, $total, SyncProgressCallback $callback = null)
    {
        $this->assertValidArguments($initial, $total);
        $this->current = (int) $initial;
        $this->total = (int) $total;

        if (isset($callback)) {
            $this->callback = $callback;
        }
    }

    public function step(TrackInfo $currentItem)
    {
        $this->current++;
        $this->callProgressCallback($currentItem);
    }

    private function callProgressCallback($currentItem)
    {
        if ($this->callback instanceof SyncProgressCallback) {
            $this->callback->onProgress($this->current, $this->total, $currentItem);
        }
    }

    private function assertValidArguments($initial, $total)
    {
        if (!is_int($initial)) {
            throw new InvalidArgumentException('The initial value should be an integer, instead got:' . gettype($initial));
        }
        if ($initial < 0) {
            throw new InvalidArgumentException('The initial value should be a positive integer');
        }
        if (!is_int($total)) {
            throw new InvalidArgumentException('The total value should be an integer, instead got:' . gettype($total));
        }
        if ($total < 0) {
            throw new InvalidArgumentException('The total value should be a positive integer');
        }
        if ($total <= $initial) {
            throw new InvalidArgumentException('The total value should be a greater than the initial value.');
        }
    }
}


1		<?php
2
3		namespace HansOtt\Lastify;
4
5		use InvalidArgumentException;
6
7		final class SyncProgress
8		{
9		private $current;
10
11		private $total;
12
13		private $callback = null;
14
15	3	public function __construct($initial = 0, $total, SyncProgressCallback $callback = null)
16		{
17	3	$this->assertValidArguments($initial, $total);
18	3	$this->current = (int) $initial;
19	3	$this->total = (int) $total;
20
21	3	if (isset($callback)) {
22	3	$this->callback = $callback;
23	3	}
24	3	}
25
26	3	public function step(TrackInfo $currentItem)
27		{
28	3	$this->current++;
29	3	$this->callProgressCallback($currentItem);
30	3	}
31
32	3	private function callProgressCallback($currentItem)
33		{
34	3	if ($this->callback instanceof SyncProgressCallback) {
35	3	$this->callback->onProgress($this->current, $this->total, $currentItem);
36	3	}
37	3	}
38
39	3	private function assertValidArguments($initial, $total)
40		{
41	3	if (!is_int($initial)) {
42		throw new InvalidArgumentException('The initial value should be an integer, instead got:' . gettype($initial));
43		}
44	3	if ($initial < 0) {
45		throw new InvalidArgumentException('The initial value should be a positive integer');
46		}
47	3	if (!is_int($total)) {
48		throw new InvalidArgumentException('The total value should be an integer, instead got:' . gettype($total));
49		}
50	3	if ($total < 0) {
51		throw new InvalidArgumentException('The total value should be a positive integer');
52		}
53	3	if ($total <= $initial) {
54		throw new InvalidArgumentException('The total value should be a greater than the initial value.');
55		}
56	3	}
57		}
58

hansott / lastify

Push — master ( aed180...cec8e0 )

SyncProgress::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like