[MySQL][PHP]Weryfikacja kodu, obiektowe php Opcje

[pablo_83]

Witam,
ponieważ uczę się dopiero obiektowego php, proszę o pomoc. Napisałem poniższy kod, który wyświetla nazwiska z BD. W tej postaci działa poprawnie.
Proszę o zweryfikowanie kodu bo na pewno zawiera on błędy, których ja nie widzę.
Czy w funkcji ShowClient może być echo? Wydaje mi się że powinno być to chyba inaczej rozwiązane ale nie wiem jak.
Nie mam konstruktora ale nie wiem co do niego mam wstawić, defakto można by całą funkcje connecting zawrzeć jako konstruktor ale czy to dobre rozwiązanie?
Proszę o weryfikacje i pomoc przy poprawkach kodu, dzięki.

[PHP] pobierz, plaintext 
<?php 
 
class Db_connect
{
 
    private function connecting ()
    {
        $host = 'localhost';
         $user = 'root';
         $password = 'xxx';
         $db = 'Serwis';
 
        $this->db = mysql_connect($host, $user, $password, $db) or die (mysql_error('Nie mogę nawiązać połaczenia'));
        mysql_select_db($db, $this->db) or die (mysql_error('Nie mogę połaczyć z bazą danych'));
        mysql_query("Set Names 'utf8'");
        mysql_query("Set collation_connection = utf8_polish_ci");
 
    }
 
    private function closing ()
    {
        $this->connecting ();
        $close =$this->db;
        mysql_close($close);
    }
 
    public function ShowClient ()
    {
        $this->connecting();
        //$nazwisko = $this->nazwisko;
 
        $query = ('Select * from Klient');
        $results = mysql_query($query);
        $ile = mysql_num_rows($results) or die (mysql_error('Nie mogę wykonać zapytania.'));
        if ($ile>0)
        {
        for ($i=0;$i<$ile;$i++)
            {
            $row = mysql_fetch_array($results);
            echo $row['Nazwisko'].'</br>';
            }
       }    
        $this->closing();
    }
}
 
$baza = new Db_connect();
$baza->ShowClient();
?>
[PHP] pobierz, plaintext 

jak mogę wstawić hasła z innego pliku? tzn jak użyć include/require i jak ma wyglądać plik z hasłami?
Wystarczy że będzie on zwykłym plikiem php i będzie wyglądał tak:

[PHP] pobierz, plaintext 
<?php
$host = 'xxx';
$user = 'xxx';
itd.
?>
[PHP] pobierz, plaintext 

[askone]

Hej

Pomyśl nad dodaniem:

• konstruktora w którym ustawiać będziesz dane odnośnie bazy, tabeli. Pola te powinny być private.
• funkcji do połączenia, zadeklarowanej jako public
• każda funkcja realizująca pewne ściśle określone zadanie np. close, ShowUser powinna ograniczać się tylko to zadań dla jakich została napisana... Usuń więc łączenie do bazy z każdej z tych metod. Ewentualnie możesz dodać pole, w którym będziesz trzymał info czy jest połączenie.

To tyle...

ps. nazwa klasy sugeruje jej dużą elastyczność, jednak lista metod udostępnianych mocno ją ogranicza do konkretnych zadań. Rozwiązanie: zmień nazwę klasy - lub zamiast np. ShowUser zrób metodę Select($query).

Ten post edytował fly474 15.09.2009, 11:17:51

[paw-e-l]

co do funkcji show_client to ja bym raczej postawił na return

[PHP] pobierz, plaintext 
    public function ShowClient ()
    {
        $this->connecting();
        //$nazwisko = $this->nazwisko;
 
        $query = ('Select * from Klient');
        $results = mysql_query($query);
        $ile = mysql_num_rows($results) or die (mysql_error('Nie mogę wykonać zapytania.'));
        if ($ile>0)
        {
        for ($i=0;$i<$ile;$i++)
            {
            $row = mysql_fetch_array($results);
            $output .= $row['Nazwisko'].'</br>';
            }
       }    
        $this->closing();
        return $output;
    }
[PHP] pobierz, plaintext 

a jeszcze lepszym rozwiązaniem wydaje się zwracanie tablicy

[PHP] pobierz, plaintext 
    public function ShowClient ()
    {
        $this->connecting();
        //$nazwisko = $this->nazwisko;
 
        $query = ('Select * from Klient');
        $results = mysql_query($query);
        $ile = mysql_num_rows($results) or die (mysql_error('Nie mogę wykonać zapytania.'));
        if ($ile>0)
        {
        for ($i=0;$i<$ile;$i++)
            {
            $row = mysql_fetch_array($results);
            $output[] = $row['Nazwisko'];
            }
       }    
        $this->closing();
        return $output;
    }
[PHP] pobierz, plaintext 

i potem przy wypisywaniu tylko

[PHP] pobierz, plaintext 
$tablica = $twoja_klasa->ShowClient();
foeach ($tablica as $row) {
        echo $row.'<br />';
}
[PHP] pobierz, plaintext 

sorry jeśli namieszałem bo teraz bawie się w kohanie
Pozdrawiam

Ten post edytował paw-e-l 15.09.2009, 11:27:04