Korzystanie z utworzonego obiektu w innej klasie. Opcje

[Mr_KoKa]

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.

[phpion]

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.