GeoTimeZone\Calculator

Complexity

Total Complexity

19

Size/Duplication

Total Lines	114
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

97.78%

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	3
A	adjustLatitude()	0	11	4
A	adjustLongitude()	0	11	4
A	getTimeZoneName()	0	12	2
A	getLocalDate()	0	10	2
A	getCorrectTimestamp()	0	14	4

<?php

namespace GeoTimeZone;

use DateTime;
use DateTimeZone;
use ErrorException;
use GeoTimeZone\Quadrant\Tree;

class Calculator
{
    protected $quadrantTree;
    
    /**
     * TimeZone constructor.
     * @param $dataDirectory
     * @throws ErrorException
     */
    public function __construct($dataDirectory = null)
    {
        if (isset($dataDirectory) && is_dir($dataDirectory)) {
            $this->quadrantTree = new Tree($dataDirectory);
            $this->quadrantTree->initializeDataTree();
        }else{
            throw new ErrorException('Invalid data tree directory: ' . $dataDirectory);
        }
    }
    
    /**
     * Adjust the latitude value
     * @param $latitude
     * @return float|int
     * @throws ErrorException
     */
    protected function adjustLatitude($latitude)
    {
        $newLatitude = $latitude;
        if (null == $latitude || abs($latitude) > Tree::MAX_ABS_LATITUDE) {
            throw new ErrorException('Invalid latitude: ' . $latitude);
        }
        if (abs($latitude) == Tree::MAX_ABS_LATITUDE) {
            $newLatitude = ($latitude <=> 0) * Tree::ABS_LATITUDE_LIMIT;
        }
        return $newLatitude;
    }
    
    /**
     * Adjust longitude value
     * @param $longitude
     * @return float|int
     * @throws ErrorException
     */
    protected function adjustLongitude($longitude)
    {
        $newLongitude = $longitude;
        if (null == $longitude || abs($longitude) > Tree::MAX_ABS_LONGITUDE) {
            throw new ErrorException('Invalid longitude: ' . $longitude);
        }
        if (abs($longitude) == Tree::MAX_ABS_LONGITUDE) {
            $newLongitude = ($longitude <=> 0) * Tree::ABS_LONGITUDE_LIMIT;
        }
        return $newLongitude;
    }
    
    /**
     * Get timezone name from a particular location (latitude, longitude)
     * @param $latitude
     * @param $longitude
     * @return string
     */
    public function getTimeZoneName($latitude, $longitude)
    {
        $timeZone = Tree::NONE_TIMEZONE;
        try {
            $latitude = $this->adjustLatitude($latitude);
            $longitude = $this->adjustLongitude($longitude);
            $timeZone = $this->quadrantTree->lookForTimezone($latitude, $longitude);
        }catch (ErrorException $error){

The class ErrorException does not exist. Is this class maybe located in a folder that is not analyzed, or in a newer version of your dependencies than listed in your composer.lock/composer.json?

            echo $error->getMessage() . "\n";
        }
        return $timeZone;
    }
    
    /**
     * Get the local date belonging to a particular latitude, longitude and timestamp
     * @param $latitude
     * @param $longitude
     * @param $timestamp
     * @return DateTime
     */
    public function getLocalDate($latitude, $longitude, $timestamp)
    {
        $timeZone = $this->getTimeZoneName($latitude, $longitude);
        $date = new DateTime();
        $date->setTimestamp($timestamp);
        if ($timeZone != Tree::NONE_TIMEZONE) {
            $date->setTimezone(new DateTimeZone($timeZone));
        }
        return $date;
    }
    
    /**
     * Get timestamp from latitude, longitude and localTimestamp
     * @param $latitude
     * @param $longitude
     * @param $localTimestamp
     * @return mixed
     */
    public function getCorrectTimestamp($latitude, $longitude, $localTimestamp)
    {
        $timestamp = $localTimestamp;
        $timeZoneName = $this->getTimeZoneName($latitude, $longitude);
        if ($timeZoneName != Tree::NONE_TIMEZONE) {
            $date = new DateTime();
            $date->setTimestamp($localTimestamp);
            if ($timeZoneName != null) {
                $date->setTimezone(new DateTimeZone($timeZoneName));
            }
            $timestamp = $date->getOffset() != false ? $localTimestamp - $date->getOffset() : $localTimestamp;

It seems like you are loosely comparing $date->getOffset() of type integer to the boolean false. If you are specifically checking for non-zero, consider using something more explicit like > 0 or !== 0 instead.

        }
        return $timestamp;
    }
}