Witam, nie dawno zająłem się klasami w PHP i po napisaniu klasy DataBase i klasy User pojawił się problem jak klasa User ma korzystać z utworzonego obiektu klasy DataBase, aby wykonywać operacje na bazie danych. Aby ominąć chwilowo problem napisałem takie coś:

[PHP] pobierz, plaintext 
<?php
 class DB {
 
     protected $db, $select_db;
  
     private function DBEx($Ex){
         die($Ex);         
     }
  
     public function Connect($host, $port, $user, $password){
  
         $this->db = mysql_connect($host.':'.$port, $user, $password);
              
     }
     public function isConnected(){
  
         if(!$this->db) return FALSE;
         else return TRUE;
  
     }
  
     public function query($sql){
         if($this->isConnected()){
             $result = mysql_query($sql, $this->db);
             if(is_numeric($result)) return $result;
             $table = array();
             if (@mysql_num_rows($result) > 0)
             {
                 $i = 0;
                 while($table[$i] = mysql_fetch_assoc($result)) 
                     $i++;
                 unset($table[$i]);                                                                                  
             }                                                                                                                                     
             @mysql_free_result($result);
             return $table; 
         } else $this->DBEx('DB_NOT_CONNECTED');
     }
  
 }
 
class User {
  
     function User(){
         $this->db = $GLOBALS['DB'];
     }
  
     private function UserEx($Ex){
         die($Ex);         
     }
  
     public function find($name, $email = false){
         if($email) $res = $this->db->query('SELECT * FROM `user` WHERE `email` = ''.$name.''');
         else $res = $this->db->query('SELECT * FROM `user` WHERE `name` = ''.$name.''');
         foreach($res as $user);
         {
             $this->data['id'] = $user['id'];
             $this->data['name']    = $user['name'];
             $this->data['password'] = $user['password'];
             $this->data['salt'] = $user['salt'];
             $this->data['email'] = $user['email'];
             $this->data['confirm_mail'] = $user['confirm_mail'];
             $this->data['points'] = $user['points'];
             $this->data['access'] = $user['access'];
         }
      
     }
}
 
 
 
$GLOBALS['DB'] = new DB();
 
$DB->connect('host','port','user','haslo')
 
$user = new User();
 
$user->find('nick');
?>
[PHP] pobierz, plaintext 

I takie rozwiązanie się sprawdza, ale przeczytałem gdzieś, że nie jest poprawne, bezpieczne, etc.



Tu pojawia się moje pytanie, jak nie tak to zrobić, aby nie musieć przy każdym stworzeniu obiektu User() podawać w parametrze obiektu bazy danych?

Jestem początkujący w klasach, nie wiem czy dobrze wszystko nazwałem, ale myślę, że jakoś wytłumaczyłem, o co mi chodzi ;].

Dziękuję za przeczytanie tego postu.

Ja bym to zrobił mniej-więcej w ten sposób.

[PHP] pobierz, plaintext 
<?php
class Database {
    private $db;
    
    public function connect(array $params) {
        //
    }
    
    public function disconnect() {
        //
    }
    
    public function query($query) {
        //
    }
    
    // itd
}
 
abstract class Model {
    protected $db;
 
    public function __construct(Database $db) {
        $this->db = $db;
    }
}
 
class User extends Model {
    public function get() {
        $this->db->query('SELECT * FROM user');
        //
    }
}
?>
[PHP] pobierz, plaintext 

Pozwól, że nie będę opisywał kodu - chyba jest on dość czytelny.

a dlaczego w ogóle upierasz się przy:

przecież przekazanie jednego parametru do konstruktora nie jest żadnym obciążeniem kodu...

możesz napisać klasę DB wg wzorca SINGLETON - ale jest to równie nie polecane, co korzystanie ze zmiennych globalnych

Racja. Jeżeli napiszesz klasę jako singleton to zamykasz sobie drogę do równoczesnego korzystania z kilku baz (tego samego lub różnego typu). Niby jest to wykorzystywane bardzo rzadko ale lepiej pozostawić sobie taką furtkę.

Po prostu obiekt ten w innej klasie nie jest już widziany. Zamiast przekazywać ten obiekt możesz sprawić aby był widziany globalnie. Wstaw to w metodzie w której korzystasz z obiektu.

[PHP] pobierz, plaintext 
<?php
global $nazwaobiektu;
?>
[PHP] pobierz, plaintext 

No i to własnie najgorsze rozwiązanie, gorsze niż singelton. :/