Seboettg\Collection\Map\Functions

Complexity

Total Complexity

3

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	0	Features	0

6 Methods

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

<?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();
        $map->setArray(array_combine(
            array_map(fn (Pair $pair) => $pair->getKey(), $pairs), //keys
            array_map(fn (Pair $pair) => $pair->getValue(), $pairs) //values
        ));
        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);
}