AbstractList::lastIndexOf() - Code Metrics - Inspection of "Merge pull request #5 from peter279k/test_enhancem..." - tebru/php-collections - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( e5d322...90ee42 )

by Nate

created 2019-08-18 17:46 UTC

AbstractList::lastIndexOf() A

↳ Parent: AbstractList

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes

Metric	Value
dl	0
loc	9
ccs	0
cts	6
cp	0
rs	9.9666
c	0
b	0
f	0
cc	2
nc	2
nop	1
crap	6

<?php
/*
 * Copyright (c) Nate Brunette.
 * Distributed under the MIT License (http://opensource.org/licenses/MIT)
 */

namespace Tebru\Collection;

/**
 * Class AbstractList
 *
 * A skeletal implementation of the [@see ListInterface]
 *
 * @author Nate Brunette <[email protected]>
 */
abstract class AbstractList extends AbstractCollection implements ListInterface
{
    /**
     * Returns the index of the first instance of the element, -1 if the element
     * doesn't exist
     *
     * @param mixed $element
     * @return int
     */
    public function indexOf($element): int
    {
        $index = array_search($element, $this->toArray(), true);

        return false === $index ? -1 : $index;
    }

    /**
     * Returns the index of the last instance of the element, -1 if the element
     * doesn't exist
     *
     * @param mixed $element
     * @return int
     */
    public function lastIndexOf($element): int
    {
        $elements = array_reverse($this->toArray());

        $index = array_search($element, $elements, true);

        // return -1 if not found or (size - index found - 1) to return the index of the element after reverse
        return false === $index ? -1 : $this->count() - $index - 1;
    }
}


1			<?php
2			/*
3			* Copyright (c) Nate Brunette.
4			* Distributed under the MIT License (http://opensource.org/licenses/MIT)
5			*/
6
7			namespace Tebru\Collection;
8
9			/**
10			* Class AbstractList
11			*
12			* A skeletal implementation of the [@see ListInterface]
13			*
14			* @author Nate Brunette <[email protected]>
15			*/
16			abstract class AbstractList extends AbstractCollection implements ListInterface
17			{
18			/**
19			* Returns the index of the first instance of the element, -1 if the element
20			* doesn't exist
21			*
22			* @param mixed $element
23			* @return int
24			*/
25			public function indexOf($element): int
26			{
27			$index = array_search($element, $this->toArray(), true);
28
29			return false === $index ? -1 : $index;
30			}
31
32			/**
33			* Returns the index of the last instance of the element, -1 if the element
34			* doesn't exist
35			*
36			* @param mixed $element
37			* @return int
38			*/
39			public function lastIndexOf($element): int
40			{
41			$elements = array_reverse($this->toArray());
42
43			$index = array_search($element, $elements, true);
44
45			// return -1 if not found or (size - index found - 1) to return the index of the element after reverse
46			return false === $index ? -1 : $this->count() - $index - 1;
47			}
48			}
49

tebru / php-collections

Push — master ( e5d322...90ee42 )

AbstractList::lastIndexOf() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like