Home / Questions / 413819
Accepted
Claudio
Asked: 2021-12-12 08:49:15 +0800 CST 2021-12-12 08:49:15 +0800 CST

Why does nodejs/express respond with a res.status(400) in the data that arrives from the request?

  • 772

Since I Angularsend the data to nodejs, when it receives it I ask id_userif it is nullor undefinedin the following way:

if(!req.params.id_user){
res.status(400).send('FALTA CONTENIDO EN EL CUERPO, falta el id_user');

}

The weird thing is that since AngularI'm sending it as id_user = 1, so nodejsit shouldn't respond with a res.status(400). The data arrives fine because if I remove the if(!req.paramas.id_user)the data is saved perfectly in the database.

here from nodejstheorder.controller.ts

export const createOrder = async (req: Request, res: Response): Promise<Response> => {

  if(!req.params.id_user){
    res.status(400).send('FALTA CONTENIDO EN EL CUERPO, falta el id_user');
  }

  //recibo los datos (de un form, insomnia rest, etc..)
  const { id_user, adress, phone_number, total_price, provincia, localidad, order_date } = (req.body);
  console.log(id_user, adress, phone_number, total_price, provincia, localidad, order_date)

  let idUser = parseInt(id_user);
  let totalPrice = parseInt(total_price);

  const response: QueryResult = await pool.query('INSERT INTO orders (id_user, adress, phone_number, total_price, provincia, localidad, order_date) VALUES ($1, $2, $3, $4, $5, $6, $7)', [idUser, adress, phone_number, totalPrice, provincia, localidad, order_date]);
  return res.json({
    message: 'La orden ah sido creado exitosamente!',
    body: {
      orders: {
        adress
      }
    }
  })
}

Here Angular, where I send the data to nodejswith theid_user = 1

createOrder() {
    this.order.id_user = 1;
    this.order.order_date = this.currentDate;
    this.order.total_price = this.total;
    this.orderService.createOrder(this.order).subscribe(
      res => {
        console.log('La orden fue creada exitosamente');
        this.createOrderDetail();
      },
      err => console.log('No se pudo crear la orden ' + err)
    );
  }

my serviceorder.service.ts

createOrder(order: Order){
    return this.http.post('http://localhost:4000/orders/create', order);
  }

nodejs

1 Answers

  1. Best Answer

    req.paramsreceive the data from the url. you must do areq.body.id_user

    req.params

    This property is an object that contains the properties mapped to the route called "parameters". For example, if you have the route /user/:name, then the "name" property will be available as req.params.name. The default value of this object is{}

    In your service you are not sending any parameters by url. You send everything through the body.

    createOrder(order: Order){
                           //Orden contiene id_user
     return this.http.post('http://localhost:4000/orders/create', order);
  }

    req.body

    Contains key-value data pairs sent in the body of the request. By default it is undefined and is populated when you use body parsing middleware like express.json() or express.urlencoded()

    Your code works if you remove the line because in the end you are using req.body:

      //En esta linea usas req.body para asignar los datos
  const { id_user, adress, phone_number, total_price, provincia, localidad, order_date } = (req.body);
    • 1