ХОЖДЕНИЕ ПО БАЗАМ. 2 753 просмотра

Статья в помощь таким же нубам, как и не я. Поэтому при наездах попрошу сильно не умничать.
Решил я для себя набросать компонент для музыкальной странички. Но вот что то шибко много получилось запросов к БД. И тогда я решил сначала их поделить, но затем подумав пришёл к выводу, что будет лучше вообще посадить компонент на другую базу. Что бы «не мешался под ногами». А так как программер из меня не ахти, пришлось затарится травой и сесть курить манны.

Примерно к окончанию вторых суток я понял, что «лучше б маленьким я сдох», ибо настала пора выбрать какое использовать расширение для моего подключения. И здесь пришлось принимать во внимание не только качество этих самых расширений, но и уровень моей подготовки. Поэтому как бы мне не хотелось попробовать этот класс, но у него все пояснения только на чешском. Был просмотрен ещё зтот, но что то я не проникся. В конце концов встал выбор Mysqli или PDO. Последний критиковали поменьше, поэтому выбор пал на него. Да и хотелось попробовать что то новое, с одной стороны, а с другой, что бы присутствовала совместимость. Мануал на русском здесь. Статья на хабре здесь. В двух словах из контекста.
Расширение Объекты данных PHP (PDO) определяет простой и согласованный интерфейс для доступа к базам данных в PHP.
PDO обеспечивает абстракцию (доступа к данным). Это значит, что вне зависимости от того, какая конкретная база данных используется, вы можете пользоваться одними и теми функциями для выполнения запросов и выборки данных.
Сказано-сделано. Ещё пару дней курения и хождения по простором страны Google и оформился классдля подключения базы к нашей, горячо любимой системе. Как говорится в какой то рекламе, «теперь стало ещё легче», или как то так. Короче я сделал так.
Поставил на hosts Ubuntu Server 14.04. и такой же на гостевую на Vbox.
Залил на них мой сайт.
Для начала я решил попробовать перенести скрипт для работы с файлами(загрузка, проверки, редактирование и т. п.) и сами файлы на гостевой сервер. Так как на гостевой мне нужна была только страница загрузки файлов моего компонента, поэтому все лишние модули, компоненты и таблицы я удалил.
Закинул класс в соответственное для него место на хосте/core/classes/.
Способов для подключения второй базы есть несколько. Я выбрал следующий. В/includes/ поместил файл конфигурации conf.ini с настройками для подключения.

db_driver=mysql 
db_user=nik 
db_password=123456  
[dsn] 
host=music2 
port=3306 
dbname=music2 
[db_options] 
PDO::MYSQL_ATTR_INIT_COMMAND=set names utf8 
[db_attributes] 
ATTR_ERRMODE=ERRMODE_EXCEPTION

Здесь, думаю всё понятно.
Код для подключения такой:

 
$DBH = new PDO("mssql:host=$host;dbname=$dbname", $user, $pass); // красавец!

В методе класса он превращается в такой:

 
public static function getLink ( ) { 
$ini = PATH.'/includes/config.ini' ; 
$parse = parse_ini_file ( $ini , true ) ; 
$driver = $parse [ "db_driver" ] ; 
$dsn = "${driver}:" ; 
$user = $parse [ "db_user" ] ; 
$password = $parse [ "db_password" ] ; 
$options = $parse [ "db_options" ] ; 
$attributes = $parse [ "db_attributes" ] ; 
foreach ( $parse [ "dsn" ] as $k => $v ) { 
$dsn .= "${k}=${v};" ; 
} 
[b]$db_link = new PDO ( $dsn, $user, $password, $options ) ; [/b]
$db_link->exec('SET NAMES utf8'); 
foreach ( $attributes as $k => $v ) { 
$db_link -> setAttribute ( constant ( "PDO::{$k}" ) 
, constant ( "PDO::{$v}" ) ) ; 
} 
return $db_link ; 
}

Далее, все подготовительные запросы, я решил тоже помещать в этот класс, так как пока тестируемый компонент только один. А так, удобней им наверное будет в model.php. Разница в синтаксисе не большая. Для примера:
Mysqli

public function get_table($table, $where='', $fields='*'){ 
$list = array();
$sql = "SELECT $fields FROM $table"; 
if ($where) { $sql .= ' WHERE '.$where; } 
$result = $this->query($sql); 
if ($this->num_rows($result)){ 
while($data = $this->fetch_assoc($result)){ 
$list[] = $data; 
} 
return $list; 
} else { 
return false; 
}  
}

PDO

public function get_table($table, $where='', $fields='*'){ 
$db_link = self::getLink(); 
$list = array(); 
$sql = "SELECT $fields FROM $table"; 
if ($where) { $sql .= ' WHERE '.$where; }
$result = $db_link->query($sql); 
while($data = $result->fetch()){ 
$list[] = $data; 
} 
return $list; 
}

Вот собственно и всё. Ещё немного о безопасности. В системе встречаются не подготовленные запросы. Т.е. запросы формирующиеся в самом скрипте.В основном это INSERT. Например в файле /components/users/file.php

$sql = "INSERT INTO cms_user_files(user_id, filename, pubdate, allow_who, filesize, hits, vimage,time,title) 
VALUES ({$usr['id']}, '$name', NOW(), 'all', '$size', 0, '$vimage','$time', '$uname')";

В таких случаях безопасней использовать placeholder. Для PDO данный запрос выглядит так

$STH = $DBH->prepare("INSERT INTO cms_user_files(user_id, filename, pubdate, allow_who, filesize, hits, vimage,time,title) 
VALUES ({$usr['id']}, '$name', NOW(), 'all', '$size', 0, '$vimage','$time', '$uname')");
$STH->execute();

Двухшаговый способ дает все преимущества prepared statements(связываемые переменные).
Использование prepared statements укрепляет защиту от SQL-инъекций.
Так как теоритически становится невозможно провести SQL-инъекцию через данные, используемые в placeholder’ах. Вот так выглядет более «безопасный» вариант.

$data = array(user_id, filename, pubdate, allow_who, filesize, hits, vimage,time,title); 
$STH = $DBH->prepare( "INSERT INTO cms_user_files(user_id, filename, pubdate, allow_who, filesize, hits, vimage,time,title) 
        VALUES (?,?,?,?,?,?,?,?,?)");
$STH->execute($data);

Вот теперь пожалуй и всё. Надеюсь моя писанина поможет тому, кто думает попробовать тоже, что и я.

История рейтинга комментариев

pdo база данных подключение

24 комментария

Реклама #

r2 10 лет назад #

Не проще ли было занаследовать cmsDatabase и переопределить только метод initConnection()? Чтобы не изобретать велосипед

Lora 10 лет назад #

Наверное, да. Но вот с этим наследием пока не разобрался. Разберусь и поправлю. Так возможно и осилю все эти процедуры,методы, ф-ции и классы. smile

r2 10 лет назад #

С наследием все просто. Допустим есть файл first.php с таким классом:

Код PHP:

//first.php
class first {
    public function sayOne() {
        echo '111';
    }
}

тогда можно создать другой класс в файле second.php:

Код PHP:

//second.php
class second extends first {
    public function sayTwo() {
        echo '222';
    }
}

Потом вы подключаете оба файла и создаете объект второго класса:

Код PHP:

include 'first.php'; include 'second.php';
$obj = new second();

Этот объект будет иметь все методы своего класса, и всех родительских:

Код PHP:

 
$obj->sayOne();
$obj->sayTwo();
// выведет 111222;

Из родительского класса доступны только публичные (public) и защищенные (protected) методы.

Lora 10 лет назад #

Спасибо, с этим понятно, но при применении к данному случаю приходится private function __construct() прописывать в дочернем классе и превращать в public function __construct(), иначе ругается. Это на что нибудь отразится?

r2 10 лет назад #

да, при наследовании от cmsDatabase вы должны определить свой конструктор, в котором можете сразу создать подключение к нужной базе. Алгоритм такой:
1. Создаете файл /core/classes/mydb.class.php
2. Определяете в нем класс, наследуемый от cmsDatabase:

Код PHP:

class myDatabase extends cmsDatabase {
    public function __construct() {
        // ... получаете реквизиты из конфига ...
        // и создаете соединение:
        $this->db_link = mysqli_connect($host, $user, $pass, $base);
    }
}

3. В нужном месте вашего компонента подключаете свой класс:

Код PHP:

cmsCore::loadClass('mydb');

4. Создаете и используете объект своего класса. У него будут доступны все методы cmsDatabase:

Код PHP:

$myDB = new myDatabase();
$result = $myDB->query("SELECT * FROM table");
// и т.д.

Lora 10 лет назад #

С базой всё получилось, а вот с model.php не хочет. Запутался в переменных. В конструкторе model.php пишу

Код PHP:

cmsCore::loadClass('my_pdo');
 $this->inDB_S = new cmsMy_pdo();

при

Код PHP:

$result =  $this->inDB_S->query($sql);

Fatal error

Fuze 10 лет назад #

Fatal error

Так а текст ошибки то какой, на что ругается? Метод query в вашем классе есть?

Код PHP:

$result =  $this->inDB_S->query($sql);

Где пишите?

Вы в объекте модели (это так же обычный класс) создаете свойство inDB_S, в которое помещаете объект своего класса работы с базой. Если это свойство вы не определили как приватное, то к нему можно обращаться через объект модели извне, например $model->inDB_S->название_метода(); Или же внутри самого класса модели, в его методах через $this, при условии, что метод, из которого вы обращаетесь к свойству объекта не статический.

Lora 10 лет назад #

Вроде так и делаю.

Код PHP:

class cmsMy_pdo extends cmsDatabase {
 
public function __construct(){
		$this->db_link   = self::initConnection();
}
 
private static function initConnection(){
....................................................................
$db_link = new PDO ( $dsn, $user, $password, $options ) ;
........................................................
}

class cms_model_music{

Код PHP:

	public function __construct(){
cmsCore::loadClass('my_pdo');
 $this->inDB_S = new cmsMy_pdo();
$DB =  $this->inDB_S;
}
   public function selectMusic_Albums(){
.......................................
$sql =("SELECT 1");
$result = $DB->query($sql); // ругается на эту строку
}
}

Код PHP:

Fatal error: Call to a member function query() on a non-object in

Я понимаю, что что то с этими $this, но что..., так как если я подключаю свой вариант класса на прямую, то всё работает. Ладно, утро вечера мудренее.

r2 10 лет назад #

Так вы объявляете локальную переменную $DB в конструкторе, а затем в другом методе пытаетесь к ней обратиться. Разумеется, будет ошибка. У вас должно быть обращение к $this->inDB_S, а не к $DB, которая в этом методе не существует.

Lora 10 лет назад #

С утра отключили сеть, так что "от нечего делать" я допёр, что

Код PHP:

$DB = $this->inDB_S;

не работает. Но прикол в том, что

Код PHP:

$this->inDB_S

тоже не работает, а работает только так

Код PHP:

 
$result = $this->inDB_S->db_link->query($sql); // в модели
$result = $this->db_link->query($sql);                 //  в class cmsMy_pdo extends cmsDatabase

Подскажите можно ли как то избавиться от этого тройного обращения? ( Поэтому я и пытался $this->db_link поместить в переменную $DB).

Fuze 10 лет назад #

Подскажите можно ли как то избавиться от этого тройного обращения?

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

не работает. Но прикол в том, что

дак ясное дело, r2 выше вам и описал почему так.

тоже не работает

значит в $this->inDB_S не объект базы со всеми методами, а что-то, ведомое только вам, некий объект класса cmsMy_pdo. В общем, вам надо прекратить делать методом научного тыка и пытаться понять что вы пишите.

а работает только так

так правильно, потому что вы сами же отправили в свойство вашего класса объект PDO судя по всему, ибо я не вижу что возвращает ваш метод initConnection. Иными словами ваш класс cmsMy_pdo не выполняет ничего и смысла в нем нет никакого. Проще в модели сразу подключать объект PDO

Код PHP:

$this->inDB_S = new PDO ( $dsn, $user, $password, $options );

$this->db_link - тут подразумевается, что должен быть указатель соединения с базой, возвращаемый например функцией mysqli_connect и подобными.

Lora 10 лет назад #

Что значит

cmsMy_pdo не выполняет ничего и смысла в нем нет никакого

. Это дочерний класс cmsDatabase.

Спойлер

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

r2 10 лет назад #

Так вы определитесь что вы хотите использовать PDO или cmsDatabase. Оба этих класса имеют реализацию метода query(). Если вы наследуетесь от cmsDatabase то в $this->db_link должен быть линк на подключение, который возвращается из mysqli_connect(). Нет смысла в db_link пихать объект PDO. Смысл наследования от cmsDatabase как раз заключается в том, чтобы обойтись без всяких PDO, а использовать стандартный класс системы, только подключенный к другой базе.

Lora 10 лет назад #

Понятно.

maxisoft 10 лет назад #

и то и другое, и всего побольше. Это как в мультике про Палкана.

Lora 10 лет назад #

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

Fuze 10 лет назад #

Вот и весь код

который подтвердил мои предположения о ненужности представленного вами класса.
В общем, в вашем случае, что пытается до вас донести r2, нужно сделать примерно вот так (актуально для InstantCMS 1.10.4 и php 5.3):
1. в системном классе cmsDatabase изменить строки

Код PHP:

private function __construct(){
    $this->db_link   = self::initConnection();
    $this->db_prefix = cmsConfig::getConfig('db_prefix').'_';
}

на

Код PHP:

protected function __construct(){ // сменили область видимости метода
    $this->db_link   = static::initConnection(); 
    $this->db_prefix = cmsConfig::getConfig('db_prefix').'_';
}

строку

Код PHP:

private static function initConnection(){

заменить на

Код PHP:

protected static function initConnection(){

Все вышеперечисленное будет в системе со следующего релиза.
2. Создать свой класс (имя файла например mydb.class.php), для подключения другой базы

Код PHP:

<?php
class cmsMyDatabase extends cmsDatabase {
 
    private static $this_instance;
 
    protected function __construct(){
        parent::__construct();
    }
 
    public static function getInstance() {
        if (self::$this_instance === null) {
            self::$this_instance = new self;
        }
        return self::$this_instance;
    }
 
    protected static function initConnection(){
 
        $ini = PATH.'/includes/config.ini';
 
        $parse = parse_ini_file($ini , true) ;
 
        $db_link = mysqli_connect($parse['db_host'],
 $parse['db_user'],
 $parse['db_password'],
 $parse['db_base']);
 
        if (mysqli_connect_errno()) {
           die('Cannot connect to MySQL server: ' . mysqli_connect_error());
        }
 
        mysqli_set_charset($db_link, 'utf8');
 
        return $db_link;
 
    }
 
}

3. Использовать этот класс например так (в модели):
в конструкторе:

Код PHP:

cmsCore::loadClass('mydb');
$this->myDB = cmsMyDatabase::getInstance();

в методах модели:

Код PHP:

$result = $this->myDB->query('SELECT 1');

p.s. писал тут в окне, не проверял, но по идее все должно работать, если синтаксических ошибок не допустил)

Lora 10 лет назад #

В принципе у меня так и получилось, только я не в private изменил, в public. И в начале не понял r2 о смысле использования дочернего класса. Из за этого и затупил.

Lora 10 лет назад #

И наверное в модели

Код PHP:

$this->myDB = new cmsMydb();

вместо

Код PHP:

 $this->myDB = cmsMyDatabase::getInstance();

Fuze 10 лет назад #

И наверное в модели

нет (будет фатал еррор), исходя из моего примера так работать не будет, да и наличие синглтона гарантирует, что соединение с базой будет один раз.

Lora 10 лет назад #

Красиво, особенно сразу не заметный синглтон. Разве для него не обязательны __clone и __wakeup?

Lora 10 лет назад #

P.S. Спасибо за совет.

Alexneva 10 лет назад #

Я так и не понял для чего все эти танцы... зачем компонент в другую базу пихать?

Lora 10 лет назад #

Что бы когда приспичит, знать что и как делать.

ХОЖДЕНИЕ ПО БАЗАМ.

Реклама #

r2 10 лет назад #

Lora 10 лет назад #

r2 10 лет назад #

Lora 10 лет назад #

r2 10 лет назад #

Lora 10 лет назад #

Fuze 10 лет назад #

Lora 10 лет назад #

r2 10 лет назад #

Lora 10 лет назад #

Fuze 10 лет назад #

Lora 10 лет назад #

r2 10 лет назад #

Lora 10 лет назад #

maxisoft 10 лет назад #

Lora 10 лет назад #

Fuze 10 лет назад #

Lora 10 лет назад #

Lora 10 лет назад #

Fuze 10 лет назад #

Lora 10 лет назад #

Lora 10 лет назад #

Alexneva 10 лет назад #

Lora 10 лет назад #

Еще от автора

Загрузка файлов на яндекс диск

Компонент розыгрыши

Уведомления с вашего сайта на телефон

InstantCMS Team

О проекте

Поддержка

Дополнения