Problema con if para verificar que la consulta no este vacia [Resuelto]

[jereje]

Miren les muestro mi codigo:

Código PHP:

  include('../includes/conf.php');
if (isset($_GET['id']))
{
$sql = "SELECT * FROM comentarios WHERE noticia ='".$id."'";
$result = mysql_query($sql);
$row = mysql_fetch_array($result); 
if(count($row)>0)
{
echo '<TABLE class="central" >
<TR>  
   <TH widht="30%">Titulo</TH>
   <TH widht="30%">Contenido</TH>
   <TH widht="30%">Eliminar</TH> 
</TR>';
include ('../includes/funciones.php');
$id = verificarNumero($_GET['id']);
 
while($tablas = mysql_fetch_array($result)) 
{
echo '<TR>
    <TD widht="30%">'.$tablas[titulo].'</TD>
    <TD widht="30%">'.$tablas[texto].'</TD>
    <TD widht="30%"><a href=?del_id='.$tablas[id].'&not_id='.$tablas[noticia].'><img src="img/del.png"><A></TD>
    </TR>';
 
}
echo '</table><br>';
}
else {
    echo '<div class="actual">Esta noticia no tiene ningun comentario</div><br>';
}
}
if (isset($_GET['del_id']) && isset($_GET['not_id']))
{
$id = $_GET['del_id'];
$sqll="DELETE FROM comentarios WHERE id =".$id;
if (mysql_query($sqll, $conectar)){
    echo 'jorge';
}
else{
    echo 'juan';
}
$not_id=$_GET['not_id'];
echo "<meta http-equiv='refresh' content='0;URL=man_coment.php?id=".$not_id."'>"; 
} 
   

Mi problema esta en el if que dice:

Código PHP:

  if(count($row)>0) 
   

El problema es que no funciona ese if, deja pasar siempre a la primer clausula y encima no hace el while :S
Alguien se da cuenta de algun error o por que no anda?
Saludos y gracias

[Ronruby]

En tu consulta, no es necesario ponerle las comillas a "id" si es un entero.

Y usa, mysql_num_rows($resultado) > 0 en tu if.

[jereje]

Iniciado por Ronruby

En tu consulta, no es necesario ponerle las comillas a "id" si es un entero.

Y usa, mysql_num_rows($resultado) > 0 en tu if.

Lo puse como me dijiste:

Código PHP:

  include('../includes/conf.php');
if (isset($_GET['id']))
{
$sql = "SELECT * FROM comentarios WHERE noticia =".$id;
$result = mysql_query($sql);
if(mysql_num_rows($result) > 0 )
{
echo '<TABLE class="central" >
<TR>  
   <TH widht="30%">Titulo</TH>
   <TH widht="30%">Contenido</TH>
   <TH widht="30%">Eliminar</TH> 
</TR>';
include ('../includes/funciones.php');
$id = verificarNumero($_GET['id']);

while($tablas = mysql_fetch_array($result)) 
{
echo '<TR>
    <TD widht="30%">'.$tablas[titulo].'</TD>
    <TD widht="30%">'.$tablas[texto].'</TD>
    <TD widht="30%"><a href=?del_id='.$tablas[id].'&not_id='.$tablas[noticia].'><img src="img/del.png"><A></TD>
    </TR>';

}
echo '</table><br>';
}
else {
    echo '<div class="actual">Esta noticia no tiene ningun comentario</div><br>';
}
}
if (isset($_GET['del_id']) && isset($_GET['not_id']))
{
$id = $_GET['del_id'];
$sqll="DELETE FROM comentarios WHERE id =".$id;
if (mysql_query($sqll, $conectar)){
    echo 'jorge';
}
else{
    echo 'juan';
}
$not_id=$_GET['not_id'];
echo "<meta http-equiv='refresh' content='0;URL=man_coment.php?id=".$not_id."'>"; 
} 
   

Y me tira esto:

Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource in C:\xampp\htdocs\blog\admin\man_coment.php on line 21
Esta noticia no tiene ningun comentario

Ya habia probado como me dijiste y me pasaba eso, y estuve leyendo y decia que le ponga comillas al id, con las comillas en el id y el mysql_num_rows() , me tira siempre la segunda clausula, sin errores, pero siempre la segunda.

Espero que me puedan ayudar, muchas gracias, saludos

EDIT: Ya lo arregle, lo que pasaba es que en la consulta la variable $id no estaba declarada, y nose por que php no la convertia automaticamente, se soluciono con agregar $id=$_GET['id'];
quedo asi:

Código PHP:

  include('../includes/conf.php');
if (isset($_GET['id']))
{
$id=$_GET['id'];
$sql = "SELECT * FROM comentarios WHERE noticia =".$id;
$result = mysql_query($sql);
if(mysql_num_rows($result) > 0 )
{
echo '<TABLE class="central" >
<TR>  
   <TH widht="30%">Titulo</TH>
   <TH widht="30%">Contenido</TH>
   <TH widht="30%">Eliminar</TH> 
</TR>';
include ('../includes/funciones.php');
$id = verificarNumero($_GET['id']);

while($tablas = mysql_fetch_array($result)) 
{
echo '<TR>
    <TD widht="30%">'.$tablas[titulo].'</TD>
    <TD widht="30%">'.$tablas[texto].'</TD>
    <TD widht="30%"><a href=?del_id='.$tablas[id].'&not_id='.$tablas[noticia].'><img src="img/del.png"><A></TD>
    </TR>';

}
echo '</table><br>';
}
else {
    echo '<div class="actual">Esta noticia no tiene ningun comentario</div><br>';
}
}
if (isset($_GET['del_id']) && isset($_GET['not_id']))
{
$id = $_GET['del_id'];
$sqll="DELETE FROM comentarios WHERE id =".$id;
if (mysql_query($sqll, $conectar)){
    echo 'jorge';
}
else{
    echo 'juan';
}
$not_id=$_GET['not_id'];
echo "<meta http-equiv='refresh' content='0;URL=man_coment.php?id=".$not_id."'>"; 
} 
   

Muchas gracias por la ayuda

[Triby]

php no la convertia automaticamente, se soluciono con agregar $id=$_GET['id'];

El comportamiento de PHP donde automaticamente crea variables provenientes de GET, POST o COOKIE aplica exclusivamente cuando tienes register_globals en on, lo que representa serios problemas de seguridad. Por default en versiones nuevas de PHP register_globals esta en off y es por eso que debes declarar la variable tal como lo hiciste.

[ZICCO]

Bueno... ahora tienes un hueco de seguridad en tu variable $id... te podrían hacer mucho daño en tu base de datos solo por ello. Te recomiendo hacer una validación de tus datos y utilizar mysql_real_escape() (o algo así, no lo recuerdo muy bien) para evitar problemas en un futuro.

Saludos.

[jereje]

Iniciado por ZICCO

Bueno... ahora tienes un hueco de seguridad en tu variable $id... te podrían hacer mucho daño en tu base de datos solo por ello. Te recomiendo hacer una validación de tus datos y utilizar mysql_real_escape() (o algo así, no lo recuerdo muy bien) para evitar problemas en un futuro.

Saludos.

Ya le agregue lo siguiente:

Código PHP:

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

donde:

Código PHP:

  function verificarNumero($numero){
    $numero=addslashes($numero);
    $numero=mysql_real_escape_string($numero);
    $numero= intval($numero);
    return $numero;
} 
   

Saludos

[Ronruby]

No es necesario el uso de addslashes si vas a usar mysql_real_escape_string.

[Triby]

Si esperas un numero entero no es necesaria toda la funcion que hiciste, basta con intval() para obtener lo que necesitas.... pero si $id es menor que 1?

[jereje]

Iniciado por Ronruby

No es necesario el uso de addslashes si vas a usar mysql_real_escape_string.

Muchas gracias
Quedo asi:

Código PHP:

  function verificarPalabra($palabra){
    $palabra=mysql_real_escape_string($palabra);
    return $palabra;
}
function verificarNumero($numero){
    $numero=mysql_real_escape_string($numero);
    $numero= intval($numero);
    return $numero;
} 
   

[jereje]

Iniciado por Triby

Si esperas un numero entero no es necesaria toda la funcion que hiciste, basta con intval() para obtener lo que necesitas.... pero si $id es menor que 1?

Entonces con:

Código PHP:

  function verificarNumero($numero){
    $numero= intval($numero);
    return $numero;
} 
   

ya evito cualquier tipo de injeccion sql?

[Triby]

ya evito cualquier tipo de injeccion sql?

Si, en un valor entero solo hay numeros y tal vez el signo menos.

Código php:

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

[jereje]

Iniciado por Triby

Si, en un valor entero solo hay numeros y tal vez el signo menos.

Código php:

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

Muchas gracias a todos por la ayuda, si tienen algun otro consejo soy todo oidos.
Saludos

[Ronruby]

Iniciado por jereje

Muchas gracias a todos por la ayuda, si tienen algun otro consejo soy todo oidos.
Saludos

Haz un if y verifica que $id sea mayor que 0. Creo que eso es todo.

[jereje]

Iniciado por Ronruby

Haz un if y verifica que $id sea mayor que 0. Creo que eso es todo.

Muchas gracias