4

我正在使用此函数在需要时连接到我的 MySQL 数据库，并在同一个 php 脚本中为我可能需要的任何进一步查询重新使用相同的连接对象。

function cnn() {
    static $pdo;
    if(!isset($pdo)) {
        try {
            $pdo = new PDO('mysql:host='.DB_HOST.';dbname='.DB_NAME, DB_USER, DB_PASS);
            $pdo->setAttribute(PDO::ATTR_TIMEOUT, 30);
            $pdo->setAttribute(PDO::ATTR_PERSISTENT, true);
            $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
            $pdo->setAttribute(PDO::ATTR_DEFAULT_FETCH_MODE, PDO::FETCH_OBJ);
            return $pdo;
        } catch(PDOException $e) {
            http_response_code(503);
            echo $e->getCode.': '.$e->getMessage();
            die(); //or whatever error handler you use
        }
    } else {
        return $pdo;
    }
}

第一个查询（创建对象）

echo cnn()->query('SELECT firstname FROM user WHERE id=4;')->fetch(PDO::FETCH_COLUMN)

第二次查询（对象被重用）

echo cnn()->query('SELECT title FROM news WHERE id=516;')->fetch(PDO::FETCH_COLUMN)

你同意这种方法吗？你觉得可以优化吗？感谢您的意见。

4

1 回答 1

3

我同意这种方法，尽管很多人会告诉你这种“单例”方法很糟糕，很糟糕。

但是，我不同意您的实施。它应该是：

function cnn() {
    static $pdo;
    if(!$pdo) {
        $conf = array(PDO::ATTR_TIMEOUT => 30,
                  PDO::ATTR_PERSISTENT => true,
                  PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
                  PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_OBJ,
            );
        $dsn = 'mysql:host='.DB_HOST.';dbname='.DB_NAME;
        $pdo = new PDO($dsn, DB_USER, DB_PASS, $conf);
    } 
    return $pdo;
}

此外，将处理程序代码移动到处理程序中看起来是明智的（当然没有无条件地回显错误！）

function my_exceptionHandler($exception) {
    http_response_code(503);
    if (ini_get('display_errors')) {
        echo $e->getMessage().$e->getTrace();
    } else {
        log_error($e->getMessage().$e->getTrace());
    }
    die(); //or whatever error handler you use
}
set_exception_handler("my_exceptionHandler");

此外，我会将其扩展为接受一个参数以使多个连接成为可能。

于 2013-06-03T04:01:15.427 回答