Actualizar muchisimos campos en MySQL usando php

[thewayox]

Hola, gracias de antemano a quienes me puedan ayudar.

Tengo un formulario con 47 campos que leo de una tabla en MySQL, en este formulario edito estos campos.

La pregunta es, existe una forma de actualizarlos que no sea la clasica...

Código PHP:

  $consulta="update tabla set campo1='$campo1', campo2='$campo2', campo3='$campo3',...., campo47='$campo47' where id='$id'";  
   

Porque tendría que declarar todas esas variables, hacer ese chorizo gigante y de verdad que me parece que hay una forma mas facil, la cual no conozco aún.

Gracias por su tiempo y respuesta.

[pateketrueke]

Para temas de sintaxis SQL te sugiero consultar una referencia de SQL, aunque dudo que exista otra forma menos "clásica".

Ahora, por el tema de PHP definitivamente debes crear las variables sí o sí, que claro podrías crear una función a la cual le pases un array y construya la cadena de SETs por ti.

Pero sólo son ideas, usa tu ingenio.

[thewayox]

Uso esto:

Código PHP:

  $result=mysql_query("select * from tabla where id='$id'",$link); 
while ($rows = mysql_fetch_array($result, MYSQL_ASSOC)) { 
    foreach ($rows as $key => $value) { 
        $$key = $value; 
    } 
}  
   

Con estas 6 lineas me ahorre el declarar todas las variables, o sea, un trabajon, ademas de que el mismo codigo me servirá para otros formularios de igual (47) o mas campos... Lo unico que tendría que cambiar es la consulta.

Creo que debe haber una forma similar de hacer el update... Esa es mi pregunta.

[andresdzphp]

Recorre $_POST con un foreach y formas tu consulta. No olvides las validaciones.

[thewayox]

Eeehmmm... Disculpa... No entendi :(

Una explicación para alguien nivel basico para mi?

[iniciopublic]

Iniciado por thewayox

Uso esto:

Código PHP:

  $result=mysql_query("select * from tabla where id='$id'",$link);
while ($rows = mysql_fetch_array($result, MYSQL_ASSOC)) {
    foreach ($rows as $key => $value) {
        $$key = $value;
    }
}  
   

Con estas 6 lineas me ahorre el declarar todas las variables, o sea, un trabajon, ademas de que el mismo codigo me servirá para otros formularios de igual (47) o mas campos... Lo unico que tendría que cambiar es la consulta.

Creo que debe haber una forma similar de hacer el update... Esa es mi pregunta.

Hola disculpa mi ignorancia soy nuevo en la programacion, pero como dices que ese codigo de ahora poner todas esas variables para que te impriman ¿para eso es?

[gnzsoloyo]

Iniciado por thewayox

Hola, gracias de antemano a quienes me puedan ayudar.

Tengo un formulario con 47 campos que leo de una tabla en MySQL, en este formulario edito estos campos.

La pregunta es, existe una forma de actualizarlos que no sea la clasica...

Código PHP:

  $consulta="update tabla set campo1='$campo1', campo2='$campo2', campo3='$campo3',...., campo47='$campo47' where id='$id'";  
   

Porque tendría que declarar todas esas variables, hacer ese chorizo gigante y de verdad que me parece que hay una forma mas facil, la cual no conozco aún.

Gracias por su tiempo y respuesta.

Por lo pronto, NO, el SQL es un lenguaje de consultas declarativo, por lo que para que algo se genere, hay que indicarlo explícitamente.
Por otro lado, 47 campos, en general, hacen pensar en una entidad muy mal definida, con probabilidad de faltas de normalización. Pero es tema de BBDD y no de PHP, por lo que no profundizaré en eso.

Resumiendo: No, no hay forma de evitarlo. El UPDATE tiene una única sintaxis. Como mucho se le agregan condiciones de JOIN, si es que son necesarias, pero los campos a modificar hay que indicarlos expresamente.

[thewayox]

Hice esto

Código PHP:

  $arraypost=array(); 
foreach($_POST as $nombre_campo => $valor){ 
    $asignacion = "\$" . $nombre_campo . "='" . $valor . "';"; 
    eval($asignacion); 
    $arraypost[]="$nombre_campo='$valor'"; 
} 
 
    $consulta="UPDATE tabla SET ".implode(', ',$arraypost)." where id='$id'";  
   

Y ya casi me funciona. Solo que hay una variable que no me interesa y se me viene, estoy pensando en eliminarla... Pero bueno, por aqui va la cosa.

Que opinan?

[pateketrueke]

Iniciado por thewayox

[...] Que opinan?

Usar eval() es lo peor que puedes hacer.

Si en lugar de eso usaras campos de formulario de tipo array sería más simple.

Código PHP:

<input name="foo[bar]" />
<input name="foo[candy]" />
 

Y al recibir:

Código PHP:

foreach ($_POST['foo'] as $key => $value) // ...
 

Me parece que es más limpio, además de que sólo recibes los campos agrupados y no todos, y encima te evitas usar eval() que ni al caso va.

[thewayox]

Oye pateketrueke... Que buen momento para hacer la pregunta...

Por qué eval() no es recomendado?

[gnzsoloyo]

Manual de referencia de PHP: http://php.net/manual/es/function.eval.php

Precaución

El constructor de lenguaje eval() es muy peligroso porque permite la ejecución de código de PHP arbitrario. Su uso está totalmente desaconsejado. Si se ha verificado cuidadosamente que no existe otra opción que usar este constructor, se ha de poner especial atención en no pasar ninguna información proporcionada por el usuario a esta función sin haberla validado apropiadamente con anterioridad.

¿Queda suficientemente claro?

Los manuales de referencia son el primer sitio donde debes buscar las respuestas...

[thewayox]

Claro que allí lo lei, fue el primer lugar donde busque...

Esperaba un ejemplo mas "en criollo"... En otras palabras...

[AlanChavez]

Con respecto a eval(),

como dice el manual: eval te permite ejecutar codigo PHP arbitrario y por lo visto tu MySQL ya es vulnerable a inyecciones SQL y esta mas que claro que no estas validando los datos enviados por el usuario.

Que pasaría si un usuario, en lugar de enviarte

Código PHP:

$_POST['nombre'] = "Alan";
 

Te envía

Código PHP:

$_POST['nombre'] = "shell_exec('rm -rf ~')";
 

y luego utilizas

Código PHP:

eval($_POST['nombre']);
 

Lo que PHP realmente entiende es:

Código PHP:

shell_exec('rm -rf ~');
 

O en otras palabras, PHP le esta diciendo al sistema que borre todos los archivos de la carpeta "home" del usuario que ejecuta php (generalmente el usuario apache en linux).

Por eso es que eval() es un comando peligroso y no se debe usar a menos de que estes 100% seguro de que nadie te va a borrar archivos.

[thewayox]

Oye Alan, muy agradecido... Eso era lo que necesitaba leer... Así si lo logré entender... Puntos positivos para ti!

Saludos amigo.