1

A mi parecer todo va Ok, pero cuando hace la sentencia me da el siguiente error:

Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[HY093]: Invalid parameter number: parameter was not defined' in /Applications/XAMPP/xamppfiles/htdocs/APP2/include/crud_usuario.php:119 Stack trace: #0 /Applications/XAMPP/xamppfiles/htdocs/APP2/include/crud_usuario.php(119): PDOStatement->execute() #1 /Applications/XAMPP/xamppfiles/htdocs/APP2/include/administrar_usuario.php(42): CrudUsuario->actualizar(Object(Usuario)) #2 {main} thrown in /Applications/XAMPP/xamppfiles/htdocs/APP2/include/crud_usuario.php on line 119

public function actualizar($usuario){
    $db = BaseDatos::conectar();
    $actualizar = $db->prepare('UPDATE Usuario SET usuario=:usuario,contrasena=:contrasena,nombre=:nombre,apellido=:apellido,dni=:dni,email=:email,telefono=:telefono,direccion=:direccion,cod_postal=:cod_postal,fecha_nac=:fecha_nac,curso=:cruso,foto=:foto,estado=:estado,tipo=:tipo WHERE id_usuario=:id_usuario');

    $actualizar->bindValue(':id_usuario', NULL);
    $actualizar->bindValue(':nombre',$usuario->getNombre());
    $actualizar->bindValue(':apellido',$usuario->getApellido());
    $actualizar->bindValue(':dni',$usuario->getDni());
    $actualizar->bindValue(':usuario',$usuario->getUsuario());
    $actualizar->bindValue(':contrasena',$usuario->getContrasena());
    $actualizar->bindValue(':email',$usuario->getEmail());
    $actualizar->bindValue(':telefono',$usuario->getTel());
    $actualizar->bindValue(':direccion',$usuario->getDireccion());
    $actualizar->bindValue(':cod_postal',$usuario->getCodPostal());
    $actualizar->bindValue(':fecha_nac',$usuario->getFechaNac());
    $actualizar->bindValue(':curso',$usuario->getCurso());
    $actualizar->bindValue(':foto',$usuario->getFoto());
    $actualizar->bindValue(':estado',$usuario->getEstado());
    $actualizar->bindValue(':tipo',$usuario->getTipo());

    $actualizar->execute();
}
3
  • 1
    ¿Te has fijado que estás actualizando únicamente el id_usuario que vale NULL? Te recomiendo ENCARECIDAMENTE que tabules el SQL como en la última respuesta que te di. Mejorará mucho la depuración. Commented el 23 mar. 2018 a las 12:49
  • Hola gracias por la recomendación así lo tengo y es más ágil y comprensible gracias. He borrado $actualizar->bindValue(':id_usuario', NULL); pero me sigue saliendo el error. Siento mi falta de comprensión pues estoy aprendiendo y estos errores entiendo que son cosas básicas. Gracias
    – sergibarca
    Commented el 23 mar. 2018 a las 12:55
  • 1
    Mira mi respuesta (la estaba redactando cuando escribí el comentario). Tu problema está en que no coinciden el nombre que has dado a una variable dentro del SQL y fuera de éste. Si tabulas correctamente el código como te pongo en mi respuesta podrás depurar estas cosas de manera más sencilla. Commented el 23 mar. 2018 a las 12:57

1 respuesta 1

2

El problema que estás sufriendo es que estás usando en el SQL un marcador (placeholder) con nombre :cruso (curso=:cruso) pero luego le asignas valor al marcador :curso en el bindValue.

Al ejecutar la consulta PDO te genera una excepción porque MySQL no encuentra valores para todos los marcadores, por eso te aparece ese mensaje de error:

Invalid parameter number: parameter was not defined

En castellano:

Número de parámetros inválido: el parámetro no fue definido

Lo ideal sería que bindValue fuera quien detectara las incoherencias, pero por desgracia no es así y de ahí que éste sea un error muy común.

He arreglado tu código para que en el SQL se muestre el nombre correcto:

public function actualizar($usuario) {
    $db = BaseDatos::conectar();
    $actualizar = $db->prepare('
      UPDATE Usuario
      SET
        usuario = :usuario,
        contrasena = :contrasena,
        nombre = :nombre,
        apellido = :apellido,
        dni = :dni,
        email = :email,
        telefono = :telefono,
        direccion = :direccion,
        cod_postal = :cod_postal,
        fecha_nac = :fecha_nac,
        curso = :curso,
        foto = :foto,
        estado = :estado,
        tipo = :tipo
      WHERE id_usuario = :id_usuario
    ');

    $actualizar->bindValue(':usuario', $usuario->getUsuario());
    $actualizar->bindValue(':contrasena', $usuario->getContrasena());
    $actualizar->bindValue(':nombre', $usuario->getNombre());
    $actualizar->bindValue(':apellido', $usuario->getApellido());
    $actualizar->bindValue(':dni', $usuario->getDni());
    $actualizar->bindValue(':email', $usuario->getEmail());
    $actualizar->bindValue(':telefono', $usuario->getTel());
    $actualizar->bindValue(':direccion', $usuario->getDireccion());
    $actualizar->bindValue(':cod_postal', $usuario->getCodPostal());
    $actualizar->bindValue(':fecha_nac', $usuario->getFechaNac());
    $actualizar->bindValue(':curso', $usuario->getCurso());
    $actualizar->bindValue(':foto',$usuario->getFoto());
    $actualizar->bindValue(':estado',$usuario->getEstado());
    $actualizar->bindValue(':tipo',$usuario->getTipo());
    $actualizar->bindValue(':id_usuario', $usuario->getIdUsuario());

    $actualizar->execute();
}
15
  • 1
    No te actualiza los datos porque no estás poniendo un valor en :id_usuario que exista en la tabla. Si usas $usuario->getIdUsuario() te funcionará. Commented el 23 mar. 2018 a las 12:59
  • 1
    @sergibarca, ¿existe un usuario con id igual a NULL? ¿No será este el motivo?
    – Marcos
    Commented el 23 mar. 2018 a las 13:01
  • 1
    @OscarGarcia, es un error tipográfico desde el momento en que ha escrito mal el nombre de la variable y siendo este el motivo del error.
    – Marcos
    Commented el 23 mar. 2018 a las 13:02
  • 1
    @sergibarca, esa misma pregunta te la hice yo en tu pregunta y te he respondido en estos comentarios. Editaré mi pregunta para incluir ese cambio. Commented el 23 mar. 2018 a las 13:03
  • 1
    Me ayudáis un montón gracias, voy a tratar de que me haga eso!
    – sergibarca
    Commented el 23 mar. 2018 a las 13:04

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.