mysql_real_escape_string - fejl

har lavet det sådan her nu..

Index:
$name = $sql->Q($_POST["username"]);
$pass = 		md5($_POST["password"]);

$login = $sql->SendQuery("SELECT * FROM mobler_users WHERE (user='$name') AND (pass='$pass')");

Class:
function Q($str){
if ($this->Connect()) {
			$string = return mysql_real_escape_string($st);
			$this->Disconnect();
		}

}

takker for hjælpen =) smid et svar så jeg kan få lukket tråden

Det er da absolut super dumt at åbne og lukke forbindelsen til databasen hver gang du laver en query.
Hvad hvis nu du har 20 queries (overdrevet, men hvad nu hvis). Så har du åbnet og lukket forbindelsen 20 gange på et pageload. Det kan da kaldes for resourcespild.

Åben forbindelsen een gang og luk den een gang. Jeg kan garantere dig at du vil se en forskel i tiden det ta'r at udføre et pageload.

Det er da absolut super dumt at åbne og lukke forbindelsen til databasen hver gang du laver en query.
Hvad hvis nu du har 20 queries (overdrevet, men hvad nu hvis). Så har du åbnet og lukket forbindelsen 20 gange på et pageload. Det kan da kaldes for resourcespild.

Åben forbindelsen een gang og luk den een gang. Jeg kan garantere dig at du vil se en forskel i tiden det ta'r at udføre et pageload.

Helt enig. Hold forbindelsen åben gennem hele scriptet.
Men 20 queries er ikke overdrevet. Der kunne godt blive brug for at udføre et par queries for hvert resultat i et andet query, og så løber det pludselig op.

oki, kan bare connecte i __construct så, har bare gjort det til en vane at lukke efter brug :/

nu er det sådan her:

	public function __construct(){
		$this->Connect();
	}

	public function __destruct(){
		$this->Disconnect();
	}

det må være fint =)