Malfunzionamento su fetch_assoc

[cyberfido]

ciao a tutti,
stavo facendo una classe che estende mysqli
e nell'implementazione del metodo che dovrebbe fare da wrapper a mysqli_result_fetch_assoc()
ho un comportamento sbagliato che non riesco a correggere.
riducendo all'osso il codice ecco quello che ho scritto.

<?

class db extends mysqli
{
	public $result;
	public $db_row;
	public $sql;
	
[...]
	public function db_query()
	{
		if($this->sql == "")
			die("Stringa SQL vuota!");
		$this->result = parent::query(parent::real_escape_string($this->sql));
		if($this->result === FALSE)
		{
			die('Errore nella query '.msqli_error());
			return false;
		}
		else
			return true;
	}
	
	public function db_fetch()
	{

		$this->db_row = $this->result->fetch_assoc();
	}
	
}
?>

se lo utilizzo in maniera ortodossa e cioè così, non funziona

$ldb = new db;
$ldb->sql = "SELECT * FROM usrusers;";
$ldb->db_query();
while($ldb->db_fetch())
	echo "

".$ldb->db_row['usrlogin']." - ".$ldb->db_row['usrpwd']." - ".$ldb->db_row['usrlabel']."</p>";

se lo utilizzo così e cioè senza usare il mio metodo di db_fetch() invece funziona

$ldb = new db;
$ldb->sql = "SELECT * FROM usrusers;";
$ldb->db_query();
while($row = $ldb->result->fetch_assoc())
echo "

".$row['usrlogin']." - ".$row['usrpwd']." - ".$row['usrlabel']."</p>";

sulla pagina quando uso la versione ortodossa non mi dà nessun errore.
semplicemente non stampa niente
cosa sbaglio?

grazie mille in anticipo a tutti
cyb

[Grino]

Guardando un po' il codice mi vien da chiedere:

parent::real_escape_string($this->sql)

Cos'è questo? Sei sicuro vada utilizzato così?

die('Errore nella query '.msqli_error());
return false;

a che serve la return se invochi la die?

while($ldb->db_fetch())

Come fa a funzionare questa roba se db_fetch non ritorna nulla? Quando finisce/ripete il ciclo?

Ti dirò di più, dato che non ritorna nulla la condizione mancante forse è interpretata come false ed essendo ripetuto il ciclo while su condizione true, il ciclo non è eseguito neanche una volta ecco perchè non ti appare nulla.

[luca200]

Il tuo metodo db_fetch() non restituisce nulla...

[cyberfido]

Originariamente inviato da Grino
Guardando un po' il codice mi vien da chiedere:

parent::real_escape_string($this->sql)

Cos'è questo? Sei sicuro vada utilizzato così?

richiama il metodo real_escape_string sulla stringa sql della classe ereditata mysqli.
vengo da .net, lì si adopererebbe così, ma ancora non l'ho testato, per cui no, non sono assolutamente sicuro.

die('Errore nella query '.msqli_error());
return false;

a che serve la return se invochi la die?

a niente, è un refuso. grazie!

while($ldb->db_fetch())

Come fa a funzionare questa roba se db_fetch non ritorna nulla? Quando finisce/ripete il ciclo?

Ti dirò di più, dato che non ritorna nulla la condizione mancante forse è interpretata come false ed essendo ripetuto il ciclo while su condizione true, il ciclo non è eseguito neanche una volta ecco perchè non ti appare nulla.

sì sì è proprio lì l'errore. ora l'ho fatto ritornare e funziona tutto.
scusate mi ero incaponito e non vedevo una cosa semplice.

grazie mille!

[cyberfido]

Originariamente inviato da luca200
Il tuo metodo db_fetch() non restituisce nulla...

sì è vero è proprio quello l'errore.
ho corretto e ora funziona.
grazie anche a te!