SmsGate\ResultCollection

Complexity

Total Complexity

12

Size/Duplication

Total Lines	116
Duplicated Lines	13.79 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	findByPhone()	0	13	3
A	getSuccessfullyResults()	8	8	1
A	getFailedResults()	8	8	1
A	current()	0	4	1
A	next()	0	4	1
A	key()	0	4	1
A	valid()	0	4	1
A	rewind()	0	4	1
A	count()	0	4	1

<?php

declare(strict_types = 1);

/*
 * This file is part of the SmsGate package
 *
 * (c) SmsGate
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code
 */

namespace SmsGate;

use SmsGate\Exception\ResultNotFoundException;

/**
 * The collection for store results of send sms.
 *
 * @author Vitaliy Zhuk <zhuk2205@gmail.com>
 */
class ResultCollection implements \Iterator, \Countable
{
    /**
     * @var Result[]
     */
    private $results;

    /**
     * Constructor.
     *
     * @param Result[] ...$results
     */
    public function __construct(Result ...$results)
    {
        $this->results = $results;

It seems like $results of type array<integer,array<integer,object<SmsGate\Result>>> is incompatible with the declared type array<integer,object<SmsGate\Result>> of property $results.

    }

    /**
     * Find the result by phone
     *
     * @param Phone $phone
     *
     * @return Result
     *
     * @throws ResultNotFoundException
     */
    public function findByPhone(Phone $phone): Result
    {
        foreach ($this->results as $result) {
            if ($result->getPhone()->getValue() === $phone->getValue()) {
                return $result;
            }
        }

        throw new ResultNotFoundException(sprintf(
            'Not found result for phone "%s".',
            $phone
        ));
    }

    /**
     * Get successfully results
     *
     * @return ResultCollection
     */
    public function getSuccessfullyResults(): ResultCollection

This method seems to be duplicated in your project.

    {
        $successfullyResults = array_filter($this->results, function (Result $result) {
            return $result->isSuccessfully();
        });

        return new self(...$successfullyResults);
    }

    /**
     * Get failed results
     *
     * @return ResultCollection
     */
    public function getFailedResults(): ResultCollection

This method seems to be duplicated in your project.

    {
        $failedResults = array_filter($this->results, function (Result $result) {
            return $result->isFailed();
        });

        return new self(...$failedResults);
    }

    /**
     * {@inheritdoc}
     */
    public function current(): Result
    {
        return current($this->results);
    }

    /**
     * {@inheritdoc}
     */
    public function next(): void
    {
        next($this->results);
    }

    /**
     * {@inheritdoc}
     */
    public function key()
    {
        return key($this->results);
    }

    /**
     * {@inheritdoc}
     */
    public function valid(): bool
    {
        return key($this->results) !== null;
    }

    /**
     * {@inheritdoc}
     */
    public function rewind(): void
    {
        reset($this->results);
    }

    /**
     * {@inheritdoc}
     */
    public function count(): int
    {
        return count($this->results);
    }
}