Netgen\Bundle\OpenWeatherMapBundle\Controller\AirPollutionController

Complexity

Total Complexity

11

Size/Duplication

Total Lines	87
Duplicated Lines	57.47 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getOzoneData()	25	25	5
A	getCarbonMonoxideData()	25	25	5

<?php

namespace Netgen\Bundle\OpenWeatherMapBundle\Controller;

use DateTime;
use Netgen\Bundle\OpenWeatherMapBundle\API\OpenWeatherMap\Weather\AirPollutionInterface;
use Netgen\Bundle\OpenWeatherMapBundle\Exception\NotAuthorizedException;
use Netgen\Bundle\OpenWeatherMapBundle\Exception\NotFoundException;
use Symfony\Component\HttpFoundation\Response;

/**
 * Class AirPollutionController.
 */
class AirPollutionController
{
    /**
     * @var \Netgen\Bundle\OpenWeatherMapBundle\API\OpenWeatherMap\Weather\AirPollutionInterface
     */
    protected $airPollution;

    /**
     * AirPollutionController constructor.
     *
     * @param \Netgen\Bundle\OpenWeatherMapBundle\API\OpenWeatherMap\Weather\AirPollutionInterface $airPollution
     */
    public function __construct(AirPollutionInterface $airPollution)
    {
        $this->airPollution = $airPollution;
    }

    /**
     * Returns ozone data.
     *
     * @param float $latitude
     * @param float $longitude
     * @param string $datetime ISO 8601 date string
     *
     * @return \Symfony\Component\HttpFoundation\Response
     */
    public function getOzoneData($latitude, $longitude, $datetime = 'current')

This method seems to be duplicated in your project.

    {
        if ($datetime !== 'current') {
            $datetime = DateTime::createFromFormat('c', $datetime);

            if ($datetime === false) {
                $datetime = 'current';
            }
        }

        $response = new Response();

        try {
            $data = $this->airPollution->fetchOzoneData($latitude, $longitude, $datetime);
            $response->setContent($data);
        } catch (NotAuthorizedException $e) {
            $response->setContent($e->getMessage());
            $response->setStatusCode(Response::HTTP_UNAUTHORIZED);
        } catch (NotFoundException $e) {
            $response->setContent($e->getMessage());
            $response->setStatusCode(Response::HTTP_NOT_FOUND);
        }

        return $response;
    }

    /**
     * Returns carbon monoxide data.
     *
     * @param float $latitude
     * @param float $longitude
     * @param string $datetime ISO 8601 date string
     *
     * @return \Symfony\Component\HttpFoundation\Response
     */
    public function getCarbonMonoxideData($latitude, $longitude, $datetime = 'current')

This method seems to be duplicated in your project.

    {
        if ($datetime !== 'current') {
            $datetime = DateTime::createFromFormat('c', $datetime);

            if ($datetime === false) {
                $datetime = 'current';
            }
        }

        $response = new Response();

        try {
            $data = $this->airPollution->fetchCarbonMonoxideData($latitude, $longitude, $datetime);
            $response->setContent($data);
        } catch (NotAuthorizedException $e) {
            $response->setContent($e->getMessage());
            $response->setStatusCode(Response::HTTP_UNAUTHORIZED);
        } catch (NotFoundException $e) {
            $response->setContent($e->getMessage());
            $response->setStatusCode(Response::HTTP_NOT_FOUND);
        }

        return $response;
    }
}