Jeancsil\FlightSpy\Api\Flights\LivePrice

Complexity

Total Complexity

5

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getDeals()	0	4	1
A	getMultiDeals()	0	18	4

<?php
/**
 * @author Jean Silva <me@jeancsil.com>
 * @license MIT
 */
namespace Jeancsil\FlightSpy\Api\Flights;

use Jeancsil\FlightSpy\Api\DataTransfer\SessionParameters;
use Jeancsil\FlightSpy\Api\Http\TransportAwareTrait;

class LivePrice
{
    use TransportAwareTrait;

    /**
     * @param SessionParameters $parameters
     * @param bool $newSession
     * @return array

Should the return type not be \stdClass?

     */
    public function getDeals(SessionParameters $parameters, $newSession)
    {
        return $this->transport->findQuotes($parameters, $newSession);
    }

    /**
     * @param array $sessionParameters
     * @return array
     */
    public function getMultiDeals(array $sessionParameters)
    {
        $responses = [];

Equals sign not aligned with surrounding assignments; expected 7 spaces but found 1 space

        static $requests = 0;
        foreach ($sessionParameters as $sessionParameter) {
            if (!$sessionParameter instanceof SessionParameters) {
                throw new \LogicException(sprintf('Instance of SessionParameters need. Given %s.', $sessionParameter));
            }

            $newSession = ($requests % 20) == 0;
            if ($response = $this->getDeals($sessionParameter, $newSession)) {
                $responses[] = $response;
                $requests++;
            }
        }

        return $responses;
    }
}