Любые предложения по улучшению моего класса подключения PDO?

Question

Эй, ребята, я довольно новичок в pdo, поэтому я просто собрал простой класс соединений, используя информацию из вводной книги, которую я читал, но эффективно ли это соединение?Если у кого-то есть какие-либо информативные предложения, я был бы очень признателен.

class PDOConnectionFactory{

    public $con = null;
    // swich database?
    public $dbType  = "mysql";

    // connection parameters

    public $host    = "localhost";
    public $user    = "user";
    public $senha   = "password";
    public $db  = "database";


    public $persistent = false;

    // new PDOConnectionFactory( true ) <--- persistent connection
    // new PDOConnectionFactory()       <--- no persistent connection
    public function PDOConnectionFactory( $persistent=false ){
        // it verifies the persistence of the connection
        if( $persistent != false){ $this->persistent = true; }
    }

    public function getConnection(){
            try{
                $this->con = new PDO($this->dbType.":host=".$this->host.";dbname=".$this->db, $this->user, $this->senha, 
                array( PDO::ATTR_PERSISTENT => $this->persistent ) );
                // carried through successfully, it returns connected
                return $this->con;
            // in case that an error occurs, it returns the error;
            }catch ( PDOException $ex ){  echo "We are currently experiencing technical difficulties.  ".$ex->getMessage(); }

    }

    // close connection
    public function Close(){
        if( $this->con != null )
            $this->con = null;
    }

}

Answer 1

При реализации «Фабрики» обычно так, что другие классы, методы и т. Д., Использующие ее, не должны знать или заботиться о соединениях, именах пользователей, паролях и т. Д.

Я бы сделал что-то более похожее на:

static class PDOConnectionFactory {
    // database
    private $dbType = "mysql";

    // connection parameters
    private $host = "localhost";
    private $user = "user";
    private $senha = "password";
    private $db = "database";

    // new CreateNewConnection( true ) <--- persistent connection
    // new CreateNewConnection()       <--- no persistent connection
    public function CreateNewConnection($persistent = false) {
        try {
            $con = new PDO($dbType . ":host=" . $host . ";dbname=" . $db, $user, $senha, array(PDO::ATTR_PERSISTENT => $persistent));
            // carried through successfully, it returns connected
            return $con;
        }
        catch (PDOException $ex) {
            // in case that an error occurs, it returns the error;
            echo "We are currently experiencing technical difficulties. We have a bunch of monkies working really hard to fix the problem. Check back soon: " . $ex->getMessage();
        }
    }
}

Затем вы используете соединение, возвращаемое CreateNewConnection (), любым необходимым способом.

Я не проверял, компилируется ли вышеупомянутый код, может быть несколько опечаток / проблем, но вы поняли идею. Теперь вам нужно сделать еще один шаг и реализовать что-то вроде шаблона хранилища:)

Answer 2

Мое предложение состоит в том, чтобы вы реализовали синглтон, чтобы ограничить создание экземпляра PDO одним объектом. Это может выглядеть так:

class Database {

    protected static $_instance;
    protected $_connection;
    protected $_dns = 'mysql:host=localhost;dbname=mydbname';
    protected $_username = 'myusername';
    protected $_password = 'mypassword';

    /**
    * Singleton pattern implementation makes "new" unavailable
    */
    protected function __construct()
    {
        $this->_connection = 
            new PDO($this->_dns, $this->_username, $this->_password);
    }

    public function getConnection()
    {
        return $this->_connection;
    }

    public static function getInstance()
    {
        if (null === self::$_instance) {
            self::$_instance = new self();
        }
        return self::$_instance;
    }

    /**
    * Singleton pattern implementation makes "clone" unavailable
    */
    protected function __clone()
    {}
}


$dbc = Database::getInstance()->getConnection();

Answer 3

Эта статья может оказаться полезной, написана Эриком Вурцером и опубликована в Nettuts, одном из моих любимых веб-сайтов (помимо этого). Почему вы должны использовать PHP PDO для доступа к базе данных

Надеюсь, это поможет.