DatabaseResult::getNumRecords() - Code Metrics - Inspection of "Merge pull request #1094 from hemberger/database-r..." - smrealms/smr - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Push — master ( f174b5...646f17 )

by Dan

created 2021-07-01 07:44 UTC

DatabaseResult::getNumRecords() A

↳ Parent: DatabaseResult

Complexity

Conditions	1
Paths	1

Size

Total Lines	2
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
eloc	1
c	1
b	0
f	0
nc	1
nop	0
dl	0
loc	2
rs	10

<?php declare(strict_types=1);

namespace Smr;

/**
 * Holds the result of a Database query (e.g. read or write).
 */
class DatabaseResult {

	public function __construct(
		private \mysqli_result $dbResult
	) {}

	/**
	 * Use to iterate over the records from the result set.
	 * @return \Generator<DatabaseRecord>
	 */
	public function records() : \Generator {
		foreach ($this->dbResult as $dbRecord) {
			yield new DatabaseRecord($dbRecord);
		}
	}

	/**
	 * Use when exactly one record is expected from the result set.
	 */
	public function record() : DatabaseRecord {
		if ($this->getNumRecords() != 1) {
			throw new \RuntimeException('One record required, but found ' . $this->getNumRecords());
		}
		return new DatabaseRecord($this->dbResult->fetch_assoc());
	}

	public function getNumRecords() : int {
		return $this->dbResult->num_rows;
	}

	public function hasRecord() : bool {
		return $this->getNumRecords() > 0;
	}

}


1			<?php declare(strict_types=1);
2
3			namespace Smr;
4
5			/**
6			* Holds the result of a Database query (e.g. read or write).
7			*/
8			class DatabaseResult {
9
10			public function __construct(
11			private \mysqli_result $dbResult
12			) {}
13
14			/**
15			* Use to iterate over the records from the result set.
16			* @return \Generator<DatabaseRecord>
17			*/
18			public function records() : \Generator {
19			foreach ($this->dbResult as $dbRecord) {
20			yield new DatabaseRecord($dbRecord);
21			}
22			}
23
24			/**
25			* Use when exactly one record is expected from the result set.
26			*/
27			public function record() : DatabaseRecord {
28			if ($this->getNumRecords() != 1) {
29			throw new \RuntimeException('One record required, but found ' . $this->getNumRecords());
30			}
31			return new DatabaseRecord($this->dbResult->fetch_assoc());
32			}
33
34			public function getNumRecords() : int {
35			return $this->dbResult->num_rows;
36			}
37
38			public function hasRecord() : bool {
39			return $this->getNumRecords() > 0;
40			}
41
42			}
43

smrealms / smr

Scrutinizer GitHub App not installed

Push — master ( f174b5...646f17 )

DatabaseResult::getNumRecords() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like