Zones::search() - Code Metrics - Inspection of "Merge branch 'release/1.0.2'" - filmtools/commons - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( e47858...55d38d )

by Carsten

created 2018-07-30 20:32 UTC

Zones::search() A

↳ Parent: Zones

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes

Metric	Value
dl	0
loc	4
ccs	2
cts	2
cp	1
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	1
crap	1

<?php
namespace FilmTools\Commons;


class Zones extends MinMaxArrayIterator implements ZonesInterface, ZonesProviderInterface
{
    /**
     * @inheritDoc
     * @return $this
     */
    public function getZones() : ZonesInterface
    {
        return $this;
    }

    /**
     * @inheritDoc
     */
    public function search( float $zone )
    {
        return array_search($zone, $this->getArrayCopy());

    }


    /**
     * @inheritDoc
     * @return Exposures
     */
    public function getExposures() : ExposuresInterface
    {
        return new Exposures( array_map( function( $zone ) {
            return $zone * log10(2);
        }, $this->getArrayCopy()));
    }
}


1		<?php
2		namespace FilmTools\Commons;
3
4
5		class Zones extends MinMaxArrayIterator implements ZonesInterface, ZonesProviderInterface
6		{
7		/**
8		* @inheritDoc
9		* @return $this
10		*/
11	9	public function getZones() : ZonesInterface
12		{
13	9	return $this;
14		}
15
16		/**
17		* @inheritDoc
18		*/
19	9	public function search( float $zone )
20		{
21	9	return array_search($zone, $this->getArrayCopy());
		0 ignored issues – show Bug Compatibility introduced 2018-07-30 20:35 UTC by Report Bug Copy Issue Report The expression `array_search($zone, $this->getArrayCopy());` of type `false\|integer\|string` adds the type `string` to the return on line 21 which is incompatible with the return type declared by the interface `FilmTools\Commons\SearchableInterface::search` of type `integer\|false`. Loading history...
22		}
23
24
25		/**
26		* @inheritDoc
27		* @return Exposures
28		*/
29		public function getExposures() : ExposuresInterface
30		{
31	9	return new Exposures( array_map( function( $zone ) {
32	9	return $zone * log10(2);
33	9	}, $this->getArrayCopy()));
34		}
35		}
36

filmtools / commons

Push — master ( e47858...55d38d )

Zones::search() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like