Форум PHP

вроде бы работает вот такой код, дальше завяз

<?php class db {          protected static $_instance = NULL;          private function __construct() {}         private function __clone()     {}     private function __wakeup()    {}          public static function getInstance() {         if (self::$_instance === NULL) {             $config = db_config::getInstance();             try {                 self::$_instance = new PDO ("mysql:host=$config->db_host;dbname=$config->db_base",                                                           $config->db_user,                                                           $config->db_password,                                                           array(PDO::ATTR_PERSISTENT => true,                                                                PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES $config->encoding"));                 self::$_instance->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);             } catch(PDOException $e) {                   echo "Ой!";                   file_put_contents(__DIR__."/log/db_error.log", $e->getMessage()."\r\n", FILE_APPEND | LOCK_EX);                 }             return self::$_instance;         }     }  }

но если в нем сделать еще какие-нибудь методы, то выскакивает ошибка, типа
Call to undefined method PDO::db_select() in

как лучше и где реализовать обертку для обработки запросов?

всё верно, потому что вы добавляете метод db_select классу db, а не PDO. и вообще, сиглетон тут неуместен. что если захотите работать например с двумя базами?

спасибо за подсказку, поставлю на заметку в будущем.
кстати нашел решение своего вопроса

не за что. просто имейте ввиду, что этот паттерн достаточно узкоспециализированный. то есть нужно сначала очень хорошо подумать насколько оправдан такой подход, и применять его только если нет совсем никаких сомнений, что будет не более одного экземпляра.

да, можно поразмышлять на тему использования Singleton в PDO

>сиглетон тут неуместен. что если захотите работать например с двумя базами?
что ж так сразу то? две и более БД далеко не самый распространенный случай

естественно, но периодически такое случается. класс такого рода должен быть максимально универсальным

в таком случае фабрику правильнее использовать?

да

DIC правильнее использовать.

DIC можно поподробнее?

>catch(PDOException $e) {
> echo "Ой!";
забавно)

я не знаю как на счет фабрики пока не применял этот патерн но скажу точно что нужен отдельный класс для соединения с базой ,а ваш так скажем simple должен только принимать входящий ресурс соединения и еще очень важно пака не будет готова ваша обвертка не используйте ее иначе придется исправлять все ссылки в коде а так я только за такие решения ведь если что то устареет вы только правите вашу обвертку но опять же повторюсь очень важно продумать функционал прежде чем его использовать например если у вас есть метод getAllObj то он должен так и называться не зависимо от модификаций ну естественно и роль его тоже не должна меняться

спасибо за мнение, с ссылками проблем нет, если грамотно прописать пути, то проблем нет где лежит файл и даже можно не париться об этом