Seboettg\Collection\Map\Functions

Complexity

Total Complexity

5

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	hp$0 ➔ emptyMap()	0	5	1
	emptyMap()	0	5	?
A	hp$0 ➔ pair()	0	3	1
	pair()	0	3	?
A	hp$0 ➔ mapOf()	0	11	3
	mapOf()	0	11	?

<?php
declare(strict_types=1);
/*
 * Copyright (C) 2022 Sebastian Böttger <seboettg@gmail.com>
 * You may use, distribute and modify this code under the
 * terms of the MIT license.
 *
 * You should have received a copy of the MIT license with
 * this file. If not, please visit: https://opensource.org/licenses/mit-license.php
 */

namespace Seboettg\Collection\Map;

use InvalidArgumentException;

final class Functions
{
    /**
     * @return MapInterface
     */
    public static final function emptyMap(): MapInterface

As per PSR2, final should precede the visibility keyword.

    {
        return new class() implements MapInterface {
            private $array = [];

Private member variable "array" must contain a leading underscore

The private property $array is not used, and could be removed.

            use MapTrait;
        };
    }

    /**
     * @param array<Pair> ...$pairs
     * @return MapInterface
     */
    public final static function mapOf(...$pairs): MapInterface

As per PSR2, final should precede the visibility keyword.

    {
        $map = emptyMap();
        foreach ($pairs as $pair) {
            if (!$pair instanceof Pair) {
                throw new InvalidArgumentException(sprintf(
                    "At least one pair is not of type %s", Pair::class));
            }
            $map->put($pair->getKey(), $pair->getValue());
        }
        return $map;
    }

    /**
     * @param scalar $key
     * @param mixed $value
     * @return Pair
     */
    public final static function pair($key, $value): Pair

As per PSR2, final should precede the visibility keyword.

    {
        return new Pair($key, $value);
    }
}

/**
 * @return MapInterface
 */
function emptyMap(): MapInterface
{
    return Functions::emptyMap();
}

/**
 * @param array<Pair> ...$pairs
 * @return MapInterface
 */
function mapOf(...$pairs): MapInterface
{
    return Functions::mapOf(...$pairs);
}

/**
 * @param scalar $key
 * @param mixed $value
 * @return Pair
 */
function pair($key, $value): Pair
{
    return Functions::pair($key, $value);
}