FastD\Model\Database

Complexity

Total Complexity

9

Size/Duplication

Total Lines	70
Duplicated Lines	28.57 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	reconnect()	0	4	1
A	checkGoneAway()	0	7	3
A	query()	10	10	2
A	exec()	10	10	2

<?php
/**
 * @author    jan huang <bboyjanhuang@gmail.com>
 * @copyright 2016
 *
 * @see      https://www.github.com/janhuang
 * @see      http://www.fast-d.cn/
 */

namespace FastD\Model;

use Medoo\Medoo;

/**
 * Class Database.
 */
class Database extends Medoo
{
    /**
     * @var array|null
     */
    protected $config = [];

    /**
     * Database constructor.
     *
     * @param null $options
     */
    public function __construct($options = null)
    {
        $this->config = $options;

        parent::__construct($this->config);
    }

    /**
     * reconnect database.
     */
    public function reconnect()
    {
        $this->__construct($this->config);

It seems like $this->config can also be of type array; however, FastD\Model\Database::__construct() does only seem to accept null, maybe add an additional type check?

    }

    /**
     * check database gone away.
     */
    public function checkGoneAway()
    {
        list(, $code) = $this->pdo->errorInfo();

The property pdo does not exist. Did you maybe forget to declare it?

        if (2006 === $code || 2013 === $code) {
            $this->reconnect();
        }
    }

    /**
     * @param $query
     *
     * @return bool|\PDOStatement
     */
    public function query($query)

This method seems to be duplicated in your project.

    {
        $result = parent::query($query);
        if (false === $result) {
            $this->checkGoneAway();
            $result = parent::query($query);
        }

        return $result;
    }

    /**
     * @param $query
     *
     * @return bool|int
     */
    public function exec($query)

This method seems to be duplicated in your project.

    {
        $result = parent::exec($query);
        if (false === $result) {
            $this->checkGoneAway();
            $result = parent::exec($query);
        }

        return $result;
    }
}