No Actualiza Update MySQL - PHP

mpaglia · #1 (**permalink**) 05/05/2011, 22:23

Buenas gente, mi primer pregunta por aqui. Soy novato, muy novato con PHP.
Estoy armando una base de datos, en la cual uno se registra y luego el usuario puede modificar los datos ya ingresados.
Todo perfecto, solo que el formulario de actualizacion realiza las funciones pero no actualiza la base MySQL.
El formulario que muestra los datos a modificar es el siguiente

Código:

<?php
include "conexion.php";
$id=$_GET['id'];
$result=mysql_query("select * from corredores where id=$id",$conexion);
  while($row = mysql_fetch_array($result)) { 
      echo("<form name=corredores action=grabaract.php?id=$row[0] method=post>
DNI: <input type=text name=dni value=$row[1]><br>
Nombre: <input type=text name=nombre value=$row[2]><br>
Apellido: <input type=text name=apellido value=$row[3]><br>
<input name=enviar' type=submit value=Enviar />
</form>"); 
   }         
include "cerrarconexion.php"
?>

Y el formulario "grabaract.php" que realiza la acción seria el siguiente:

Código:

<?php
$id=$_GET['id'];
$dni=$_POST['dni'];
$nombre=$_POST['nombre'];
$apellido=$_POST['apellido'];
include "conexion.php";
$sql = "SELECT * FROM corredores WHERE id=$id"; 
$result = mysql_query($sql);
$query = "UPDATE corredores SET dni=$dni,nombre=$nombre,apellido=$apellido";
$result=mysql_query($query);
if($query) 
{ 
echo "Actualizacion OK ! "; 
} 
else 
{ 
echo " No hubo actualizacion"; 
} 
//.Cerramos Conexion
include "cerrarconexion.php";
?>

No encuentro la falla, y me esta rompiendo la cabeza!
Muchas gracias de antemano por responder.
Saludos.

Sourcegeek · #2 (**permalink**) 05/05/2011, 22:47

El error clásico, las comillas.. Recuerda que en una query debes comillar texto!

Código PHP:

  //Mal
$query = "UPDATE esto(uno,dos) SET uno=$uno, dos=$dos";

//Bien
$query = "UPDATE esto(uno,dos) SET uno='$uno', dos='$dos'";

Corrígelo en tu código y cuéntanos qué tal

s00rk · #3 (**permalink**) 06/05/2011, 00:31

Sigue estando mal el codigo para grabar bien tambien deves usar el WHERE sino estaras actualizando todos los registros en la Tabla.

Tamoco nunca ejecutas la query, el codigo deveria ser asi:

Código PHP:

  <?php
/// ESTE TIPO DE INCLUDE ES MEJOR PONERLO SOBRE TODO PARA APRECIARLO MEJOR ///
include "conexion.php";
/// RECUERDA LIMPIAR LAS VARIABLES ///
$id=$_GET['id'];
$dni=$_POST['dni'];
$nombre=$_POST['nombre'];
$apellido=$_POST['apellido'];
/// USAMOS WHERE EN EL UPDATE PARA ACTUALIZAR UN SOLO REGISTRO SINO ACTUALIZARA TODOS ///
$query = "UPDATE corredores SET dni='".$dni."',nombre='".$nombre."',apellido='".$apellido."' WHERE ID='".$id."'";
$result=mysql_query($query);
/// COMPROVAMOS QUE SE HAYA EJECUTADO LA CONSULTA(QUERY) , TU COMPROVAVAS EL TEXTO POR LO CUAL SIEMPRE SALDRIA QUE ESTABA BIEN ///
if($result) 
{ 
    echo "Actualizacion OK ! "; 
} else { 
    echo "No hubo actualizacion"; 
} 

//.Cerramos Conexion
include "cerrarconexion.php";
?>

mpaglia · #4 (**permalink**) 06/05/2011, 07:13

Cita:

Iniciado por s00rk

Sigue estando mal el codigo para grabar bien tambien deves usar el WHERE sino estaras actualizando todos los registros en la Tabla.

Tamoco nunca ejecutas la query, el codigo deveria ser asi:

Código PHP:

  <?php 
/// ESTE TIPO DE INCLUDE ES MEJOR PONERLO SOBRE TODO PARA APRECIARLO MEJOR /// 
include "conexion.php"; 
/// RECUERDA LIMPIAR LAS VARIABLES /// 
$id=$_GET['id']; 
$dni=$_POST['dni']; 
$nombre=$_POST['nombre']; 
$apellido=$_POST['apellido']; 
/// USAMOS WHERE EN EL UPDATE PARA ACTUALIZAR UN SOLO REGISTRO SINO ACTUALIZARA TODOS /// 
$query = "UPDATE corredores SET dni='".$dni."',nombre='".$nombre."',apellido='".$apellido."' WHERE ID='".$id."'"; 
$result=mysql_query($query); 
/// COMPROVAMOS QUE SE HAYA EJECUTADO LA CONSULTA(QUERY) , TU COMPROVAVAS EL TEXTO POR LO CUAL SIEMPRE SALDRIA QUE ESTABA BIEN /// 
if($result)  
{  
    echo "Actualizacion OK ! ";  
} else {  
    echo "No hubo actualizacion";  
}  
 
//.Cerramos Conexion 
include "cerrarconexion.php"; 
?>

Acabo de probar, me funcionó excelente! Muchisimas gracias, evidentemente soy muy novato en PHP jaja.
Saludos, y gracias nuevamente!

detective_jd · #5 (**permalink**) 06/05/2011, 07:50

Hola ¿cómo estás?, quiero darte un aporte, después de declarar las variables para modificar haz esto:

<?php
include "conexion.php";
$id=$_GET['id'];
$dni=$_POST['dni'];
$nombre=$_POST['nombre'];
$apellido=$_POST['apellido'];
if($id="" or $dni="" or $nombre="" or $apellido="")
{
$query = "UPDATE corredores SET dni='".$dni."',nombre='".$nombre."',apellido='".$a pellido."' WHERE ID='".$id."'";
$result=mysql_query($query);
}
else
{

}

//.Cerramos Conexion
include "cerrarconexion.php";
?>

detective_jd · #6 (**permalink**) 06/05/2011, 07:55

Hola ¿cómo estás?, quiero darte un aporte, después de declarar las variables para modificar haz esto:

<?php
include "conexion.php";
$id=$_GET['id'];
$dni=$_POST['dni'];
$nombre=$_POST['nombre'];
$apellido=$_POST['apellido'];
if($bandera>0) // este es el hidden mayor a 0
{
if($id="" or $dni="" or $nombre="" or $apellido="") si al menos una de las variables está vacía
{
echo "FALTAN INGRESAR DATOS"; // muestre error
}
else //sino modifique la base de datos
{
$query = "UPDATE corredores SET dni='".$dni."',nombre='".$nombre."',apellido='".$a pellido."' WHERE ID='".$id."'";
$result=mysql_query($query);
}
}
include "cerrarconexion.php";
?>

Espero que te sirva

detective_jd · #7 (**permalink**) 06/05/2011, 08:01

y para el cuerpo de la página es lo sgte:

<?php
include "conexion.php";
$id=$_GET['id'];
$result=mysql_query("select * from corredores where id=$id",$conexion);
while($row = mysql_fetch_array($result)) {
echo("<form name=corredores action=grabaract.php?id=$row[0] method=post>
DNI: <input type=text name=dni value=$row[1]><br>
Nombre: <input type=text name=nombre value=$row[2]><br>
Apellido: <input type=text name=apellido value=$row[3]><br>
<input name='enviar' type=submit value=Enviar />
<input name='hdnbandera' type='hidden' value='1' />// declaro el hidden para cuando quiera modificar la BD sabré sí la página recarga al modificar o no
</form>");
}
include "cerrarconexion.php"
?>

s00rk · #8 (**permalink**) 06/05/2011, 10:07

@detective_jd: Dejame decirte primero que tus codigos tienen varios errores jeje desde comparaciones y ejecuciones.

Y lo del hidden no entiendo para que ...

detective_jd · #9 (**permalink**) 08/05/2011, 19:27

hola s00rk verás me apuré añ escribir lo que quise hacer se lo sgte:

este es el cuerpo de la página:

<?php
include "conexion.php";
$id=$_GET['id'];
$result=mysql_query("select * from corredores where id=$id",$conexion);
while($row = mysql_fetch_array($result))
{
echo("<form name=corredores action=grabaract.php?id=$row[0] method=post>
DNI: <input type=text name=dni value=$row[1]><br>
Nombre: <input type=text name=nombre value=$row[2]><br>
Apellido: <input type=text name=apellido value=$row[3]><br>
<input name='enviar' type=submit value=Enviar />
<input name='hdnbandera' type='hidden' value='1' />// declaro el hidden para cuando quiera modificar la BD sabré sí la página recarga al modificar o no
</form>");
}
include "cerrarconexion.php"
?>

y el php del update:

<?php
include "conexion.php";
$id=$_GET['id'];
$dni=$_POST['dni'];
$nombre=$_POST['nombre'];
$apellido=$_POST['apellido'];
if($bandera>0) // este es el hidden mayor a 0
{
if($id="" or $dni="" or $nombre="" or $apellido="") si al menos una de las variables está vacía
{
echo "FALTAN INGRESAR DATOS"; // muestre error
}
else //sino modifique la base de datos
{
$query = "UPDATE corredores SET dni='".$dni."',nombre='".$nombre."',apellido='".$a pellido."' WHERE ID='".$id."'";
$result=mysql_query($query);
}
}
include "cerrarconexion.php";
?>

lo q quise decir es q para asegurarse que estén escritos todas las cajas de texto q usará un if sí al menos una de las variables están vacías q en la página muestre un mje diciendo q faltan datos x ingresar y lo del hidden es para ver sí la pñagina funciona bien q haga lo mencionado anteriormente eso quise decir.

mpaglia · #10 (**permalink**) 10/05/2011, 10:40

Me funciona perfecto el código, muchas gracias!
Ahora estoy teniendo un problema, los datos que ingreso con algun espacio entre medio la base de datos no me lo devuelve correctamente en el campo, solo me devuelve la primera parte.
Si se ingresó, por ejemplo:
Nombre: Juan Perez
En la base de datos veo:
Nombre: Juan Perez
Pero en el formulario de modificar datos veo:
Nombre: Juan

No lo puedo solucionar, nuevamente mi novatez se ve a la luz!
Saludos

mpaglia · #11 (**permalink**) 10/05/2011, 10:47

Solucionado, falta de comillas simples...

Gracias igual, y perdon por robarles su tiempo!

cecitn · #12 (**permalink**) 03/12/2014, 16:09

muy bueno el codigo aunqueeee me funciona a medias :(