error en query

[charlyta]

hola a todos. Tengo un error en esta query por el uso de

Código HTML:

mysql_num_rows

. Cómo podría optimizarla? gracias por las ugerencias

Código PHP:

  if ($handle = opendir(site_root.uploaded_images_folder)) {
    while (false !== ($file = readdir($handle))) { 
        if ($file != "." && $file != "..") {
            if (is_dir(site_root.uploaded_images_folder.$file)) {
                if (!mysql_num_rows(mysql_query("select 'id' from 'galleries' where 'name' = '".$file."'"))) {
                    mysql_query("insert into 'galleries' ('name','description','order') values ('".$file."','','999')");
                    if (!file_exists(site_root.thumbnail_images_folder.$file."/")) { mkdir(site_root.thumbnail_images_folder.$file."/"); }
                }
            }    
        }
 
    }
} 
   

[PepeF]

Es mejor que utilizes mysqli es mas seguro.

[p414]

y si las manejas en variables????

Código PHP:

$query = mysql_query("select 'id' from 'galleries' where 'name' = '".$file."'");
$num_rows = mysql_num_rows($query);
.
.
.
 

y luego haces las operaciones????

[charlyta]

Pues creo que va a ser lo mejor hacerlo así!!! Gracias!!

PD: probaré también con mysqli

[p414]

PepeF podrias explicarnos porqué es más seguro con mysqli????


saludos!!!!

[PepeF]

Solo tienes que modificar
mysql_query por mysqli_query
mysql_connect por mysqli_connect
mysql_select_db por mysqli_select_db
mysql_fetch_array por mysqli_fetch_array

Y siempre detras de SELECT UPDATE DELETE INSERT INTO tienes que poner $conexion, o como se llama tu variable quedara asin

mysqli_query($conexion, "INSERT INTO .........");
mysqli_query($conexion, "SELECT .........");
mysqli_query($conexion, "DELETE .........");
mysqli_query($conexion, "UPDATE .........");

Es mas seguro para los ataques de inyeccion sql y mas. y encima los de mysql dicen que es preferible usar mysqli que mysql y encima mysqli es mas nuevo que mysqli. y entonces abran puesto mas seguro (¿No cres?).

[Carloosolrac]

Creo que son las comillas en tu query. Estás usando ' para los campos/tablas y deberías usar `

[PepeF]

Prueba esto:

Código MYSQL:

if ($handle = opendir(site_root.uploaded_images_folder)) { 
    while (false !== ($file = readdir($handle))) {  
        if ($file != "." && $file != "..") { 
            if (is_dir(site_root.uploaded_images_folder.$file)) { 
                if (!mysql_num_rows(mysql_query("select  id from galleries where name = '$file.'"))) { 
                    mysql_query("insert into 'galleries' ('name','description','order') values ('".$file."','','999')"); 
                    if (!file_exists(site_root.thumbnail_images_folder.$file."/")) { mkdir(site_root.thumbnail_images_folder.$file."/"); } 
                } 
            }     
        } 
 
    } 
}
 

[p414]

pues si es más nuevo, estás de acuerdo que también debe de estar en desarrollo todavía, hay ciertos hoyos que quizas aún no los hayan cerrado pero habrá que leer sobre mysqli para ver que cosas ofrece con respecto a mysql... y es grato encontrarte con estas cosas


saludos

[PepeF]

en desarrollo creo que ya no esta. ya an desarrollado todo

[GatorV]

@p414: http://www.forosdelweb.com/f18/anunc...ecada-1008145/

[p414]

muchas gracias @GatorV... realmente sabía de ese cambio, pero para ser sinceros no lo esperaba tan pronto :S.... pero gracias por dar este tipo de información