Witajcie piszę ponieważ mam od dłuższego czasu problem związany z programowaniem obiektowym, otóż staram się uczyć, ale nie jakoś chce do tej głowy wpadać, no ale do rzeczy otóż staram się pisać jakieś klasy w nich metody, ale większość z nich wymaga połączenia z bazą danych i wychodzi że na 10 klas w każdej mam w konstruktorze coś takiego:

[PHP] pobierz, plaintext 
<?php
function __construct()
{
  if(!$this->connect = new mysqli(HOST, USER, PASS, NAME))
  {
    throw new Exception("jak zwykle złe połączenie");
  }
}
?>
[PHP] pobierz, plaintext 

W programowaniu obiektowych chyba podstawowym założeniem jest aby kod się nie powtarzał i żeby jeden napisany kod był wykorzystywany wiele razy, a w moim przypadku wygląda to zupełnie odwrotnie.. Proszę o jakieś naprowadzenie mnie na tą właściwą drogę w mojej i tak już pomieszanej nauce.

napisz sobie prosty sterownik (klase) do mysql, potem używaj jej dowoli. Raz napisany kod wykorzystasz w każdym miejscu aplikacji. Przykładów jest bardzo dużo, poszukaj tu na formu lub google.Pozdr

Słowo kluczowe "sterownik" otworzyło mi bramę do wielu nowych lektur do analizy
Załóżmy że już mam taki sterownik i teraz mi wychodzi na to że będę musiał w konstruktorze tworzyć nowy egzemplarz klasy?

Edit: Teraz pomyślałem przecież mogę chyba skorzystać z rozszerzenia wbudowanego w php mysqli? Wiec pozostaje pytanie jak się do tego odwołać.

Nie widzę niczego złego w twojej klasie. Dodaj destruktor i wykorzystaj wzorzec Singleton, a obiekt inicjuj na początku dokumentu i się nie przejmuj połączeniem w innych klasach.

tak jak juz kolega wyżej napisał napisz sobie sterownik bazy danych a potem.

[PHP] pobierz, plaintext 
<?php
class user{
public function __construct()
{
include_once( 'sterownik.sql.class.php' );
$this->mysq = new sterownikMysql( 'host','user','pass','db' );
}
}
?>
[PHP] pobierz, plaintext 

dołącz go do każdej z klas.

@UP: Właśnie nie o to mi chodziło :]. Połączenie tworzymy tylko jedno, nie inicjujemy go w każdej klasie.
EDIT: Nie będę żałował.

[PHP] pobierz, plaintext 
<?php
class Datebase
{
   private static $s_instance;
 
   private function __construct()
   {
      // Połączenie z bazą danych
   }
 
   public static function getInstance()
   {
      if ( self::$s_instance === null )
      {
         self::$s_instance = new self;
      }
      return self::$s_instance
   }
 
   public function __destruct()
   {
      // Usunięcie połączenia
   }
}
?>
[PHP] pobierz, plaintext 

Inicjujemy na samym początku np. index.php

[PHP] pobierz, plaintext 
<?php
$Datebase = Datebase::getInstance();
//dalsze operacje
?>
[PHP] pobierz, plaintext 

Witam

Chciałbym dorzucić mój problem do tego tematu.

Napisałem klasę, która w konstruktorze tworzy połączenie z bazą danych oraz zawiera metody insertQuery, selectQuery itd, które przygotowują zapytanie do bazy, wykonywane prywatną metodą doQuery. Wszystko chodziło pięknie, ale był problem, kiedy przyszło zagnieździć zapytanie do bazy w zapytaniu. Już tłumaczę:

Załóżmy, że mamy dwie tabele.

[PHP] pobierz, plaintext 
<?php
Tabela news:
ID int
title varchar(255)
time int
text text
authorID
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
Tabela administrators:
ID int
name varchar(40)
pass char(32)
?>
[PHP] pobierz, plaintext 

Tworzę sobie egzemplarz obiektu do obsługi bazy

[PHP] pobierz, plaintext 
<?php
$sql = new mySql(HOST, USER, PASS, NAME)
?>
[PHP] pobierz, plaintext 

Wyszukuję newsy w bazie:

[PHP] pobierz, plaintext 
<?php
$sql->selectQuery('title, time, authorID', 'news');
while($news = $sql->fetchResults())
{
//wyświetlam news
//tutaj musze wykonać coś takiego:
mysql_query('SELECT name FROM administrators WHERE id='.$news['authorID']);
 
}
?>
[PHP] pobierz, plaintext 

Tyle, że nie po to pisałem klasę, żeby używać mysql_query(). To jedna sprawa, druga jest taka, że jeśli zamiast mysql_query() użyję znów $sql->selectQuery to zmieni mi to zapytanie używane do przewijania wyników w linijce while($news=$sql->fetchResults(). Muszę więc tworzyć dodatkowy obiekt (a to jest drugie połączenie z bazą danych - coś, czego trzeba unikać).
Sprawę rozwiązałem tak, że wydzieliłem kod połączenia z bazą danych do klasy mySqlConnect, a zapytania do klasy mySqlQueries. Tworzę jedno połączenie z bazą, a potem odpowiednio do potrzeb tworzę obiekty odpowiadające za przygotowanie zapytań do bazy:

[PHP] pobierz, plaintext 
<?php
$sql = new mySqlConnect(HOST, USER, PASS, DBNAME);
$query_news = new mySqlQueries($sql->handler);
$query_author = new mySqlQueries($sql->handler);
$query_news->selectQuery('id, title, text, authorID', 'news');
while($news = $query_news->fetchResults())
{
//wyświetlanie newsa...
$query_author->selectQuery('name', 'administrators', 'id='.$news['authorID']);
$admin = $query_author->fetchResults();
//wyświetlenie nazwy autora newsa
}
$query_news->__destruct();
$query_author->__destruct();
?>
[PHP] pobierz, plaintext 

Tak to działa u mnie... tylko, że coś mi mówi, że to nie jest dobre podejście... Czy ktoś podejmie się oceny takiego rozwiązania i ewnetualnie wskaże mi bardziej właściwą drogę?

Pozdrawiam
Tarcil
PS. Przepraszam, jeżeli wciąłem się w złym temacie.

Twój problem jest bardzo 'klasyczny'. Masz tak:

[PHP] pobierz, plaintext 
<?php
while($news = $query_news->fetchResults())
?>
[PHP] pobierz, plaintext 

a jeśli masz zagnieżdzenia powinieneś mieć tak:

[PHP] pobierz, plaintext 
<?php
$result =  $query_news->getResult();
while($news = $query_news->fetchResults($result))
?>
[PHP] pobierz, plaintext 

A co do wyższych postów to na dalszym etapie prac przekonacie się że singletony to bagno w które lepiej się nie pakować. Było wiele wątków na ten temat. Zainteresujcie się rozwiązaniem za pomocą kontekstu;

Tarcil kombinujesz bez singletona ale coś masz nie tak; Zrób sobie plik Bootstrap.php a w nim:

[PHP] pobierz, plaintext 
<?php
function __autoload($class)
    {
        $class = str_replace('_', '/', $class);
        require_once $class . '.php';
    }
    
    //creating core objects...
    $config = new Config();
    $log = new Log();
    $dbDriver = new Db_Driver();
    $session = new Session();
    $language = new Language();
    $user = new User();
    [b]$news= new News();[/b]
    $router = new Router();
    $buffer = new Buffer();
    
    //creating context...
    $context = new Core_Context();
    $context->setConfig($config);
    $context->setLog($log);
    $context->setDbDriver($dbDriver);
    $context->setSession($session);
    $context->setLanguage($language);
    $context->setUser($user);
    [b]$context->setNews($news);[/b]
    $context->setRouter($router);
    $context->setBuffer($buffer);
    
    //setting context...
    $log->setContext($context);
    $dbDriver->setContext($context);
    $session->setContext($context);
    $language->setContext($context);
    $user->setContext($context);
    [b]$news->setContext($context);[/b]
    $router->setContext($context);
    $buffer->setContext($context);
    
    //other...
    $config->load(MAIN_PATH . 'config.ini');
    $dbDriver->connect();
    $dbDriver->query('SET NAMES `utf8`');
    $language->initialize();
   [b] $news->initialize();[/b]
?>
[PHP] pobierz, plaintext 

Dzięki
Jak znajdę trochę czasu na przepisanie kodu to na pewno skorzystam

Black-BerryChciałem Ci dać "Pomógł" ale nie widzę nigdzie odpowiedniego przycisku...

Pozdrawiam

I nie znajdziesz jeżeli nie jesteś autorem Wątku.