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.

[nevt]

a dlaczego w ogóle upierasz się przy:

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?

(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) 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

[phpion]

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