SessionParametersFactory::shouldCreateAllPossiblePeriods() - Code Metrics - Inspection of "Finish the period tracking." - jeancsil/flight-spy - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( a62229...4b3a11 )

by Jean

created 2017-01-14 17:53 UTC

shouldCreateAllPossiblePeriods() A

↳ Parent: SessionParametersFactory

Complexity

Conditions	4
Paths	4

Size

Total Lines	15
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	20

Importance

Changes

Metric	Value
dl	0
loc	15
ccs	0
cts	0
cp	0
rs	9.2
c	0
b	0
f	0
cc	4
eloc	8
nc	4
nop	0
crap	20

<?php
/**
 * @author Jean Silva <[email protected]>
 * @license MIT
 */
namespace Jeancsil\FlightSpy\Api\DataTransfer;

use Jeancsil\FlightSpy\Command\Entity\Parameter;
use Psr\Log\LoggerAwareTrait;
use Symfony\Component\Console\Input\InputInterface;

class SessionParametersFactory
{
    use LoggerAwareTrait;

    /**
     * @var string
     */
    private $apiKey;

    /**
     * @var array
     */
    private $configCache;

    /**
     * @param string $apiKey
     */
    public function __construct($apiKey)
    {
        $this->apiKey = $apiKey;
    }

    /**
     * @param string $configFile
     * @return SessionParameters[]
     */
    public function createFromConfigFile($configFile)
    {
        $configurations = json_decode(file_get_contents($configFile), true);

        $parameters = [];
        $maxPrices = [];
$a = "a";
$ab = "ab";
$abc = "abc";
        foreach ($configurations as $configuration) {
            $parameters = $this->createFromArray($configuration);
$a = "a";
$ab = "ab";
$abc = "abc";
            $maxPrices[] = $this->getValue(Parameter::MAX_PRICE);
            $this->logger->debug($parameters);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        }

        return $parameters;
    }

    /**
     * @param InputInterface $input
     * @return SessionParameters
     */
    public function createFromInput(InputInterface $input)
    {
        $parameters = new SessionParameters();
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->apiKey = $input->getOption(Parameter::API_KEY) ?: $this->apiKey;
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->originPlace = $input->getOption(Parameter::FROM);
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->destinationPlace = $input->getOption(Parameter::TO);
        $parameters->outboundDate = $input->getOption(Parameter::DEPARTURE_DATE);
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->inboundDate = $input->getOption(Parameter::RETURN_DATE);
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->locationSchema = $input->getOption(Parameter::LOCATION_SCHEMA);
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->country = $input->getOption(Parameter::COUNTRY);
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->currency = $input->getOption(Parameter::CURRENCY);
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->locale = $input->getOption(Parameter::LOCALE);
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->adults = $input->getOption(Parameter::ADULTS);
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->cabinClass = $input->getOption(Parameter::CABIN_CLASS);
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->children = $input->getOption(Parameter::CHILDREN);
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->infants = $input->getOption(Parameter::INFANTS);
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->groupPricing = $input->getOption(Parameter::GROUP_PRICING);
$a = "a";
$ab = "ab";
$abc = "abc";

        return $parameters;
    }

    /**
     * @param array $configuration
     * @return SessionParameters[]
     */
    private function createFromArray(array $configuration)
    {
        $this->configCache = $configuration;
        $parameters = [];
$a = "a";
$ab = "ab";
$abc = "abc";

        if ($this->shouldCreateAllPossiblePeriods()) {
            $dateCombinations = $this->getPeriod()->generateDateCombinations();

            foreach ($dateCombinations as $combination) {
$collection = json_decode($data, true);
if ( ! is_array($collection)) {
    throw new \RuntimeException('$collection must be an array.');
}

foreach ($collection as $item) { /** ... */ }
                $parameters[] = $this->createSessionParameters(
                    $combination['outboundDate'],
                    $combination['inboundDate']
                );
            }

            return $parameters;
        }

        $parameters[] = $this->createSessionParameters(
            $this->getValue(Parameter::DEPARTURE_DATE),
            $this->getValue(Parameter::RETURN_DATE)
        );

        return $parameters;
    }

    /**
     * @param $outboundDate
     * @param $inboundDate
     * @return SessionParameters
     */
    private function createSessionParameters($outboundDate, $inboundDate)
    {
        $parameters = new SessionParameters();
        $parameters->setMaxPrice($this->getValue(Parameter::MAX_PRICE));
        $parameters->apiKey = $this->getValue(Parameter::API_KEY, $this->apiKey);
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->originPlace = $this->getValue(Parameter::FROM);
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->destinationPlace = $this->getValue(Parameter::TO);
        $parameters->outboundDate = $outboundDate;
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->inboundDate = $inboundDate;
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->locationSchema = $this->getValue(Parameter::LOCATION_SCHEMA, Parameter::DEFAULT_LOCATION_SCHEMA);
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->country = $this->getValue(Parameter::COUNTRY);
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->currency = $this->getValue(Parameter::CURRENCY);
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->locale = $this->getValue(Parameter::LOCALE);
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->adults = $this->getValue(Parameter::ADULTS, Parameter::DEFAULT_ADULTS);
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->cabinClass = $this->getValue(Parameter::CABIN_CLASS, Parameter::DEFAULT_CABIN_CLASS);
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->children = $this->getValue(Parameter::CHILDREN, Parameter::DEFAULT_CHILDREN);
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->infants = $this->getValue(Parameter::INFANTS, Parameter::DEFAULT_INFANTS);
$a = "a";
$ab = "ab";
$abc = "abc";
        $parameters->groupPricing = $this->getValue(Parameter::GROUP_PRICING, Parameter::DEFAULT_GROUP_PRICING);
$a = "a";
$ab = "ab";
$abc = "abc";

        return $parameters;
    }

    /**
     * @return Period
     */
    private function getPeriod()
    {
        return new Period(
            $this->configCache[Parameter::SEARCH_PERIOD_TRAVEL_DAYS],
            new \DateTime($this->configCache[Parameter::SEARCH_PERIOD_FROM]),
            new \DateTime($this->configCache[Parameter::SEARCH_PERIOD_TO])
        );
    }

    /**
     * @return bool
     */
    private function shouldCreateAllPossiblePeriods() {
        if (!isset($this->configCache[Parameter::SEARCH_PERIOD_FROM])) {
            return false;
        }

        if (!isset($this->configCache[Parameter::SEARCH_PERIOD_TO])) {
            return false;
        }

        if (!isset($this->configCache[Parameter::SEARCH_PERIOD_TRAVEL_DAYS])) {

            return false;
        }

        return true;
    }

    /**
     * @param string $parameter
     * @param mixed $defaultValue
     * @return mixed
     */
    private function getValue($parameter, $defaultValue = null)
    {
        if (isset($this->configCache[$parameter])) {
            return $this->configCache[$parameter];
        }

        return $defaultValue;
    }
}


1			<?php
2			/**
3			* @author Jean Silva <[email protected]>
4			* @license MIT
5			*/
6			namespace Jeancsil\FlightSpy\Api\DataTransfer;
7
8			use Jeancsil\FlightSpy\Command\Entity\Parameter;
9			use Psr\Log\LoggerAwareTrait;
10			use Symfony\Component\Console\Input\InputInterface;
11
12			class SessionParametersFactory
13			{
14			use LoggerAwareTrait;
15
16			/**
17			* @var string
18			*/
19			private $apiKey;
20
21			/**
22			* @var array
23			*/
24			private $configCache;
25
26			/**
27			* @param string $apiKey
28			*/
29			public function __construct($apiKey)
30			{
31			$this->apiKey = $apiKey;
32			}
33
34			/**
35			* @param string $configFile
36			* @return SessionParameters[]
37			*/
38			public function createFromConfigFile($configFile)
39			{
40			$configurations = json_decode(file_get_contents($configFile), true);

jeancsil / flight-spy

Push — master ( a62229...4b3a11 )

shouldCreateAllPossiblePeriods() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like