[MySQL][PHP] Prosta klasa do obsługi bazy danych, problem z dodaną funkcją obsługującą mysql_fetch_array Opcje

[@rtur]

Znalazłem tutaj prostą klasę do obsługi bazy mysql, ale klasa nie ma obsługi mysql_num_rows i mysql_fetch_array więc postanowiłem dodać, ale nie działa jak powinna, mogę prosić o sprawdzenie i naprowadzenie co jest nie tak?

Baza.class.php

[PHP] pobierz, plaintext 
<?
class db { 
  private $db_host = DB_HOST;
  private $db_name = DB_NAME;
  private $db_user = DB_USER;
  private $db_passw = DB_PASSW;
  // zmienne publiczne - dostępne poza klasą
  public $connection; // połączenie
  public $error; // ew. błąd zwrócony przez bazę danych
 
  public function connect(){
    if ($connection = mysql_connect($this->db_host, $this->db_user, $this->db_passw)) {
      if(mysql_select_db($this->db_name, $connection)) {
        $this->connection = $connection;
        return true;
      } else {
        $this->error = mysql_error();
        return false;
      }
    } else {
      $this->error = mysql_error();
      return false;
    }
  } // eof connect()
 
  public function select($sql) {
     if ($this->connection) {
      mysql_set_charset('utf8', $this->connection);
      if (isset($sql) && $sql != '') {
        if($result = mysql_query($sql)) {
          return $result;
        } else {
          $this->error = mysql_error();
          return false; 
        }
      } else {
        $this->error = 'Błąd zapytania SQL';
        return false;}
    } else {
      $this->error = 'Brak połączenia z bazą danych';
      return false;}
  } // eof select()
   public function query($sql){
     if (isset($sql) && $sql != '') {
      if ($this->connection) {
        mysql_set_charset('utf8', $this->connection);
        if (mysql_query($sql)){
          return true;
        } else {
          $this->error = mysql_error();
          return false;}   
      } else {
        $this->error = 'Brak połączenia z bazą danych';
        return false;}
    } else {
      $this->error = 'Błąd zapytania SQL';
      return false;} 
  } // eof query()
// start dodane przeze mnie ======================================================
   public function numrows($sql){
     if ($this->connection) {
      mysql_set_charset('utf8', $this->connection);
      if (isset($sql) && $sql != '') {
        if($result = mysql_num_rows($this->select($sql))) {
          return $result;
        } else {
          $this->error = mysql_error();
          return false;}
      } else {
        $this->error = 'Błąd zapytania SQL';
        return false;}
    } else {
      $this->error = 'Brak połączenia z bazą danych';
      return false;}
  } // eof numrows()
 
 public function fetcharray($sql){
      if ($this->connection) {
      mysql_set_charset('utf8', $this->connection);
      if (isset($sql) && $sql != '') {
        if($result = mysql_fetch_array($this->select($sql))) {
          return $result;
        } else {
          $this->error = mysql_error();
          return false; 
        }
      } else {
        $this->error = 'Błąd zapytania SQL';
        return false;
      }
    } else {
      $this->error = 'Brak połączenia z bazą danych';
      return false;}
   } // eof fetcharray()
// koniec - dodane przeze mnie ======================================================
  public function close(){
    if ($this->connection){
      if (mysql_close($this->connection)){
        return true;
      } else {
        $this->error = mysql_error();
        return false;}
    } else {
      $this->error = 'Brak aktywnego połączenia';
      return false;}
   } // eof close()
 
}
[PHP] pobierz, plaintext 

fragment kodu php z index.php (wcześniej inicjuje połączenie $db->connect() i definiuje zmienną $db odnoszącą się do klasy)

[PHP] pobierz, plaintext 
$sql = "SELECT wid,czas,podpis FROM tabela WHERE status = '1' ORDER BY 'czas' DESC;";
    if ($db->select($sql)){
       if ($db->numrows($sql) == 0) {
	echo 'brak wyników';
    } else {
	$wierszy = $db->numrows($sql);
	for ($i=0;$i<$wierszy;$i++) {
	$dane = $db->fetcharray($sql);
	echo $dane['wid'].$dane['czas'].$dane['podpis'];}
  }
  }
[PHP] pobierz, plaintext 

ten fragment pokazuje mi 17 razy jeden wiersz. Znalezionych wierszy jest 17 i to jest prawidłowy wynik, co jest nie tak że tylko jeden wiersz w pętle wrzuca? db

[nospor]

Do funkcji select() dopisz taki kod:

[PHP] pobierz, plaintext 
 public function select($sql) {
echo 'ZNOWY WYKONUJĘ ZAPYTANIE OD POCZĄTKU';
//....dalsza cześć funkcji
[PHP] pobierz, plaintext 

Zwróć uwagę co dostaniesz na ekranie po wykonaniu tego kodu a może sam się domyślisz co jest nie tak

Powód edycji: [nospor]:

[@rtur]

za każdym razem wykonuje się funkcja select w pętli for();

co mam zrobić aby się nie wykonywała?

podejrzewam jakiś błąd w składni funkcji fetcharray.... tylko brak mi pomysłu jak ją zmienić żeby działała

[nospor]

select() zwraca ci RESOURCE. fetcharray oraz numrows mają korzystac z tego RESOURCE a nie za każdym razem wywoływać select()

Powód edycji: [nospor]:

[@rtur]

zmieniłem na takie coś:

[PHP] pobierz, plaintext 
public function numrows($sql){
 
     if ($this->connection) {
      mysql_set_charset('utf8', $this->connection);
      if (isset($sql) && $sql != '') { 
 if ($results = $this->select($sql)){
  $rows = mysql_num_rows($results);
  return $rows;
  }
  else {
          $this->error = mysql_error();
          return false; 
        }
  } else {
      $this->error = 'Brak połączenia z bazą danych';
      return false; 
    }
 }
  } // eof numrows()
 
  public function fetcharray($sql){
 
     if ($this->connection) {
      mysql_set_charset('utf8', $this->connection);
      if (isset($sql) && $sql != '') { 
 if ($results = $this->select($sql)){
  $rows = mysql_fetch_array($results);
  return $rows;
  }
  else {
          $this->error = mysql_error();
          return false; 
        }
  } else {
      $this->error = 'Brak połączenia z bazą danych';
      return false; 
    }
 }
  } // eof fetcharray()
[PHP] pobierz, plaintext 

i dalej to samo 17 razy jeden wiersz.

błąd jest w kodzie samej funkcji numrows i fetcharray czy gdzie indziej?

[nospor]

Przecież ci napisałem w poprzednim poście, że fetcharray i numrows nie mają prawa korzystać z select(). A ty co? A ty nadal w nich korzystasz z select()

[@rtur]

[PHP] pobierz, plaintext 
public function numrows($sql){
 
    if ($this->connection) {
    mysql_set_charset('utf8', $this->connection);
    if (isset($sql) && $sql != '') {
    if ($rows = mysql_fetch_array($sql)){
    return $rows;
    }
    else {
    $this->error = mysql_error();
    return false;
    }
    } else {
    $this->error = 'Brak połączenia z bazą danych';
    return false;
    }
    }
    } // eof numrows()
 
public function fetcharray($sql){
 
    if ($this->connection) {
    mysql_set_charset('utf8', $this->connection);
    if (isset($sql) && $sql != '') {
    if ($rows = mysql_fetch_array($sql)){
    return $rows;
    }
    else {
    $this->error = mysql_error();
    return false;
    }
    } else {
    $this->error = 'Brak połączenia z bazą danych';
    return false;
    }
    }
    } // eof fetcharray()
[PHP] pobierz, plaintext 

coś takiego?

a później w kodzie:

[PHP] pobierz, plaintext 
$sql = "SELECT wid,czas,podpis FROM tabela WHERE status = '1' ORDER BY 'czas' DESC;";
    if ($db->select($sql)){
    if ($db->numrows($sql) == 0) {
    echo 'brak wyników';
    } else {
    $wierszy = $db->numrows($sql);
    for ($i=0;$i<$wierszy;$i++) {
    $dane = $db->fetcharray($sql);
    echo $dane['wid'].$dane['czas'].$dane['podpis'];}
    }
    }
[PHP] pobierz, plaintext 

?
pisane na szybko, nie sprawdzane, bo zaraz zmykam do pracy na 8:00 (IMG:style_emoticons/default/smile.gif)

[nospor]

Ty nie pisz na szybko tylko mysl.... mówiłem ci że select() coś zwraca. Ty to coś masz przekazać do numrows i fetcharray i na tym czymś masz operować a nie na $sql.

[@rtur]

dzięki chyba się udało, i wygląda na to że działa, teraz te 2 funkcje wyglądają tak:

[PHP] pobierz, plaintext 
  public function numrows($sql){
      if ($this->connection) {
      mysql_set_charset('utf8', $this->connection);
      if (isset($sql) && $sql != '') {
        if($result = mysql_num_rows($sql)) {
          return $result;
        } else {
          $this->error = mysql_error();
          return false; 
        }
      } else {
        $this->error = 'Błąd zapytania SQL';
        return false;
      }
    } else {
      $this->error = 'Brak połączenia z bazą danych';
      return false; 
    }
  } // eof numrows()
  public function fetcharray($sql){
      if ($this->connection) {
      mysql_set_charset('utf8', $this->connection);
      if (isset($sql) && $sql != '') {
        if($result = mysql_fetch_array($sql)) {
          return $result;
        } else {
          $this->error = mysql_error();
          return false; 
        }
      } else {
        $this->error = 'Błąd zapytania SQL';
        return false;
      }
    } else {
      $this->error = 'Brak połączenia z bazą danych';
      return false; 
    }
  } // eof fetcharray()
[PHP] pobierz, plaintext 

a kod którym pobieram dane :

[PHP] pobierz, plaintext 
$sql = "SELECT wid, czas, tresc, podpis FROM tabela WHERE status = '1' ORDER BY 'czas' DESC;";
    if ($results = $db->select($sql))
       if ($db->numrows($results) == 0) {
	echo 'nie ma żadnych danych';
    } else {
	$wierszy = $db->numrows($results);
	for ($i=0;$i<$wierszy;$i++) {
    if($dane = $db->fetcharray($results)){
	echo $dane['wid'].$dane['czas'].$dane['podpis'];}
  }}
[PHP] pobierz, plaintext 

jeszcze tylko nie działa mi jedna funkcja w tym moim programie, ale to w nowym poście napisze (IMG:style_emoticons/default/smile.gif)

@nospor, z całego serca dziękuje za okazaną pomoc, po raz kolejny przekonałem się że dobry z Ciebie nauczyciel, nie poda rozwiązania na tacy a zmusza do myślenia, i oto chodzi.

[mat-bi]

A to nie lepiej zamykać połączenie w destruktorze?

[@rtur]

tego jeszcze nie ogarnąłem (IMG:style_emoticons/default/smile.gif)