Połączenie z bazą danych, raz., Wygodne i bez zbędnego pisania Opcje

[fannet]

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.

[_olo_1984]

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

[fannet]

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ć.

Ten post edytował fannet 31.01.2009, 12:22:30

[Pax95]

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.

[skowron-line]

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.

Ten post edytował skowron-line 31.01.2009, 13:36:08

[Pax95]

@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 

Ten post edytował Pax95 31.01.2009, 13:59:28

[Tarcil]

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.

[Black-Berry]

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 

Ten post edytował Black-Berry 2.02.2009, 10:46:50

[Tarcil]

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

Ten post edytował Tarcil 2.02.2009, 12:08:22

[skowron-line]

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

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