PHP - слой абстракции DB использует статический класс vs singleton object?

Ответ 1

Что не так со следующим (упрощенным) примером:

class Database
{
    protected $_connection;

    protected $_config;

    public function __construct( array $config ) // or other means of passing config vars
    {
        $this->_config = $config;
    }

    public function query( $query )
    {
        // use lazy loading getter
        return $this->_getConnection()->query( $query );
    }

    protected function _getConnection()
    {
        // lazy load connection
        if( $this->_connection === null )
        {
            $dsn = /* create valid dsn string from $this->_config */;

            try
            {
                $this->_connection = new PDO( $dsn, $this->_config[ 'username' ], $this->_config[ 'password' ] );
            }
            catch( PDOException $e )
            {
                /* handle failed connecting */
            }
        }

        return $this->_connection;
    }
}

$db1 = new Database( array(
    'driver'   => 'mysql',
    'host'     => 'localhost',
    'dbname'   => 'test',
    'username' => 'test_root',
    'password' => '**********'
) );

$db2 = new Database( array(
    'driver'   => 'pgsql',
    'host'     => '213.222.1.43',
    'dbname'   => 'otherdb',
    'username' => 'otherdb_root',
    'password' => '**********'
) );

$someModel       = new SomeModel( $db1 );
$someOtherModel  = new SomeOtherModel( $db2 );
$yetAnotherModel = new YetAnotherModel( $db2 );

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

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

Ответ 2

В моем последнем проекте я фактически пошел против "хороших" принципов проектирования, сделав класс базы данных полностью статичным. Причина этого в том, что я использовал много кеширования на объектах PHP. Первоначально у меня была база данных, переданная через конструктор каждого объекта в качестве инъекции зависимости, однако я хотел убедиться, что база данных не должна соединяться, если это абсолютно необходимо. Таким образом, использование базы данных в качестве переменной-члена этого объекта было бы нецелесообразным, потому что если вы не инициализировали объект из кеша, вы не захотите подключаться к базе данных, если вы на самом деле не выполнили операцию над ним.

Таким образом, в итоге у меня было только две (общедоступные) статические функции, Database:: fetch() и Database:: execute(), которые проверяли бы, было ли это уже подключено, а если нет, то он будет подключаться и выполнять запрос. Таким образом, мне не пришлось бы беспокоиться о десериализации и подключаться как можно реже. Это технически делает модульное тестирование невозможным, хотя.

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

Ответ 3

Мой совет: STOP, используя Singleton и статические все вместе.

Почему? Потому что вы будете вставлять зависимости, которые сделают ваш код непригодным для использования в других проектах, и не позволят unit test его. Также забудьте о свободном соединении, если используете синглтон.

Альтернативы? Внедрение зависимости. http://www.potstuck.com/2009/01/08/php-dependency-injection

Ответ 4

Создание библиотеки библиотек БД, конечно, короче и быстрее, чем выполнение:

$db = DBSingleton::blabla(); // everytime I need ya

Но также, поскольку он глобальный, соблазн использовать всюду.

Итак, выберите другие методы, если вы хотите очистить код... и выберите статический, если нужен быстрый код;)

Ответ 5

/* Data base*/
 class Database
{
    /* Database field definition */
    private static $_instance; /instance
    private $_connection;
    private $DB_USER = "database_user_name_here";
    private $DB_PASS = "your_password_here";
    private $DB_NAME = "your_database_name_here";
    private $DB_SERVER = "localhost";

    /* Initiate the database connection */
    private function __construct()
    {
        $this->_connection = new mysqli($this->DB_SERVER ,
                                        $this->DB_USER ,
                                        $this->DB_PASS ,
                                        $this->DB_NAME);
        /* Test if connection succeeded */
        if (mysqli_connect_errno()) {
            die("Database connection failed: " .
                mysqli_connect_error() .
                " (" . mysqli_connect_errno() . ")"
            );
        }
    }

    /**
     * Instance of the database
     * @return Database
     *
     */
    public static function Instance()
    {
        if (!self::$_instance) { // If no instance then make one
            self::$_instance = new self();
        }

        return self::$_instance;
    }

    /**
     * Void duplicate connection
     */
    private function __clone() { }

    /* Return a connection */
    public function getConnection()
    {
        return $this->_connection;
    }

}

/** This is how you would use it in a different class.
  @var TYPE_NAME $connection */
$db = Database::Instance();
$connection = $db->getConnection();