MethodAdded::compare() - Code Metrics - Inspection of "Verify method, function, property, constant and in..." - Roave/BackwardCompatibilityCheck - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#38)

by Marco

created 2018-04-22 12:23 UTC

MethodAdded::compare() A

↳ Parent: MethodAdded

Complexity

Conditions	2
Paths	2

Size

Total Lines	22
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	22
rs	9.2
c	0
b	0
f	0
cc	2
eloc	15
nc	2
nop	2

<?php

declare(strict_types=1);

namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\InterfaceBased;

use Roave\ApiCompare\Change;
use Roave\ApiCompare\Changes;
use Roave\BetterReflection\Reflection\ReflectionClass;
use Roave\BetterReflection\Reflection\ReflectionMethod;
use function array_combine;
use function array_diff_key;
use function array_map;
use function array_values;
use function sprintf;
use function strtolower;

final class MethodAdded implements InterfaceBased
{
    public function compare(ReflectionClass $fromInterface, ReflectionClass $toInterface) : Changes
    {
        $fromMethods = $this->methods($fromInterface);
        $toMethods   = $this->methods($toInterface);
        $newMethods  = array_diff_key($toMethods, $fromMethods);

        if (! $newMethods) {

            return Changes::new();
        }

        return Changes::fromArray(array_values(array_map(function (ReflectionMethod $method) use (
            $fromInterface
        ) : Change {
            return Change::added(
                sprintf(
                    'Method %s() was added to interface %s',
                    $method->getName(),
                    $fromInterface->getName()
                ),
                true
            );
        }, $newMethods)));
    }

    /** @return ReflectionMethod[] indexed by lowercase method name */
    private function methods(ReflectionClass $interface) : array
    {
        $methods = $interface->getMethods();

        return array_combine(
            array_map(function (ReflectionMethod $method) : string {
                return strtolower($method->getName());
            }, $methods),
            $methods
        );
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\InterfaceBased;
6
7			use Roave\ApiCompare\Change;
8			use Roave\ApiCompare\Changes;
9			use Roave\BetterReflection\Reflection\ReflectionClass;
10			use Roave\BetterReflection\Reflection\ReflectionMethod;
11			use function array_combine;
12			use function array_diff_key;
13			use function array_map;
14			use function array_values;
15			use function sprintf;
16			use function strtolower;
17
18			final class MethodAdded implements InterfaceBased
19			{
20			public function compare(ReflectionClass $fromInterface, ReflectionClass $toInterface) : Changes
21			{
22			$fromMethods = $this->methods($fromInterface);
23			$toMethods = $this->methods($toInterface);
24			$newMethods = array_diff_key($toMethods, $fromMethods);
25
26			if (! $newMethods) {
			0 ignored issues – show Bug Best Practice introduced 2018-04-16 22:38 UTC by Report Bug Copy Issue Report The expression `$newMethods` of type `array` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
27			return Changes::new();
28			}
29
30			return Changes::fromArray(array_values(array_map(function (ReflectionMethod $method) use (
31			$fromInterface
32			) : Change {
33			return Change::added(
34			sprintf(
35			'Method %s() was added to interface %s',
36			$method->getName(),
37			$fromInterface->getName()
38			),
39			true
40			);
41			}, $newMethods)));
42			}
43
44			/** @return ReflectionMethod[] indexed by lowercase method name */
45			private function methods(ReflectionClass $interface) : array
46			{
47			$methods = $interface->getMethods();
48
49			return array_combine(
50			array_map(function (ReflectionMethod $method) : string {
51			return strtolower($method->getName());
52			}, $methods),
53			$methods
54			);
55			}
56			}
57

Roave / BackwardCompatibilityCheck

Pull Request — master (#38)

MethodAdded::compare() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like