ОПП передача наследованного объекта

Mr.M.I.T.
Класс поддержки DB - это вспомогательный класс.
По идее, от него ничего, кроме более специфических классов для работы с БД, наследоваться не должно.
Остальные классы должны его использовать, например так:

PHP:

1. <?php $data=db_class::query("...");

или так:

PHP:

1. <?php $data=db_class::getInstance()->query("...");

но уж всяко не $this->query(), иначе у тебя в каждом объекте будет куча ненужного мусора типа ресурса соединения.

 

нее... у меня например указатель соединения - зашит в статическую переменную..
и метод установки соединения сперва проверяет, а нет ли уже открытого. если есть - юзает его. по логу мускула коннект 1 используется)

 

Не наследуй класс для работы с БД. Храни в main единственный экземпляр этого класса и передавай его в comment.

PHP:

1. <?php
2. class main {
3.     private $db;
4.  
5.     public function __construct() {
6.         $this->db = new db_class();
7.     }
8.  
9.     public function comment() {
10.         $comment = new comment($this->db);
11.     }
12. }
13.  
14.  

 

Sergey89

чем плох вариант:

PHP:

1. <?php
2. class myvars{
3. public static $dbh = false;
4. }
5. class database{
6.     public function dbping(){
7.         if(!myvars::$dbh){
8.             $dbh = new mysqli();
9.             myvars::$dbh =& $dbh;
10.         }
11.     }
12. }
13. class appl extends database{
14.     public function save(){
15.         $this->dbping();
16.         //в результате имеем установленный дескриптор соединения. если он был, то существующий. если не было, то вновь созданный
17.     }
18. }
19. ?>

 

Mr.M.I.T.

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

 

Mr.M.I.T.
а если классов будет не один, два, а хз сколько? на каждую страницу

 

DarkElf, это навешивание на класс appl лишнего функционала. Часто, более удачным решением является именно агрегирование, а не наследование.

PHP:

1. <?php
2. class Appl
3.     public function __construct {
4.          $this->db = Locator::getDb();
5.     }
6. }
7.  
8. class Locator {
9.     private $db;
10.  
11.     public function getDb() {
12.         if (!$this->db) {
13.             $this->db = $this->createDefaultDbConnection();
14.         }
15.  
16.          return $this->db;
17.     }
18.  
19.     //...
20. }

 

Sergey89

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

 

Я делаю так примерно.
Где то в конфиге есть строчка

PHP:

1.  
2. <?php
3. //...
4. $DB = new class_db();
5. //...
6. ?>
7.  

И в каждом классе который использует БД

PHP:

1.  
2. <?php
3. class myclass {
4. private $DB;
5. function __construct(){
6. $this->DB = &$GLOBALS['DB'];
7. }
8.  
9. }
10. ?>
11.  

 

Вльдемар, в PHP 5 объект всегда передаётся по ссылке. А использовать $GLOBALS при работе с ООП, как-то не красиво.

 

ООП не очень хорошо знаю
Можно тогда так

PHP:

1. <?php
2. class myclass {
3. private $DB;
4. function __construct(){
5. $this->DB = class_bd::GetInstance();
6. }
7.  
8. }
9. ?>

 

На мой IMHO подход Sergey89 самый правильный.
По своей сути он делает главное, мух от котлет разводит.

Ведь что такое link. По сути вещь которая вспомогательная и жить она должна сама по себе. Сегодня я к базе link'уюсь, завтра memcached привинтить захотел, да мало ли что - вплоть до хосты могут разные стать.

Основной класс ведь не коннектами должен заниматься, а делать только то для чего он создан.
Вот и пусть link'и отдельный класс делает. А основной в нужном месте его экземпляр параметром получает. Или локатором нужный метод находит. Тут на вкус и цвет и по ситуации смотреть.

Это конкретно жизнь очень сильно облегчит впоследствии. Статейка правильная на эту тему
http://wiki.agiledev.ru/doku.php?id=ooad:manage_dependencies_in_php_code
Там не надо в код вникать. Вводную часть про гниль прочитать, а все остальное можно в двух словах выразить. Главное чтоб классы НЕ зависили друг от друга и как можно меньше знали друг о друге. А все что после введения - это примеры реализации и в каждой ситуации может разный подход оптимальным быть.

 

Вльдемар
Если у тебя класс БД хранится как глобальная переменная, зачем ее копии еще и в каждый объект-то пихать?
Я бы написал global $DB; и дальше работал с этим объектом напрямую. Гибкое и удобное решение. И хоть это и не главный показатель, по скорости/памяти, наверное, наилучшее.

 

У меня че то не работало с global $DB, а разбираться было лень.
Нужно покапаться