chillerlan\Database\DBQuery

Complexity

Total Complexity

8

Size/Duplication

Total Lines	73
Duplicated Lines	0 %

Coupling/Cohesion

Components

0

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	hp$0 ➔ database()	0	3	1
A	hp$0 ➔ table()	0	3	1
A	hp$0 ➔ index()	0	3	1
A	hp$0 ➔ view()	0	3	1
A	hp$0 ➔ trigger()	0	3	1
A	__construct()	0	5	1
B	__get()	0	36	3

<?php
/**
 * Class DBQuery
 *
 * @filesource   DBQuery.php
 * @created      03.06.2017
 * @package      chillerlan\Database
 * @author       Smiley <smiley@chillerlan.net>
 * @copyright    2017 Smiley
 * @license      MIT
 */

namespace chillerlan\Database;

use chillerlan\Database\Drivers\DBDriverInterface;
use chillerlan\Database\Query\CreateInterface;
use chillerlan\Database\Query\StatementAbstract;
use chillerlan\Database\Query\CreateDatabaseInterface;
use chillerlan\Database\Query\CreateTableInterface;
/**
 * @property \chillerlan\Database\Query\SelectInterface $select
 * @property \chillerlan\Database\Query\InsertInterface $insert
 * @property \chillerlan\Database\Query\UpdateInterface $update
 * @property \chillerlan\Database\Query\DeleteInterface $delete
 * @property \chillerlan\Database\Query\CreateInterface $create
 */
class DBQuery{

	const DIALECTS = [
		'mysql'    => 'MySQL',
		'pgsql'    => 'Postgres',
		'sqlite'   => 'SQLite',
		'firebird' => 'Firebird',
	];

	/**
	 * @var \chillerlan\Database\Drivers\DBDriverInterface
	 */
	protected $DBDriver;

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

	/**
	 * DBQuery constructor.
	 *
	 * @param \chillerlan\Database\Drivers\DBDriverInterface $DBDriver
	 */
	public function __construct(DBDriverInterface $DBDriver){
		$this->DBDriver = $DBDriver;

		$this->dialect = __NAMESPACE__.'\\Query\\Dialects\\'.self::DIALECTS[$this->DBDriver->dialect];

Accessing dialect on the interface chillerlan\Database\Drivers\DBDriverInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

	}

	/**
	 * @param string $name
	 *
	 * @return \chillerlan\Database\Query\StatementInterface|null

Should the return type not be object|null?

	 */
	public function __get(string $name){
		$name = strtolower($name);

		if(in_array($name, ['select', 'insert', 'update', 'delete'])){
			$class = $this->dialect.'\\'.ucfirst($name);
			return new $class($this->DBDriver, $this->dialect);
		}
		elseif($name === 'create'){

			return new class($this->DBDriver, $this->dialect) extends StatementAbstract implements CreateInterface{

				public function database(string $dbname = null):CreateDatabaseInterface{
					return $this->getStatementClass('CreateDatabase')->name($dbname);
				}

				public function table(string $tablename = null):CreateTableInterface{
					return $this->getStatementClass('CreateTable')->name($tablename);
				}

				public function index():CreateInterface{
					// TODO: Implement index() method.
				}

				public function view():CreateInterface{
					// TODO: Implement view() method.
				}

				public function trigger():CreateInterface{
					// TODO: Implement trigger() method.
				}
			};

		}

		return null;
	}

}