[PHP] pobierz, plaintext 
class DB {
	private $db;
	function __construct($rHost, $rUser, $rPassword, $rDB)
	{
		$this->db = @mysql_connect($rHost, $rUser, $rPassword);
		mysql_select_db($rDB, $this->db);
	}
	public function SQL($sql)
	{
		if($result = mysql_query($sql))
		{
			return $result;
		}else{
			return mysql_error();
		}
	}
	public function GetRows($sql)
	{
		while($row = mysql_fetch_array($this->SQL($sql)))
		{
			$arr[] = $row;
		}
		return $arr;
	}
}
[PHP] pobierz, plaintext 

Mam tutaj prostą klasę do łącznia się z bazą.
Dopiero od wczoraj zaczełem się bawić z klasami. Próbuje cały czas ale coś mi nie wychodzi.
Mam problem z funkcją GetRows nie wiem czemu ale strona się tak długo ładuje aż do overloadu, przez tą pętlę while.
Jak by ktoś mi mógł pomóc i wskazać gdzie jest błąd był bym bardzo wdzięczny.

Może wyciągnij $this->SQL($sql) przed pętlę?

Włącz raportowanie błędów, przypisujesz do zmiennej $this->db wskaźnik połączenia jednak z niego nie korzystasz? Sprawdź czy połączenie jest ustanowione.

Na moje oko to: mysql_fetch_array($this->SQL($sql)) nie zwraca tego co powinno - sprawdź typeof

cycofiasz dzięki wielkie działa

builq nie korzystam z $db ponieważ przy wywoływaniu klasy łącze się z bazą...

Ale wiesz o tym że mysql_query przyjmuje 2 parametry? 2 co prawda opcjonalny ale skoro już zapisujesz warto go wykorzystywać. Poza tym czy jesteś połączony do mysql sprawdzałbym przy każdym wywołaniu metody SQL. Nie widzę abyś gdziekolwiek to sprawdzał.

Ale to tylko takie tam ...

Masz rację Lepiej zablokować wszystkie zapytania...

[PHP] pobierz, plaintext 
<?
class DB {
	private $db;
	private $db_error;
	function __construct($rHost, $rUser, $rPassword, $rDB)
	{
		if(!$this->db = @mysql_connect($rHost, $rUser, $rPassword))
		{
			$this->db_error = TRUE;
		}		
		if(!mysql_select_db($rDB, $this->db))
		{
			$this->db_error = TRUE;
		}
	}
	public function SQL($sql)
	{
		if(!isset($this->db_error))
		{
			if($result = mysql_query($sql))
			{
				return $result;
			}else{
				return mysql_error();
			}
		}
	}
	public function SqlAssocRows($sql)
	{
		if(!isset($this->db_error))
		{
			$query = $this->SQL($sql);
			while($row = mysql_fetch_assoc($query))
			{
				$rows[] = $row;
			}
			return $rows;
		}
	}
	public function SqlAssocRow($sql)
	{
		if(!isset($this->db_error))
		{
			$query = $this->SQL($sql);
			return mysql_fetch_assoc($query);	
		}
	}
}
 
 
 
 
?>
[PHP] pobierz, plaintext 

Poprawiony kod Tylko jeszcze dorobie obsługę langów i będzie wywalało błąd

A mi pozostało jeszcze wskazać tylko jedno :

[PHP] pobierz, plaintext 
if($result = mysql_query($sql[b],$this->db[/b]))
[PHP] pobierz, plaintext 

Mówiłem iż $this->db zawiera wskaźnik dla połączenia, de facto nie powinien zgubić go ale gdybyś chciał połączenia do 2 baz to zawsze lepiej to wstawić.