LoopFactory::endLoop() - Code Metrics - Inspection of "rebuild" - advmaker/blade-loop - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#11)

by Maksim (Ellrion)

created 2016-04-13 10:45 UTC

LoopFactory::endLoop() A

↳ Parent: LoopFactory

Complexity

Conditions	2
Paths	2

Size

Total Lines	12
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes	2
Bugs	0	Features	0

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

<?php

namespace Advmaker\BladeLoop;

class LoopFactory
{
    /**
     * The stack of Loop instances
     *
     * @var Loop[] $stack
     */
    protected $stack = [];

    /**
     * Creates a new loop with the given array and adds it to the stack
     *
     * @param array $items The array that will be iterated
     * @return Loop
     */
    public function newLoop($items)
    {
        $loop = new Loop($items);

        // Check stack for parent loop to register it with this loop
        if (count($this->stack) > 0) {
            $loop->setParentLoop(last($this->stack));
        }

        array_push($this->stack, $loop);

        return $loop;
    }

    /**
     * Should be called after the loop has finished
     *
     * @param $loop
     */
    public function endLoop(&$loop)
    {
        array_pop($this->stack);

        if (count($this->stack) > 0) {
            // This loop was inside another loop. We persist the loop variable and assign back the parent loop
            $loop = end($this->stack);
        } else {
            // This loop was not inside another loop. We remove the var
            $loop = null;
        }
    }

    /**
     * To be called first inside the foreach loop. Returns the current loop
     *
     * @return Loop $current The current loop data
     */
    public function loop()
    {
        $current = end($this->stack);
        $current->loop();

        return $current;
    }
}


1		<?php
2
3		namespace Advmaker\BladeLoop;
4
5		class LoopFactory
6		{
7		/**
8		* The stack of Loop instances
9		*
10		* @var Loop[] $stack
11		*/
12		protected $stack = [];
13
14		/**
15		* Creates a new loop with the given array and adds it to the stack
16		*
17		* @param array $items The array that will be iterated
18		* @return Loop
19	15	*/
20		public function newLoop($items)
21	15	{
22	15	$loop = new Loop($items);
23
24		// Check stack for parent loop to register it with this loop
25		if (count($this->stack) > 0) {
26		$loop->setParentLoop(last($this->stack));
27		}
28
29	15	array_push($this->stack, $loop);
30
31		return $loop;
32	15	}
33	3
34	3	/**
35		* Should be called after the loop has finished
36	15	*
37	15	* @param $loop
38		*/
39		public function endLoop(&$loop)
40		{
41		array_pop($this->stack);
42
43		if (count($this->stack) > 0) {
44		// This loop was inside another loop. We persist the loop variable and assign back the parent loop
45		$loop = end($this->stack);
46		} else {
47		// This loop was not inside another loop. We remove the var
48		$loop = null;
49		}
50		}
51
52		/**
53		* To be called first inside the foreach loop. Returns the current loop
54	15	*
55		* @return Loop $current The current loop data
56	15	*/
57		public function loop()
58		{
59		$current = end($this->stack);
60		$current->loop();
61
62		return $current;
63		}
64		}
65

advmaker / blade-loop

Pull Request — master (#11)

LoopFactory::endLoop() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like