Повторное использование объекта подключения базы данных

Question

Я использую эту функцию для подключения к моему MySQL db, когда это необходимо, а также для повторного использования одного и того же объекта соединения для любого последующего запроса, который может понадобиться в том же скрипте 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) 

Согласны ли вы на такой подход? Как вы думаете, его можно оптимизировать? Спасибо за ваше мнение.

Answer 1

Я согласен с этим методом, хотя многие люди скажут вам, что этот подход «singleton» плох, плох.

Однако я не согласен с вашей реализацией. Оно должно быть:

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"); 

Кроме того, я бы продлить (и конечно без вторя ошибку безоговорочно!) он должен принять параметр, чтобы сделать несколько соединений возможными.