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

[piotrooo89]

jak dla mnie to tak: poczytaj o Exceptions, polskie nazwy=porażka, o wzorcu MVC poczytaj (wywal ten html z kodu), operuj na zmiennych, nie echuj co popadnie.

no to chyba tyle, powodzenia życzę. (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[wizu]

Dzięki za odpowiedź. Prosił bym jeszcze o małe rozwinięcie (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

operuj na zmiennych

Czyli jak? Mam dajmy na to, wrzucać do zmiennych np. tekst i wyświetlać go poza klasą? Bo tak to rozumiem.
Ogólnie mam zamiar używać smarty, to było tylko tak dla przykładu.

Drugą sprawą, która mnie nurtuje, to ta (nieszczęsna (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) ) metoda wyswietl.
Tak się zastanawiam, czy zbudowanie jej w ten sposób jest dobrym rozwiązaniem, czy może powinienem jakoś bardziej ją w OOP przerobić (chociaż sam nie za bardzo wiem, co chce przez to napisać...).

Z góry dzięki za następne odpowiedzi.


Ten post edytował wizu 3.07.2009, 17:34:17

[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

[wizu]

Dzięki Ci.
Właśnie głównie chodziło mi o 3 punkt (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
Tylko teraz nie wiem jak mam z nim postąpić. Powinienem budować najpierw klasy ogólne, a takie bardziej szczegółowe dodawać jako inne, ale dziedziczące po tej głównej? O to chodzi? Bo tak to w sumie rozumiem, ale ciężko mi się przestawić z strukturalnego myślenia (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
Będę musiał przyjrzeć się jakiemuś bardziej skomplikowanemu kodowi. Kod phpbb 3 będzie dobry do celów edukacyjnych? Tak, żebym też się nie rzucał nie wiadomo na co.
Z góry dzięki kolejny raz.

[erix]

Google: MVC.