Klasa bazy danych, Problem z klasą do obsługi bazy danych Opcje

[Diwi]

A więc mam sobie taką oto klasę do obsługi bazy danych:

[PHP] pobierz, plaintext 
<?php
 
class db {
 
var $connect, $sd;
 
 
				function db() {
 
				$this->connect = mysql_connect('localhost', 'root', 'gameforce') OR DIE (&#092;"Nie mogłem połączyć się z MySQL-em\");
				$this->sd = mysql_select_db('**') OR DIE ('Nie mogłem połączyć się z bazą danych: **');
 
				 }
 
				function query($z) {
 
				@mysql_query(&#092;"$z\", $this->connect) OR DIE ('Błąd zapytania SQL: <br /><b>'.mysql_errno().' : '.mysql_error().'</b>');
 
				} // function pytanie
 
				function num_rows($query) {
 
				@mysql_num_rows($query) OR DIE ('Błąd bazy danych: '.mysql_error());
 
				 }
				 
				function fetch($pytanie) {
 
				@mysql_fetch_assoc($pytanie, $this->connect) OR DIE ('Błąd bazy danych: '.mysql_error());
 
				 }
 
 } // koniec klasy
 
?>
[PHP] pobierz, plaintext 

Chyba wiadomo jak to działa: mamy konstruktor który przy wywołaniu odrazu otwiera połączenie z bazą danych + pare funkcji

Teraz mam klase do obsługi newsów "news" i chce wyświetlić newsy.

[PHP] pobierz, plaintext 
<?php
 
if (eregi('news.php', $_SERVER['PHP_SELF'])) {
exit('Hacking Attempt !!');
 }
 
include_once('./config/db.lib.php');
include_once('./templates/smarty.class.php');
class news {
 
var $db, $tpl;
				function news() {
 
				$this->db = new db;
				$this->tpl = new smarty;
				$this->tpl->template_dir = 'templates/tpl';
				$this->tpl->compile_dir = 'templates/tpl_c';
 
				 }
 
				function wyswietl_newsy() {
 
				$result = $this->db->query(&#092;"SELECT * FROM `news` ORDER BY `add_date` DESC\");
 
				while($row = $this->db->fetch($result)) {
				 // tutaj sobie robimy różne rzeczy na SMARTY
				}
 
				}
}
 
$news = new news;
$news->wyswietl_newsy();
?>
[PHP] pobierz, plaintext 

Wydaje mi się że wszystko jest dobrze . Lecz dostaje komunikat:

Kod

Błąd bazy danych:

Jak widać w klasie DB jest ustawione raportowanie błędów lecz po "Błąd bazy danych:" powinien się wyświetlić komunikat

Po usunięciu warunku OR DIE a także znaku @ przed funkcją dostaje taki komunikat:

Kod

Warning: mysql_fetch_assoc(): supplied argument is not a valid MySQL result resource in C:\WebServ\httpd\aa\config\db.lib.php on line 31

Zaznaczam że zapytanie SQL które wykonuje jest poprawne

Co tu się dzieje

Pozdrawiam

[nospor]

funkcja query() nie zwraca ci nic. a ty mimo wszystko z niej cos pobierasz, a potem to cos (czyli nic), przekazuje do fetch. A w fetch masz mysql_fetch_assoc, której argumentem nie może być nic. Kapisz?

Funkcja query powinna zwracac wynik funkcji mysql_query

[SHiP]

Ja bym to zrobił tak

[PHP] pobierz, plaintext 
<?php
 
  function sql_query($query) 
  { 
    if(!$this -> result = @mysql_query($query)) $this -> errors[]=&#092;"Błędne zapytanie\";
  }
 
?>
[PHP] pobierz, plaintext 

I wtedy funkcje do wyciągania danych z bazy..

[PHP] pobierz, plaintext 
<?php
 
  function sql_fetch_array($method) 
  { 
   if($method>3 || $method<1) return 0; 
    /*
    1=ASSOC 
    2=NUM 
    3=BOTH (ASSOC + NUM) 
    */ 
    return @mysql_fetch_array($this -> result,$method);
  }
 
?>
[PHP] pobierz, plaintext 

Ten post edytował SHiP 1.04.2005, 16:03:13

[Diwi]

Acha czyli musze dać return ?

Myśle o czymś takim:

[PHP] pobierz, plaintext 
<?php
 
// ...
function query($z) {
 
return @mysql_query(&#092;"$z\", $this->connect) OR DIE ('Błąd zapytania SQL: <br /><b>'.mysql_errno().' : '.mysql_error().'</b>');
 
				} // function query
 
// ...
?>
[PHP] pobierz, plaintext 

Coś takiego powinno działać

Pozdrawiam

Ten post edytował Diwi 1.04.2005, 15:59:40

[SHiP]

Musisz po prostu rezultat funkcji mysql_query przypisać do jakiejść zmiennej - u mnie $this -> result

Tylko wtedy wyświetlanie będzie lekko inne

[PHP] pobierz, plaintext 
<?
$db -> sql_query(&#092;"SELECT * FROM `news` ORDER BY `add_date` DESC\");
while($row = $db -> sql_fetch_array(1))
{
coś tam
}
?>
[PHP] pobierz, plaintext 

Poza tym nie ruzumiem poco tworzyć obiekt jako zmienną klasy. Ja bym obiekt $db stworzył poza klasą news. I udostępnił go globalnie w wybranych funkcjach.

Ten post edytował SHiP 1.04.2005, 16:12:00

[dasko]

@SHiP: Głupi jest pomysł z przetrzymywaniem wyniku zapytania w $this->result. Takie rozwiązanie pozwala na istnienie tylko jednego wyniku w jednym czasie.

Wg mnie, najlepiej jest stworzyć klasę Result, która będzie zawierała wynik zapytania i metody operujące na wyniku SQL(fetchArray(), fetchObject, numRows() itd.) i właśnie instancję takiej klasy zwracać z metody db::sql_query()

[Ociu]

Ja to zrobiłem tak:

[PHP] pobierz, plaintext 
<?php
# var $sql_result; etc.
function query($sql)
						{
										$this -> sql_result = '';
 
										$start = $this -> _get_time();
								if ($this -> connected && $this -> selected) # sprawdza czy jest polaczenie
								{
 
										$this -> sql_result = mysql_query($sql, $this -> connected);
 
										if(!$this -> sql_result)
										{
												$this -> err('query');
												return false;
										} else {
												$this -> execute++;  #liczy zapytania
												return true;
										}
								} else {
										$this -> err('notconnect');
								}
						}
?>
[PHP] pobierz, plaintext 

fetch_row:

[PHP] pobierz, plaintext 
<?php
function fetch_row($fetch_row = '')
						{
								$fetch_row = ((empty($fetch_row)) ? $this -> sql_result : $fetch_row);
 
								$r = mysql_fetch_array($fetch_row);
 
								if($r)
								{
												return $r;	   
								} else {
												 return false;
 
								}
						}
?>
[PHP] pobierz, plaintext