library\search\indexer\InverseDocumentFrequency

Complexity

Total Complexity

4

Size/Duplication

Total Lines	50
Duplicated Lines	20 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
B	execute()	10	24	3

<?php
/**
 * User: jensk
 * Date: 1-3-2017
 * Time: 11:41
 */

namespace library\search\indexer;


class InverseDocumentFrequency
{
	/**
	 * @var \PDO
	 */
	protected $dbHandle;
	protected $documentCount;

	/**
	 * InverseDocumentFrequency constructor.
	 *
	 * @param resource $dbHandle
	 * @param int      $documentCount
	 */
	public function __construct($dbHandle, $documentCount)
	{
		$this->dbHandle = $dbHandle;

It seems like $dbHandle of type resource is incompatible with the declared type object<PDO> of property $dbHandle.

		$this->documentCount = $documentCount;
	}

	/**
	 * Formula to calculate:
	 * 		idf(t) = 1 + log ( totalDocuments / (documentsThatContainTheTerm + 1))
	 * @throws \Exception
	 */
	public function execute()
	{
		$db = $this->dbHandle;
		$db->sqliteCreateFunction('log', 'log', 1);
		$sql = '
		INSERT INTO inverse_document_frequency (term, inverseDocumentFrequency)
		SELECT DISTINCT term, (1+(log(:documentCount / COUNT(documentPath) + 1))) as inverseDocumentFrequency
					  FROM term_count
				  GROUP BY term
		';

		if (!$stmt = $db->prepare($sql)) {

This code seems to be duplicated across your project.

			$errorInfo = $db->errorInfo();
			$errorMsg = $errorInfo[2];
			throw new \Exception('SQLite Exception: ' . $errorMsg . ' in SQL: <br /><pre>' . $sql . '</pre>');
		}
		$stmt->bindValue(':documentCount', $this->documentCount);
		$result = $stmt->execute();
		if ($result === false) {

This code seems to be duplicated across your project.

			$errorInfo = $db->errorInfo();
			$errorMsg = $errorInfo[2];
			throw new \Exception('SQLite Exception: ' . $errorMsg . ' in SQL: <br /><pre>' . $sql . '</pre>');
		}
	}
}