循环遍历请求体节点 js + express

Question

Im sending a request with data as an array of objects:

 [
  {id: "1"},
  {id: "2"},
  {id: "3"}
 ]

im using JSON.stringify() and my req.body looks like this: { '{"id":"1"},{"id":"2"},{"id":"3"}': '' }

Now i want to loop through req.body and get all the ids so i can delete them from the SQL DB. Im using Sequelize. The back end:

exports.deleteIds = (req, res, next) => {
    console.log(req.body)
//here should be loop so i can delete all the ids one by one.
    Model.destroy({
        where: {
            id: 
        }
    })
}

The post request (client):

let ids = []
//maybe here is the problem?
for (var i = 0; i < selectedRow.length; i++) {
   ids.push({id:selectedData[i].id})          
}

let Url = "/admin/deleteIds"
let data = JSON.stringify(ids)

event.preventDefault();

$.post(Url, data, function (data, status) {

  }).done(function (res) {
     if (res.ids.length == 0) {
        $('#mainContent').html('<h1>0 users found</h1>')
     }
  })
  .fail(function (err) {
    console.log(err.responseJSON.message)
  })

Answer 1

EDIT

We we do is all even easier, by sending and array of ids and using it in the server directly.

Client:

let ids = []
for (var i = 0; i < selectedRow.length; i++) {
   ids.push(selectedData[i].id)  // <-- this is the change         
}

let Url = "/admin/deleteIds"
let data = {items: ids}
//...

Server:

exports.deleteIds = (req, res, next) => {
   const ids = req.body.items; // <-- no mapping needed
   Model.destroy({
      where: {id: ids}
   })
}

---

POST calls body should be a valid JSON, which means that it should be an js object.

Assuming you use Fetch

Sending data to server with Fetch API

 const rawResponse = await fetch('https://httpbin.org/post', {
    method: 'POST',
    headers: {
      'Accept': 'application/json',
      'Content-Type': 'application/json'
    },
    body: JSON.stringify({items:  [ {id: "1"}, {id: "2"}, {id: "3"} ]})
  });

First, if you get the req.body as an object, there is no reason to use JSON.stringify . if not use JSON.parse or Express's body-parser

Second you probably have a way in the DB library to send multiple IDs to destroy.

If you use Sequelize.js for example:

exports.deleteIds = (req, res, next) => {
   const ids = req.body.items.map(({id}) => id); 
   Model.destroy({
      where: {id: ids}
   })
}

In case this option does not exist, lets loop:

exports.deleteIds = (req, res, next) => {
   req.body.items.forEach(({id}) => {
      Model.destroy({
        where: {id}
      })
  }) 
}

Your fixed POST call:

let ids = []
for (var i = 0; i < selectedRow.length; i++) {
   ids.push({id:selectedData[i].id})          
}

let Url = "/admin/deleteIds"
let data = {items: ids} // <-- this is the change

event.preventDefault();

$.post(Url, data, function (data, status) {

  }).done(function (res) {
     if (res.ids.length == 0) {
        $('#mainContent').html('<h1>0 users found</h1>')
     }
  })
  .fail(function (err) {
    console.log(err.responseJSON.message)
  })

Answer 2

As another user mentions, req.body should always be valid json, so receiving an array is unlikely.

When you receive a valid json request, in Sequelize you can use three methods:

1. A get() function that determines what you retrieve from the database: https://sequelize.org/master/manual/getters-setters-virtuals.html

2. A set() method on the model that runs a function on what you want to save to the database:

const User = sequelize.define('user', {
  username: DataTypes.STRING,
  password: {
    type: DataTypes.STRING,
    set(value) {
      this.setDataValue('password', hash(value));
    }
  }
});

3. (Probably my recommendation here) A function directly on the controller, like in this example where I slugify an organization name:

  try {
    const result = await Organization.create({
      name: req.body.name,
      type: req.body.type,
      slug: slugify(req.body.name, { lower: true, remove: /[*+~.()'"!:@]/g, strict: true }),
      fields: allowedFields
    }) catch(err) { etc.}