Fhp\Model\StatementOfAccount\Statement

Complexity

Total Complexity

11

Size/Duplication

Total Lines	135
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	0

Test Coverage

Coverage

79.31%

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	getTransactions()	0	4	1
A	setTransactions()	0	6	1
A	addTransaction()	0	4	1
A	getStartBalance()	0	4	1
A	getSignedStartBalance()	0	8	2
A	setStartBalance()	0	6	1
A	getCreditDebit()	0	4	1
A	setCreditDebit()	0	6	1
A	getDate()	0	4	1
A	setDate()	0	6	1

<?php

namespace Fhp\Model\StatementOfAccount;

/**
 * Class Statement
 * @package Fhp\Model\StatementOfAccount
 */
class Statement
{
	const CD_CREDIT = 'credit';
	const CD_DEBIT = 'debit';

	/**
	 * @var array of Transaction
	 */
	protected $transactions = array();

	/**
	 * @var float
	 */
	protected $startBalance = 0.0;

	/**
	 * @var string
	 */
	protected $creditDebit;

	/**
	 * @var \DateTime|null
	 */
	protected $date;

	/**
	 * Get transactions
	 *
	 * @return Transaction[]
	 */
	public function getTransactions()
	{
		return $this->transactions;
	}

	/**
	 * Set transactions
	 *
	 * @param array $transactions
	 *
	 * @return $this
	 */
	public function setTransactions(array $transactions = null)
	{
		$this->transactions = $transactions;

It seems like $transactions can be null. However, the property $transactions is declared as array. Maybe change the type of the property to array|null or add a type check?

		return $this;
	}

	public function addTransaction(Transaction $transaction)
	{
		$this->transactions[] = $transaction;
	}

	/**
	 * Get startBalance
	 *
	 * @return float
	 */
	public function getStartBalance()
	{
		return $this->startBalance;
	}

	public function getSignedStartBalance()
	{
		$s = 1;
		if ($this->getCreditDebit() == self::CD_DEBIT) {
			$s = -1;
		}
		return $s * $this->getStartBalance();
	}

	/**
	 * Set startBalance
	 *
	 * @param float $startBalance
	 *
	 * @return $this
	 */
	public function setStartBalance($startBalance)
	{
		$this->startBalance = (float) $startBalance;

		return $this;
	}

	/**
	 * Get creditDebit
	 *
	 * @return string
	 */
	public function getCreditDebit()
	{
		return $this->creditDebit;
	}

	/**
	 * Set creditDebit
	 *
	 * @param string|null $creditDebit
	 *
	 * @return $this
	 */
	public function setCreditDebit($creditDebit)
	{
		$this->creditDebit = $creditDebit;

		return $this;
	}

	/**
	 * Get date
	 *
	 * @return \DateTime
	 */
	public function getDate()
	{
		return $this->date;
	}

	/**
	 * Set date
	 *
	 * @param \DateTime $date
	 *
	 * @return $this
	 */
	public function setDate(\DateTime $date)
	{
		$this->date = $date;

		return $this;
	}
}