prezentowanie wyników w mysqli Opcje

[PawelC]

Hej Wreszcie udało mi się, nawiązać połączenie z bazą przy pomocy mysqli i wykorzystaniem połączenia do pobierania danych. Wrzuciłem połączenie i wybór bazy w konstruktor, bo jest on zawsze automatycznie ładowany, i w momencie pobierania danych narodziło się pytanie. Mam taki kod:

[PHP] pobierz, plaintext 
<?php
 
 
        class sqli
            {
                
            public function __construct($host,$login,$haslo,$baza)
            {
                $this->mysqli=new mysqli($host,$login,$haslo,$baza);
            }
            
            
            
            public function dane($query)
            {
                $result=$this->mysqli->query($this->query=$query);
                while($data=$result->fetch_object()){
                
                    echo $data->kategoria;
                }
            }
        
            
        }
 
$sqli=new sqli('localhost','root','','katalog');
$sqli->dane("select kategoria from kategorie");
 
?>
[PHP] pobierz, plaintext 

I teraz chodzi mi o pobieranie danych, czy za każdym razem muszę robić osobno takie coś:

[PHP] pobierz, plaintext 
<?php
public function dane($query)
             {
                 $result=$this->mysqli->query($this->query=$query);
                 while($data=$result->fetch_object()){
                 
                     echo $data->kategoria;
                 }
?>
[PHP] pobierz, plaintext 

W tym kodzie powyżej pobieram nazwę kategorii, a teraz jakbym chciał pobrać podkategorie z innej tabeli, to muszę na nową stworzyć to samo, tyle że, ze zmienionymi danymi, czyli:

[PHP] pobierz, plaintext 
<?php
public function podkategorie($query)
             {
                 $result=$this->mysqli->query($this->query=$query);
                 while($data=$result->fetch_object()){
                 
                     echo $data->podkategoria;
                 }
?>
[PHP] pobierz, plaintext 

Czy można to zrobić w jakiś inny sposób?
Polskie nazewnictwo funkcji stosuję tylko w celach pokazowych

Ten post edytował ExPlOiT 25.04.2009, 10:50:23

[skowron-line]

Zrób sobie jedna metode do pobierania danych z zapytania.

coś ala

[PHP] pobierz, plaintext 
<?php
public function dane( $query )
{
$q = mysqli->query( $query );
return $q->fetch_object(); // zwroci Ci obiekty ktore sobie pozniej w petli wyswietlisz
}
?>
[PHP] pobierz, plaintext 

[PawelC]

Zrobiłem coś na podobę jak napisałeś:

[PHP] pobierz, plaintext 
<?php
public function dane($query)
            {
                $result=$this->mysqli->query($this->query=$query);
                return $result->fetch_object();
            }
                    
            
        
 
$sqli=new sqli('localhost','root','','katalog');
foreach($sqli->dane("select kategoria from kategorie") as $key)
{
    echo $key;
}
?>
[PHP] pobierz, plaintext 

I wszystko ładnie pięknie pobiera itp, ale wyświetla mi tylko jeden rekord!Dlaczego?

Ten post edytował ExPlOiT 25.04.2009, 11:10:39

[skowron-line]

Kod

foreach($sqli->dane("select kategoria from kategorie") as $key)

Tak nie rob bo przy kazdym obruceniu sie petli bedzie sie to zapytane wykonywalo

[PHP] pobierz, plaintext 
<?php
$sqli=new sqli('localhost','root','','katalog');
$x = $sqli->dane("select kategoria from kategorie");
echo'<pre>';
print_r( $x  );
?>
[PHP] pobierz, plaintext 

zobacz co będzie.

[PawelC]

Dostałem coś takiego:

[PHP] pobierz, plaintext 
<?php
stdClass Object
(
    [kategoria] => Biznes i Ekonomia
 
)
?>
[PHP] pobierz, plaintext 

Tylko dlaczego pobiera jeden rekord?

[skowron-line]

ponieważ element kategoria jest nadpisywany az do ostatniego
http://pl2.php.net/manual/pl/mysqli-stmt.fetch.php#83284 zobacz to

[PawelC]

Czarowałem i wymyśliłem coś takiego:

[PHP] pobierz, plaintext 
<?php
public function dane($query)
            {
                $result=$this->mysqli->prepare($this->query=$query);
                $result->execute();
                $result->bind_result($name);
                    while ($result->fetch()) 
                  {
                    print ($name);
                  }
 
            }
                    
            
      }
$sqli=new sqli('localhost','root','','katalog');
$sqli->dane("select kategoria from kategorie");
?>
[PHP] pobierz, plaintext 

Wynik daje taki jaki chcę czyi:

Biznes i Ekonomia Blogi Internetowe Domy i Nieruchomo?ciEdukacja i Nauka Firmy wg. Bran? Handel Elektroniczny Handel Tradycyjny Hobby Katalogi Stron / Firm Komputery i Internet Kultura i Sztuka Medycyna Motoryzacja Muzyka Og?oszenia, AukcjeRegionalne Rozrywka Sport i Turystyka Szko?y i Uczelnie Telefonia Zdrowie i Uroda Zwierz?ta

I teraz jest wszystko ładnie i pięknie he he, podany przykład pobiera tylko jedną kolumne z bazy, a co w wypadku pobierania kilku kolumn?
Wiem że mogę to zrobić tak:

[PHP] pobierz, plaintext 
<?php
public function dane($query)
            {
                $result=$this->mysqli->prepare($this->query=$query);
                $result->execute();
                $result->bind_result($name,$names);
                    while ($result->fetch()) {
                    
                    print ($name.'<br />');
                    print ($names);
    }
            }
?>
[PHP] pobierz, plaintext 

A jest jakiś bardziej optymalne rozwiązanie, żeby za każdym razem nie podawać ilości kolumn?
Bo przykładowo mam w bazie tabele, podkategorie, a w niej kolumny, id, podkategoria i kategoria, i teraz musiałbym przebudować, trochę kod dane($query);

Ten post edytował ExPlOiT 25.04.2009, 12:10:02

[Crozin]

Genralnie możesz zrobić sobie dwie metody: getRow(), getRows() które działają mniej więcej tak:

getRow():
wykonuje zapytanie
zwraca fetchObject();

getRows():
deklarujesz sobie pustą tablicę
wykonujesz zapytanie
w pętli dopisujesz wynik fetchObject() jako kolejny element tablicy
zwracasz tablicę

[PawelC]

z getRow() to wymyśliłem coś takiego:

[PHP] pobierz, plaintext 
<?php
public function getRow($query)
            {
                $result=$this->mysqli->query($this->query=$query);
                return $result->fetch_object();
                
            }
?>
[PHP] pobierz, plaintext 

Tak jak pisałeś, wykonuje zapytanie i zwraca fetch_object()
Z getRows()
coś takiego wyczarowałem:

[PHP] pobierz, plaintext 
<?php
public function getRows($query)
            {
                $dane=array();
                $result=$this->mysqli->query($this->query=$query);
                while($wynik=$result->fetch_object()){
                    $data[]=$wynik;
                    return $data;
                    
                }
            }
?>
[PHP] pobierz, plaintext 

Fakt faktem zwraca id i nazwę kategorii Tylko niewiem czy dobrze to zrobiłem, wynik mi zwraca w postaci:

[PHP] pobierz, plaintext 
<?php
Array (     [0] => stdClass Object         (             [id] => 1             [kategoria] => Biznes i Ekonomia          )  )
?>
[PHP] pobierz, plaintext 

Ten post edytował ExPlOiT 25.04.2009, 13:00:35

[Crozin]

W getRow() możesz ograniczyć się do

[PHP] pobierz, plaintext 
<?php
public function getRow($query)
           {
               return $this->mysqli->query($this->query=$query)->fetch_object();
           }
?>
[PHP] pobierz, plaintext 

getRows() zrobiłeś dobrze z tym, że return powinno być po pętli while(), nie w jej środku.

Ten post edytował Crozin 25.04.2009, 13:01:10

[PawelC]

A z tym ograniczeniem w getRow() to nawet nie wiedziałem że tak można, teraz już na przyszłość będę wiedział

W getRows() wstawiłem return poza pętle i pobiera wszystkie kategorie wraz z ID
Teraz mi daje taki wynik:

[PHP] pobierz, plaintext 
<?php
Array (     [0] => stdClass Object         (             [id] => 1             [kategoria] => Biznes i Ekonomia          )      [1] => stdClass Object         (             [id] => 2             [kategoria] => Blogi Internetowe          )      [2] => stdClass Object         (             [id] => 3             [kategoria] => Domy i Nieruchomo?ci         )
?>
[PHP] pobierz, plaintext 

czyli zadowalający i teraz rozumiem, że muszę przepuścić przez pętle tablice aby wyświetliło normalnie
czyli id kategoria, a nie tak jak wyżej jest.

[Crozin]

Tak.

[PHP] pobierz, plaintext 
<?php
$rows = $...->getRows('....');
foreach($rows as $row){
  echo $row->kategoria;
}
?>
[PHP] pobierz, plaintext 

[PawelC]

Działa super, tak jak chciałem. I mam jeszcze jedno pytanie co powinna zawierać klasa do obsługi bazy danych? Domyślam się że najlepiej wszystko do obsługi konkretnej bazy.

Kolejne pytanie: jak rozwiązać w klasie update,insert i delete, stworzyć osobne metody?
Jak widać do wykonania zapytania używam takiego kodu:

[PHP] pobierz, plaintext 
<?php
$result=$this->mysqli->query($this->query=$query);
?>
[PHP] pobierz, plaintext 

I jedyne czym by się różniły metody, to komunikatem końcowym.

Ten post edytował ExPlOiT 25.04.2009, 23:28:31

[Crozin]

Działa super, tak jak chciałem. I mam jeszcze jedno pytanie co powinna zawierać klasa do obsługi bazy danych? Domyślam się że najlepiej wszystko do obsługi konkretnej bazy.

Nie. Ona jest bardzo abstrakcyjna/uniwersalna. Nie działa z żadną konkretną bazą. Oferuje jedynie zestaw metod ułatwijących innym częściom kodu (modelom) korzystanie z bazy.
Innymi słowy - lepiej korzystaj z PDO niż pisz własną klasę, bo to jest bez sensu - chyba, że robisz to jedynie w celach trenigowych

[PawelC]

Poczytam o PDO, w książce jest trochę o tym
Klasę do obsługi bazy oczywiście piszę w celach treningowych, ponieważ na pisaniu czegoś najlepiej się uczyć, choćby dlatego że doświadczenie idzie nabyć podczas rozwiązywania takich problemów.

Dokończyłem klasę update, select i insert dałem na tej zasadzie:

[PHP] pobierz, plaintext 
<?php
public function update($query)
            {
                $result=$this->mysqli->query($this->query=$query);
                if($result===true)
                {
                    echo "Dane zostały zaktualizowane!";
                }
                else {
                    echo "Wystąpił błąd podczas aktualizowania danych!";
                }
            }
?>
[PHP] pobierz, plaintext 

Teraz czas napisać jakiś logger, który zapisze zapytanie do pliku i czas wykonania

Ten post edytował ExPlOiT 26.04.2009, 22:13:34

[Crozin]

No i to jest źle zrobione.
1) Po co mi takie komunikaty?
2) Jak wszystko poszło dobrze, to nie ma potrzeby niczego zwracać (może być zwykłe TRUE). Jak coś poszło źle to rzuć wyjątkiem, który nada się tu lepiej
3) Metody update/insert/delete to dobry pomysł w ORMach, ale w zwykłych "klasach obsługi baz danych" nie

[PawelC]

3) Metody update/insert/delete to dobry pomysł w ORMach, ale w zwykłych "klasach obsługi baz danych" nie

To co mogę zrobić w zwykłej klasie, zamiast insert etc...

1) Po co mi takie komunikaty?
2) Jak wszystko poszło dobrze, to nie ma potrzeby niczego zwracać (może być zwykłe TRUE). Jak coś poszło źle to rzuć wyjątkiem, który nada się tu lepiej

Wiem, komunikaty dałem tylko tak żeby sprawdzić czy działa Normalnie ich niema, jest tylko w razie błędu.