problema con mysql_num_rows()

[rizz1]

Salve, ho un db mysql collegato ad un localhost(apache)....
Ho fatto delle funzioni in php per la connessione col db....
Ho un problema con il conteggio delle righe....
Le funzioni e' così strutturata:

 public function num_righe($risultato)  {    
if($this->attiva)    {      
$numrows = mysql_num_rows($risultato);      
  return $numrows;    }    
else      
  return false;  }

public function estrai($risultato) {  
 if($this->attiva)   {     
$r = mysql_fetch_object($risultato);     
return $r;    }    
else      
return false; }

public function query($sql)     {       
if($this->attiva)       {        
 $sql = mysql_query($sql) or die (mysql_error());           return $sql;       
} 	  
else 	    
return false;     }

e la richiamo cosi:

$query = $data->query("SELECT ..."); 
$res = $data->estrai($query); 
$n_righe = $data->num_righe($res);

ma quando eseguo mi da il seguente errore:

Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource in C:\Program Files ....
e quindi la variabile $n_righe nn sara' opportunamente settata...
Qualcuno sa il perchè?
Grazie

[leon-kennedy]

Ciao scusa,forse sbaglio io,ma quando metti "else" non dovresti aprire una parentesi graffa?

public function num_righe($risultato)  {    
if($this->attiva)  {      
$numrows = mysql_num_rows($risultato);      
return $numrows; 
}else{      
return false;
}
} 

[bstefano79]

Originariamente inviato da leon-kennedy
Ciao scusa,forse sbaglio io,ma quando metti "else" non dovresti aprire una parentesi graffa?

public function num_righe($risultato)  {    
if($this->attiva)  {      
$numrows = mysql_num_rows($risultato);      
return $numrows; 
}else{      
return false;
}
} 

se l'else è composta da una solo istruzione non è necessario anche se sarebbe meglio metterle per la leggibilità del codice

[bstefano79]

cambia

$n_righe = $data->num_righe($res); 

con

$n_righe = $data->num_righe($query); 

[rizz1]

Ok... ora funziona
grazie