Proszę o opinię, czy to jest dobry krok w stronę OOP Opcje

[wizu]

Witam.
Napisałem swoją pierwszą klasę (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) Chciałem napisać ją do obsługi mysql, w taki sposób, żeby wyświetlała wszystkie rekordy z tabeli według podanych pól. Ogólnie działa jak trzeba, ale jakoś dziwnie mi to wygląda. Dopiero zaczynam (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

[PHP] pobierz, plaintext 
<?php
class sql
 {
     private $id_zapytania, $mysql;
     
     public function __construct( $host, $user, $pass, $db )
     {
         $this -> mysql = mysql_connect( $host, $user, $pass ) or die('Nie mogę połączyć się z bazą danych: '.mysql_error());
         mysql_select_db( $db ) or die('Nie mogę wybrać bazy: '.mysql_error());
     }
     
     public function __destruct()
     {
         mysql_close( $this->mysql );
     }
     
     private function query( $query )
     {
         if ( !$this->id_zapytania = @mysql_query( $query ) )
         {
             die( 'Bład zapytania MySQL, odpowiedź serwera: '.mysql_error() );
             exit;
         }
         else
         {
             return $this->id_zapytania;
         }
     } 
     
     public function wyswietl( $pola )
     {
         $zapytanie = '';
         $ile = count( $pola );
         for( $i = 0; $i < $ile; $i++ )
         {
             if( $i == 0 )
                 $zapytanie = '`'.$pola[$i].'`';
             else
                 $zapytanie = $zapytanie.', `'.$pola[$i].'`';
         }
         $query = "SELECT $zapytanie FROM `phpbb_nhl_teams`";
         $this->query( $query );
         
         while( $wiersz = mysql_fetch_row( $this->id_zapytania ) )
         {
             for( $i = 0; $i < $ile; $i++ )
             {
                 echo $wiersz[$i].' ';
             }
             echo '<br />';
         }
     }
 }
 
 $sql = new sql( 'localhost', 'root', '', 'nhl' );
 $pola = array( 'id', 'name', 'conference', 'division' );
 $sql->wyswietl( $pola );
?>
[PHP] pobierz, plaintext 

Najbardziej nie pasuje mi tutaj funkcja wyswietl, jest dla mnie jakby strukturalna... nie wiem co o tym myśleć. Potrzebuję kogoś, kto mnie trochę ukierunkuje. Z góry dziękuję za każdą krytykę (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Ten post edytował wizu 3.07.2009, 14:53:54

[Crozin]

1) Nie ukrywaj żadnych błędów (@) bo Twój kod nie powinien żadnych generować
2) Jak już poprzednik zauważył, do obsługi błędów dużo lepsze są wyjątki niż wyświetlenie informacji
3) Metody "wyświetl" w ogóle nie powinno być w tej klasie