Mvg\TextOutput\Departures

Complexity

Total Complexity

12

Size/Duplication

Total Lines	101
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	8
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getFilter()	0	3	1
A	setFilter()	0	3	1
A	getDeparturesFactory()	0	3	1
A	setDeparturesFactory()	0	3	1
A	mb_str_pad()	0	5	1
B	getOutput()	0	33	6

<?php
/**
 * User: ms
 * Date: 29.08.15
 * Time: 11:00
 */

namespace Mvg\TextOutput;

/**
 * Class Departures
 * @package Mvg
 */
class Departures {

	/**
	 * @var \Mvg\Factories\Departures
	 */
	protected $departuresFactory = null;

	/**
	 * @var mixed
	 */
	protected $filter = null;

	/**
	 * @param \Mvg\Factories\Departures $departuresFactory
	 * @param mixed $filter
	 */
	public function __construct($departuresFactory, $filter = null) {
		$this->setDeparturesFactory($departuresFactory);
		$this->setFilter($filter);
	}

	/**
	 * @return mixed
	 */
	protected function getFilter() {
		return $this->filter;
	}

	/**
	 * @param mixed $filter
	 */
	protected function setFilter($filter) {
		$this->filter = $filter;
	}


	/**
	 * @return \Mvg\Factories\Departures
	 */
	protected function getDeparturesFactory() {
		return $this->departuresFactory;
	}


	/**
	 * @param \Mvg\Factories\Departures $departuresFactory
	 */
	protected function setDeparturesFactory($departuresFactory) {
		$this->departuresFactory = $departuresFactory;
	}

	/**
	 * found here http://stackoverflow.com/a/11871948
	 * @param $input
	 * @param $pad_length
	 * @param string $pad_string
	 * @param int $pad_type
	 * @return string
	 */
	public static function mb_str_pad($input, $pad_length, $pad_string = ' ', $pad_type = STR_PAD_RIGHT) {

function mb_str_pad() does not seem to conform to the naming convention (^(?:[a-z]|__)[a-zA-Z0-9]*$).

$pad_length does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

The parameter $pad_length is not named in camelCase.

The parameter $pad_string is not named in camelCase.

The parameter $pad_type is not named in camelCase.

This method is not in camel caps format.

		mb_internal_encoding('utf-8'); // @important
		$diff = strlen($input) - mb_strlen($input);
		return str_pad($input, $pad_length + $diff, $pad_string, $pad_type);

$pad_length does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

	}

	public function getOutput() {
		$maxLenLineNumber = 0;

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

		$maxLenDestination = 0;
		$maxLenTime = 0;

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

		$departuresItems = $this->getDeparturesFactory()->getItems($this->getFilter());

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

		foreach ($departuresItems as $departureObject) {

			if (mb_strlen($departureObject->lineNumber) > $maxLenLineNumber) {
				$maxLenLineNumber = mb_strlen($departureObject->lineNumber);
			}

			if (mb_strlen($departureObject->destination) > $maxLenDestination) {
				$maxLenDestination = mb_strlen($departureObject->destination);
			}
			if (mb_strlen($departureObject->time) > $maxLenTime) {
				$maxLenTime = mb_strlen($departureObject->time);
			}
		}
		$str = sprintf('Abfahrtzeiten %s %s',
			$this->getDeparturesFactory()->getStation()
			, $this->getDeparturesFactory()->getCurrentTime()
		);

		$str .= "\n";

		foreach ($departuresItems as $departureObject) {
			$str .= self::mb_str_pad($departureObject->lineNumber, $maxLenLineNumber + 3, ' ', STR_PAD_RIGHT);
			$str .= self::mb_str_pad($departureObject->destination, $maxLenDestination + 3, ' ', STR_PAD_RIGHT);
			$str .= self::mb_str_pad($departureObject->time, $maxLenTime + 3, ' ', STR_PAD_RIGHT);
			$str .= "\n";
		}
		return $str;
	}


}