[PHP][MYSQL]pseudo-sterownik do obsługi bazy danych - problem Opcje

[Czapla]

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

[erix]

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

Ubezpieczasz zapytanie, a i tak z tego nie korzystasz.

ale zamiast tego otrzymuje błąd.

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

2.Co można było by tu zoptymalizować, napisać prościej, może wprowadzić modyfikatory dostępu, etc?

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.

[Czapla]

Ubezpieczasz zapytanie, a i tak z tego nie korzystasz.

poprawione

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

Bool na wyjściu nic nie mówi.

dodałem return false

[erix]

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?

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

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

[Czapla]

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

[nospor]

nie zwraca

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

[Czapla]

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

[nospor]

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?