Mvg\RequestHandler\Html\HttpGetDepartures

Complexity

Total Complexity

15

Size/Duplication

Total Lines	148
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	1
Bugs	0	Features	0

14 Methods

Rating	Name	Duplication	Size	Complexity
A	getStations()	0	3	1
A	getNewsTicker()	0	3	1
A	getSchema()	0	3	1
A	setSchema()	0	3	1
A	getHost()	0	3	1
A	setHost()	0	3	1
A	getPath()	0	3	1
A	setPath()	0	3	1
A	getParameter()	0	3	1
A	addParameter()	0	3	1
A	setParameter()	0	3	1
A	__construct()	0	15	1
A	doGetRequest()	0	17	2
A	getDeparturesForStation()	0	5	1

<?php
/**
 * User: ms
 * Date: 29.08.15
 * Time: 09:35
 */
namespace Mvg\RequestHandler\Html;

use Zend\Http\Client;

/**
 * Class Http
 * @package Mvg
 */
class HttpGetDepartures {
	/**
	 * @var string
	 */
	protected $schema;
	/**
	 * @var string
	 */
	protected $host;
	/**
	 * @var string
	 */
	protected $path;
	/**
	 * @var array
	 */
	protected $parameter;

	/**
	 * @param $schema
	 * @param $host
	 * @param $path
	 */
	public function __construct($schema, $host, $path) {
		$this->setHost($host);
		$this->setSchema($schema);
		$this->setPath($path);

		$this->setParameter(
			[
				'ubahn' => 'checked'
				, 'bus' => 'checked'
				, 'tram' => 'checked'
				, 'sbahn' => 'checked'
			]
		);

	}

	/**
	 * @return string
	 * @throws \Exception
	 */
	protected function doGetRequest() {

		$url = sprintf('%s://%s/%s?%s'

Equals sign not aligned with surrounding assignments; expected 4 spaces but found 1 space

			, $this->getSchema()
			, $this->getHost()
			, $this->getPath()
			, http_build_query($this->getParameter(), PHP_QUERY_RFC3986)
		);
		$client = new Client();
		$client->setUri($url)
			->setMethod(\Zend\Http\Request::METHOD_GET);
		$response = $client->send();
		if ($response->getStatusCode() !== 200) {
			throw new \Exception(sprintf('Request Response: Status Code %d', $response->getStatusCode()));
		}
		return utf8_encode($response->getBody());
	}

	/**
	 * @return string

Should the return type not be string|null?

	 */
	public function getStations() {
		//@todo implement me

Comments for TODO tasks are often forgotten in the code; it might be better to use a dedicated issue tracker.

	}

	/**
	 * @param $station
	 * @return string
	 */
	public function getDeparturesForStation($station) {
		$this->addParameter('haltestelle', $station);
		$htmlResponse = $this->doGetRequest();
		return $htmlResponse;
	}

	public function getNewsTicker() {

	}

	/**
	 * @return string
	 */
	protected function getSchema() {
		return $this->schema;
	}

	/**
	 * @param string $schema
	 */
	protected function setSchema($schema) {
		$this->schema = $schema;
	}

	/**
	 * @return string
	 */
	protected function getHost() {
		return $this->host;
	}

	/**
	 * @param string $host
	 */
	protected function setHost($host) {
		$this->host = $host;
	}

	/**
	 * @return string
	 */
	protected function getPath() {
		return $this->path;
	}

	/**
	 * @param string $path
	 */
	protected function setPath($path) {
		$this->path = $path;
	}

	/**
	 * @return string

Should the return type not be array? Also, consider making the array more specific, something like array<String>, or String[].

	 */
	public function getParameter() {
		return $this->parameter;
	}

	/**
	 * @param string $key
	 * @param string $value
	 */
	protected function addParameter($key, $value) {
		$this->parameter[$key] = utf8_decode($value);
	}

	/**
	 * @param array $parameter
	 */
	protected function setParameter($parameter) {
		$this->parameter = $parameter;
	}


}