Pisze sobie pseudo sterownik do obsługi bazy danych narazie mam tyle

[PHP] pobierz, plaintext 
<?php
class bazaMySQL
 {
  var $BazaPolaczenie;
  var $BazaWynik;
  var $BazaHost;
  var $BazaNazwa;
  var $BazaUzytkownik;
  var $BazaHaslo;
  var $BazaWiersz;
 
    function __construct ($b_Host, $b_Nazwa, $b_Uzytkownik, $b_Haslo)
    {
    $this->BazaHost   = $b_Host;
    $this->BazaNazwa = $b_Nazwa;
    $this->BazaUzytkownik   = $b_Uzytkownik;
    $this->BazaHaslo   = $b_Haslo;
    $this->BazaPolaczenie = mysql_connect( $this->BazaHost, $this->BazaUzytkownik, $this->BazaHaslo );
      mysql_select_db( $this->BazaNazwa, $this->BazaPolaczenie ) || die ("Baza danych wygenerowała błąd: ".mysql_error());
    }
 
   function wykonajZapytanie ($b_SQL)
    {
    $this->BazaWynik = mysql_escape_string($b_SQL);
    $this->BazaWynik = mysql_query ($b_SQL, $this->BazaPolaczenie);
    $this->BazaWiersz = mysql_fetch_array( $this->BazaWynik );
        return true;
    return false;
    }
 
   function kolejnyWiersz ()
    {
    $this->BazaWiersz = mysql_fetch_array( $this->BazaWynik );
    if (empty ($this->BazaWiersz))
        return false;
    return true;
    }    
 
   function pokazWiersz ($b_Wiersz)
    {
    if ( isset ($this->BazaWynik))
        {        
        $wiersz = $this->BazaWiersz[$b_Wiersz];
        return $wiersz;
        }
    }
 
    function pokazStatystyke ($b_SQL)
     {
     return mysql_num_rows($this->BazaWynik[$b_SQL]);
     }
 
   function  __destruct() {
       mysql_close($this->BazaPolaczenie);
       print "<br>Zakończono połączenie z baza danych.";
    }
  }
?>
[PHP] pobierz, plaintext 

Sposób wywołania

[PHP] pobierz, plaintext 
<?php
require "sterownikidefinicja_baza_danych.php";
require "sterownikisterownik_baza_danych.php";
 
$polaczenie = new bazaMySQL($host,$baza,$uzytkownik,$haslo);
 
$zapytanie = "SELECT id_test, test FROM test";
 
$polaczenie->wykonajZapytanie($zapytanie);
$polaczenie->pokazStatystyke($zapytanie);
 
do {
print ($polaczenie->pokazWiersz("test"));
} while ($polaczenie->kolejnyWiersz());
?>
[PHP] pobierz, plaintext 

1.Mam problem odnośnie metody pokazStatystyke(), chciałby wyświetlić liczbę rekordów ale zamiast tego otrzymuje błąd.
2.Co można było by tu zoptymalizować, napisać prościej, może wprowadzić modyfikatory dostępu, etc?

Pozdrawiam

Ubezpieczasz zapytanie, a i tak z tego nie korzystasz.


Błędów możesz mieć N!^N.


Ok, polskie nazwy zmiennych, to jest zły pomysł. Poza tym, mógłbyś pluć wyjątkami, a nie tylko false przy niepowodzeniu. Bool na wyjściu nic nie mówi.

poprawione





dodałem return false

Ale false nie powie, że błędem jest brak połączenia z bazą, niepoprawne zapytanie, uwalona baza, czy jeszcze jakieś inne nieprawidłowości. Chyba w jakimś celu wymyślono wyjątki, nie?


A od kiedy to mysql_query" title="Zobacz w manualu PHP" target="_manual zwraca tablicę?

nie zwraca, funkcja obsługuje tylko jedno zapytanie, co nie zmienia faktu ze dalej nie wiem jak zapisac metode mysql_num_row

pozdrawiam

Ale ty uzywasz tak jakby zwracalo
return mysql_num_rows($this->BazaWynik[$b_SQL]);

to jak w końcu ma być, bo kończą mi się pomysły....

Ale ty nie masz nic wymyslac, tylko chwile sie zastanowic... skoro:

[PHP] pobierz, plaintext 
<?php
$this->BazaWynik = mysql_query ($b_SQL, $this->BazaPolaczenie);
?>
[PHP] pobierz, plaintext 

to ma byc:

[PHP] pobierz, plaintext 
<?php
return mysql_num_rows($this->BazaWynik);
?>
[PHP] pobierz, plaintext 

Takie trudne?