mysqlims

Complexity

Total Complexity

20

Size/Duplication

Total Lines	79
Duplicated Lines	35.44 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
C	__construct()	12	32	12
A	prepare()	8	8	4
A	query()	8	8	4

<?php
/*
 * This class will run queries on master/slave servers depending on the query itself.
 */
class mysqlims extends mysqli

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    private $mysqliW;
    private $mysqliR = null;

    /*
     * Pass main and slave connection arrays to the constructor, and strict as true/false
     *
     * @param array $main
     * @param array $slave
     * @param boolean $strict
     *
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct($main, $slave = false, $strict = false)
    {
        if ($strict) {
            $this->mysqliW = new mysqli_strict($main['host'],

The call to mysqli_strict::__construct() has too many arguments starting with $main['host'].

                $main['user'], $main['pass'],
                $main['name'], $main['port']);
            if ($slave && is_array($slave) && isset($slave['enabled']) && $slave['enabled']

This code seems to be duplicated across your project.

                === true) {
                $this->mysqliR = new mysqli_strict($slave['host'],

The call to mysqli_strict::__construct() has too many arguments starting with $slave['host'].

                    $slave['user'], $slave['pass'],
                    $slave['name'], $slave['port']);
            }
        } else {
            $this->mysqliW = new mysqli($main['host'],
                $main['user'], $main['pass'],
                $main['name'], $main['port']);
            if ($slave && is_array($slave) && isset($slave['enabled']) && $slave['enabled']

This code seems to be duplicated across your project.

                === true) {
                $this->mysqliR = new mysqli($slave['host'],
                    $slave['user'], $slave['pass'],
                    $slave['name'], $slave['port']);
            }
        }

        if ($this->mysqliW->connect_errno) {
            throw new Exception("Failed to connect to MySQL: (".$this->mysqliW->connect_errno.") ".$this->mysqliW->connect_error);
        }

        if ($this->mysqliR->connect_errno) {
            throw new Exception("Failed to connect to MySQL: (".$this->mysqliR->connect_errno.") ".$this->mysqliR->connect_error);
        }
    }

    /*
     * Override standard mysqli_prepare to select master/slave server
     * @param $string query
     *
     * @return mysqli_stmt
     */
    public function prepare($query)

This method seems to be duplicated in your project.

    {
        if (stripos($query, "SELECT") && stripos($query, "FOR UPDATE") === false && $this->mysqliR !== null) {
            return $this->mysqliR->prepare($query);
        } else {
            return $this->mysqliW->prepare($query);
        }
    }

    /*
     * Override standard mysqli_query to select master/slave server
     * @param string $query
     * @param int $resultmode
     *
     * @return boolean
     * @return mixed
     */
    public function query($query, $resultmode = MYSQLI_STORE_RESULT)

This method seems to be duplicated in your project.

    {
        if (stripos($query, "SELECT") && stripos($query, "FOR UPDATE") === false && $this->mysqliR !== null) {/* Use readonly server */
            return $this->mysqliR->query($query, $resultmode);
        } else {
            return $this->mysqliW->query($query, $resultmode);
        }
    }
}