-2

¿Existe algun error en el siguiente código?

Este código corresponde al archivo php que crea una nueva entrada en mi base de datos mysql, el cual en la parte frontend está realizada con angular.js pero no guarda en la base de datos la nueva entrada, por lo que necesito saber si ustedes ven algún error en el código por favor.

<?php
// Incluir la clase de base de datos
include_once("../classes/class.Database.php");

$postdata = file_get_contents("php://input"); 

$request = json_decode($postdata);
$request =  (array) $request;


$request['marca'] = strtoupper($request['marca']);


$sql = "INSERT INTO autos(marca, modelo, color, ano, precio,)
        VALUES ('".$request['marca']."',
                '".$request['modelo']."',
                '".$request['color']."', 
                '".$request['ano']."', 
                '".$request['precio']."' )";


$Hecho = Database::ejecutar_idu($sql);
$Respuesta = "";

if ($Hecho == "1") {
    $Respuesta = json_encode( array('err' => false, 'mensaje'=>'Registro Insertado.' ));
}else{
    $Respuesta = json_encode( array('err' => true, 'mensaje'=> $Hecho ));
}

echo $Respuesta;

?>
2
  • Bienvenido a SO. No sé php, creí que la pregunta era relacionada con angular.js como dice en el título y por eso estoy aquí. Lo único que puedo decir es que el código es vulnerable a inyecciones SQL.
    – ben
    Commented el 20 ene. 2021 a las 4:49
  • 1
    ¿No obtienes ningún tipo de error? ¿ En el error log tampoco ? Si no recibes ningún error, has probado a usar var_dump o print_r con las variables para ir viendo su valor mientras se va ejecutando el código y ver si dichas variables corresponden con lo que esperas?
    – Ibai A.
    Commented el 20 ene. 2021 a las 15:35

1 respuesta 1

1

Hay una coma extra al final del listado de columnas:

$sql = "INSERT INTO autos(marca, modelo, color, ano, precio,)

Esa línea debe ser

$sql = "INSERT INTO autos(marca, modelo, color, ano, precio)
1
  • estabas en lo correcto, por eso no me insertaba en la base de datos. muchas gracias Commented el 21 ene. 2021 a las 1:49

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

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