Submarine\NbrbExchangeRatesBundle\CurrencyRateDate

Complexity

Total Complexity

5

Size/Duplication

Total Lines	57
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes	2
Bugs	1	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	createFromXML()	0	13	2
A	getRate()	0	4	1
A	getDate()	0	4	1

<?php
/**
 * (c) itmedia.by <info@itmedia.by>
 */

namespace Submarine\NbrbExchangeRatesBundle;


class CurrencyRateDate
{
    /**
     * @var \DateTime
     */
    private $date;

    /**
     * @var float
     */
    private $rate;

    /**
     * CurrencyRateDate constructor.
     * @param \DateTime $date
     * @param float $rate
     */
    public function __construct(\DateTime $date, $rate)
    {
        $this->date = $date;
        $this->rate = $rate;
    }


    static public function createFromXML(\SimpleXMLElement $xmlElement = null)

As per PSR2, the static declaration should come after the visibility declaration.

    {
        $dateArray = explode('/', (string)$xmlElement->attributes()[0]);

It seems like $xmlElement is not always an object, but can also be of type null. Maybe add an additional type check?

        if (count($dateArray) === 3) {
            $dateTime = new \DateTime();
            $dateTime->setDate($dateArray[2], $dateArray[0], $dateArray[1]);
            $dateTime->setTime(0, 0, 0);

            return new CurrencyRateDate($dateTime, (float)$xmlElement->Rate);
        }

        return new CurrencyRateDate(new \DateTime(), null);
    }


    /**
     * @return float
     */
    public function getRate()
    {
        return $this->rate;
    }

    /**
     * @return \DateTime
     */
    public function getDate()
    {
        return $this->date;
    }


}