FieldCollection::removeByName() - Code Metrics - Inspection of "remove travis php8 for now" - delboy1978uk/form - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( ca0646...6bce0e )

by Derek Stephen

created 2021-01-12 15:16 UTC

FieldCollection::removeByName() A

↳ Parent: FieldCollection

Complexity

Conditions	3
Paths	3

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	12
CRAP Score	3

Importance

Changes

Metric	Value
dl	0
loc	21
ccs	12
cts	12
cp	1
rs	9.584
c	0
b	0
f	0
cc	3
nc	3
nop	1
crap	3

<?php
/**
 * User: delboy1978uk
 * Date: 19/11/2016
 * Time: 12:18
 */

namespace Del\Form\Collection;

use Del\Form\Field\FieldInterface;

class FieldCollection extends AbstractCollection implements CollectionInterface
{
    /**
     * @param $name
     * @return FieldInterface|null
     */
    public function findByName($name)
    {
        $this->rewind();
        while ($this->valid()) {
            /** @var FieldInterface $field */
            $field = $this->current();
            if ($field->getName() == $name) {
                return $field;
            }
            $this->next();
        }
        $this->rewind();
        return null;
    }

    /**
     * @param $name
     * @return bool
     */
    public function removeByName($name): bool
    {
        $this->rewind();

        while ($this->valid()) {
            $field = $this->current();

            if ($field->getName() == $name) {
                $key = $this->key();
                $this->offsetUnset($key);
                $this->rewind();

                return true;
            }
            $this->next();
        }

        $this->rewind();

        return false;
    }

    /**
     * @return FieldInterface
     */
    public function current(): FieldInterface
    {
        return parent::current();
    }
}

1		<?php
2		/**
3		* User: delboy1978uk
4		* Date: 19/11/2016
5		* Time: 12:18
6		*/
7
8		namespace Del\Form\Collection;
9
10		use Del\Form\Field\FieldInterface;
11
12		class FieldCollection extends AbstractCollection implements CollectionInterface
13		{
14		/**
15		* @param $name
16		* @return FieldInterface\|null
17		*/
18	4	public function findByName($name)
19		{
20	4	$this->rewind();
21	4	while ($this->valid()) {
22		/** @var FieldInterface $field */
23	3	$field = $this->current();
24	3	if ($field->getName() == $name) {
25	3	return $field;
26		}
27	1	$this->next();
28		}
29	1	$this->rewind();
30	1	return null;
31		}
32
33		/**
34		* @param $name
35		* @return bool
36		*/
37	1	public function removeByName($name): bool
38		{
39	1	$this->rewind();
40
41	1	while ($this->valid()) {
42	1	$field = $this->current();
43
44	1	if ($field->getName() == $name) {
45	1	$key = $this->key();
46	1	$this->offsetUnset($key);
47	1	$this->rewind();
48
49	1	return true;
50		}
51	1	$this->next();
52		}
53
54	1	$this->rewind();
55
56	1	return false;
57		}
58
59		/**
60		* @return FieldInterface
61		*/
62	51	public function current(): FieldInterface
63		{
64	51	return parent::current();
65		}
66		}

delboy1978uk / form

Push — master ( ca0646...6bce0e )

FieldCollection::removeByName() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like