Fhp\Parser\MT940

Complexity

Total Complexity

40

Size/Duplication

Total Lines	234
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

85.11%

Importance

Changes	11
Bugs	4	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	parse()	0	10	2
D	parseToArray()	0	102	18
C	parseDescription()	0	71	18
A	getDate()	0	6	1

<?php

namespace Fhp\Parser;

use Fhp\Parser\Exception\MT940Exception;

/**
 * Class MT940
 * @package Fhp\Parser
 */
class MT940
{
	const TARGET_ARRAY = 0;

	const CD_CREDIT = 'credit';
	const CD_DEBIT = 'debit';
	const CD_CREDIT_CANCELLATION = 'credit_cancellation';
	const CD_DEBIT_CANCELLATION = 'debit_cancellation';

	/** @var string */
	protected $rawData;
	/** @var string */
	protected $soaDate;

	/**
	 * MT940 constructor.
	 *
	 * @param string $rawData
	 */
	public function __construct($rawData)
	{
		$this->rawData = (string) $rawData;
	}

	/**
	 * @param string $target
	 * @return array
	 * @throws MT940Exception
	 */
	public function parse($target)
	{
		switch ($target) {
		case static::TARGET_ARRAY:
			return $this->parseToArray();
			break;

break is not strictly necessary here and could be removed.

		default:
			throw new MT940Exception('Invalid parse type provided');
		}
	}

	/**
	 * @return array
	 * @throws MT940Exception
	 */
	protected function parseToArray()
	{
		// The divider can be either \r\n or @@

		$divider = "(@@|\r\n)";
		$result = array();
		$days = preg_split('/' . $divider . '-' . $divider . '/', $this->rawData);

		foreach ($days as $day) {
			$day = preg_split('/' . $divider . ':/', $day);
			$statement = array();
			$transactions = array();
			$cnt = 0;

$cnt is not used, you could remove the assignment.

			for ($i = 0, $cnt = count($day); $i < $cnt; $i++) {
				// handle start balance
				// 60F:C160401EUR1234,56

50% of this comment could be valid code. Did you maybe forget this after debugging?

				if (preg_match('/^60(F|M):/', $day[$i])) {
					// remove 60(F|M): for better parsing

38% of this comment could be valid code. Did you maybe forget this after debugging?

					$day[$i] = substr($day[$i], 4);
					$this->soaDate = $this->getDate(substr($day[$i], 1, 6));

					$amount = str_replace(',', '.', substr($day[$i], 10));
					$statement['start_balance'] = array(
						'amount' => $amount,
						'credit_debit' => (substr($day[$i], 0, 1) == 'C') ? static::CD_CREDIT : static::CD_DEBIT
					);
					$statement['date'] = $this->soaDate;
				} elseif (
					// found transaction
					// trx:61:1603310331DR637,39N033NONREF

56% of this comment could be valid code. Did you maybe forget this after debugging?

					0 === strpos($day[$i], '61:')
					&& isset($day[$i + 1])
					&& 0 === strpos($day[$i + 1], '86:')
				) {
					$transaction = substr($day[$i], 3);
					$description = substr($day[$i + 1], 3);

					$currentTrx = array();

					preg_match('/^\d{6}(\d{4})?(C|D|RC|RD)[A-Z]?([^N]+)N/', $transaction, $matches);

					switch ($matches[2]) {
					case 'C':
						$currentTrx['credit_debit'] = static::CD_CREDIT;
						break;
					case 'D':
						$currentTrx['credit_debit'] = static::CD_DEBIT;
						break;
					case 'RC':
						$currentTrx['credit_debit'] = static::CD_CREDIT_CANCELLATION;
						break;
					case 'RD':
						$currentTrx['credit_debit'] = static::CD_DEBIT_CANCELLATION;
						break;
					default:
						throw new MT940Exception('c/d/rc/rd mark not found in: ' . $transaction);
					}

					$amount = $matches[3];
					$amount = str_replace(',', '.', $amount);
					$currentTrx['amount'] = floatval($amount);

					$currentTrx['transaction_code'] = substr($description, 0, 3);

					$description = $this->parseDescription($description);
					$currentTrx['description'] = $description;

					// :61:1605110509D198,02NMSCNONREF

67% of this comment could be valid code. Did you maybe forget this after debugging?

					// 16 = year
					// 0511 = valuta date
					// 0509 = booking date
					$year = substr($transaction, 0, 2);
					$valutaDate = $this->getDate($year . substr($transaction, 2, 4));

					$bookingDate = substr($transaction, 6, 4);
					if (preg_match('/^\d{4}$/', $bookingDate)) {
						// if valuta date is earlier than booking date, then it must be in the new year.
						if (substr($transaction, 2, 2) == '12' && substr($transaction, 6, 2) == '01') {
							$year++;
						} elseif (substr($transaction, 2, 2) == '01' && substr($transaction, 6, 2) == '12') {
							$year--;
						}
						$bookingDate = $this->getDate($year . $bookingDate);
					} else {
						// if booking date not set in :61, then we have to take it from :60F
						$bookingDate = $this->soaDate;
					}

					$currentTrx['booking_date'] = $bookingDate;
					$currentTrx['valuta_date'] = $valutaDate;

					$transactions[] = $currentTrx;
				}
			}
			$statement['transactions'] = $transactions;
			if (count($transactions) > 0) {
				$result[] = $statement;
			}
		}

		return $result;
	}

	/**
	 * @param string $descr
	 * @return array
	 */
	protected function parseDescription($descr)
	{
		$prepared = array();
		$result = array();

		// prefill with empty values
		for ($i = 0; $i <= 63; $i++) {
			$prepared[$i] = null;
		}

		$descr = str_replace("\r\n", '', $descr);
		$descr = str_replace('? ', '?', $descr);
		preg_match_all('/\?[\r\n]*(\d{2})([^\?]+)/', $descr, $matches, PREG_SET_ORDER);

		$descriptionLines = array();
		$description1 = ''; // Legacy, could be removed.
		$description2 = ''; // Legacy, could be removed.
		foreach ($matches as $m) {
			$index = (int) $m[1];
			if ((20 <= $index && $index <= 29) || (60 <= $index && $index <= 63)) {
				if (20 <= $index && $index <= 29) {
					$description1 .= $m[2];
				} else {
					$description2 .= $m[2];
				}
				$m[2] = trim($m[2]);
				if (!empty($m[2])) {
					$descriptionLines[] = $m[2];
				}
			} else {
				$prepared[$index] = $m[2];
			}
		}

		$description = array();
		if (empty($descriptionLines) || strlen($descriptionLines[0]) < 5 || $descriptionLines[0][4] !== '+') {
			$description['SVWZ'] = implode('', $descriptionLines);
		} else {
			$lastType = null;
			foreach ($descriptionLines as $line) {
				if (strlen($line) > 5 && $line[4] === '+') {
					if ($lastType != null) {

It seems like you are loosely comparing $lastType of type string|null against null; this is ambiguous if the string can be empty. Consider using a strict comparison !== instead.

						$description[$lastType] = trim($description[$lastType]);
					}
					$lastType = substr($line, 0, 4);
					$description[$lastType] = substr($line, 5);
				} else {
					$description[$lastType] .= $line;
				}
				if (strlen($line) < 27) {
					// Usually, lines are 27 characters long. In case characters are missing, then it's either the end
					// of the current type or spaces have been trimmed from the end. We want to collapse multiple spaces
					// into one and we don't want to leave trailing spaces behind. So add a single space here to make up
					// for possibly missing spaces, and if it's the end of the type, it will be trimmed off later.
					$description[$lastType] .= ' ';
				}
			}
			$description[$lastType] = trim($description[$lastType]);
		}

		$result['description']       = $description;
		$result['booking_text']      = trim($prepared[0]);
		$result['primanoten_nr']     = trim($prepared[10]);
		$result['description_1']     = trim($description1);
		$result['bank_code']         = trim($prepared[30]);
		$result['account_number']    = trim($prepared[31]);
		$result['name']              = trim($prepared[32] . $prepared[33]);
		$result['text_key_addition'] = trim($prepared[34]);
		$result['description_2']     = trim($description2);
		return $result;
	}

	/**
	 * @param string $val
	 * @return string
	 */
	protected function getDate($val)
	{
		$val = '20' . $val;
		preg_match('/(\d{4})(\d{2})(\d{2})/', $val, $m);
		return $m[1] . '-' . $m[2] . '-' . $m[3];
	}
}