CoC_Location

Complexity

Total Complexity

17

Size/Duplication

Total Lines	137
Duplicated Lines	18.98 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	13	2
A	getLocation()	0	19	4
A	setLocationByName()	13	13	3
A	setLocationByCode()	13	13	3
A	getLocationId()	0	4	1
A	getLocationName()	0	4	1
A	isCountry()	0	4	1
A	getCountryCode()	0	11	2

<?php

class CoC_Location
{
	protected $api;
	protected $locationId;
	protected $location = NULL;
    
	/**
	 * Constructor of CoC_Location
	 * 
	 * @param $locationId
	 */
	public function __construct($location)
	{
		$this->api = new ClashOfClans();
		if(is_object($location))
		{
			$this->location = $location;
		}
		else
		{
			$this->locationId = $location;
			$this->getLocation();
		}
	}
    
	/**
	 * Gets an stdClass containing location-information
	 *
	 * @return stdClass, location

The doc-type stdClass, could not be parsed: Expected "|" or "end of type", but got "," at position 8. (view supported doc-types)

	 */
	protected function getLocation()
	{
		if($this->location == NULL)
		{
			foreach ($this->api->getLocationList()->items as $location) 
			{
				if ($location->id == $this->locationId) 
				{
					$this->location = $location;
					return $this->location;
				}
			}
			return 0;
		}
		else
		{
			return $this->location;
		}
	}
    
	/**
	 * Sets the location by providing it's exact name
	 *
	 * @param string, location name
	 * @return bool, success or fail

The doc-type bool, could not be parsed: Expected "|" or "end of type", but got "," at position 4. (view supported doc-types)

	 */
	public function setLocationByName($name)

This method seems to be duplicated in your project.

	{
		foreach ($this->api->getLocationList()->items as $location) 
		{
			if ($location->name == $name) 
			{
				$this->locationId = $location->id;
				$this->location = $location;
				return 1;
			}
		}
		return 0;
	}

	/**
	 * Sets the location by providing it's country code
	 *
     * @param string
     * @return bool
     */
	public function setLocationByCode($cc)

This method seems to be duplicated in your project.

	{
		foreach ($this->api->getLocationList()->items as $location) 
		{
			if ($location->countryCode == $cc) 
			{
				$this->locationId = $location->id;
				$this->location = $location;
				return 1;
			}
		}
		return 0;
	}

	/**
	 * Gets the location ID
	 *
	 * @return int
	 */
	public function getLocationId()
	{
		return $this->getLocation()->id;
	}

	/**
	 * Gets the location name
	 *
	 * @return string, name

The doc-type string, could not be parsed: Expected "|" or "end of type", but got "," at position 6. (view supported doc-types)

	 */
	public function getLocationName()
	{
		return $this->getLocation()->name;
	}

	/**
	 * Check whether the given location is a country or a region
	 *
	 * @return bool
	 */
	public function isCountry()
	{
		return $this->getLocation()->isCountry;
	}

	/**
	 * Gets the country code for the given location
	 *
	 * @return string, country code

The doc-type string, could not be parsed: Expected "|" or "end of type", but got "," at position 6. (view supported doc-types)

	 */
	public function getCountryCode()
	{
		if($this->isCountry())
		{
			return $this->getLocation()->countryCode;
		}
		else
		{
			return "nA";
		}
	}
}

?>

It is not recommended to use PHP's closing tag ?> in files other than templates.