Es correcto?

[xbx]

Iniciado por nicolaspar

No estoy de acuerdo con el "restar 0", no me parece lo mejor. Tampoco dejar una variable (como lo es $nombre) sin setear, ya que si no entra en la condición no existirá y además que es costoso el mysql_num_rows. Otra...usar un limit comparando un PK?...en si, lo haría así:

Código PHP:

  $id = empty($_GET['id']) ? 0 : intval($_GET['id']);
 
$sql = "
SELECT 
 nombre, chiste 
FROM 
 chistes 
WHERE 
 id = '{$id}' 
";
$r = mysql_query( $sql );  
 
if( $rs = mysql_fetch_assoc($r) ){
 $nombre = $rs['nombre'];  
 $chiste = $rs['chiste']; 
}else{
 $nombre = '';
 $chiste = '';
} 
   

Hay otras variantes, pero esta no está mal.

la otra variante es:

Código PHP:

  $id = (int)$_GET['id'];