Roave\ApiCompare\Comparator\BackwardsCompatibility\InterfaceBased\MethodAdded

Complexity

Total Complexity

3

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	compare()	0	22	2
A	methods()	0	9	1

<?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;

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) {

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.

            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
        );
    }
}