Node js从函数

时间:2018-09-30 23:09:49

标签: javascript mysql node.js node-modules

我生活在PHP世界中,但是我正在尝试使用Node构建REST Api。

我整天都试图从for循环返回结果数组。基本上,我传递的是field_name:field_value的数组。我想将更新的结果推送到数组中以返回。我可以将其登录到控制台,但无需进一步操作。

这是发布json数据的示例

{
    "first_name":"Jeff",
    "phone":"4855555555"
}

这是函数和循环

function UpdateKey(user_id, key, value, cb) {
    connection.query('UPDATE users SET ' + key + ' = ? WHERE id = ? LIMIT 1', [value, user_id], function(err, results) {
    if (err) {
        callback = key + " update failed.";
    } else {
        callback = key + " was updated.";
    }
    cb(callback);

  });
}    

for (myKey in post_data) {
  UpdateKey(user_id, myKey, post_data[myKey], function(id) {
    console.log(id);
  });
}

res.send(JSON.stringify({ "status": 200, "error": "", "response": my_results_here }));

我一直在研究异步,但是不确定最佳方法。任何帮助都会很棒!

2 个答案:

答案 0 :(得分:1)

您可以将所有结果收集到数组中,然后在数组大小等于键大小时发送该结果:

const keys = Object.keys(post_data);
const response = [];

for(const myKey of keys) {
  UpdateKey(user_id, myKey, post_data[myKey], function(id) {
    response.push(id);
    if(keys.length === response.length) {
      res.send(JSON.stringify({ 
       status: 200, 
       error: "", 
       response
      }));
    }
  });
}

答案 1 :(得分:1)

您想要的解决方案:

const updateUserField = (userId, field, value) => {
  return Promise((resolve) => {
    const query = 'UPDATE users SET ' + field + ' = ? WHERE id = ?';
    const data = [value, userId];
    connection.query(query, data, (error) => {
      if (error) return resolve(field + ' update failed');
      resolve(field + ' was updated');
    });
  });
};


router.post('/user/:id', async (req, res) => {
    const userId = req.params.id;
    const data = req.body;

    const response = [];
    for (const field in data) {
      response.push(
        await updateUserField(userId, field, data[field])
      );
    }

    res.status(200).send({
      response
    });
});


或并行:

router.post('/user/:id', async (req, res) => {
    const userId = req.params.id;
    const data = req.body;

    const response = await Promise.all(
      Object
        .keys(data)
        .map(field => updateUserField(userId, field, data[field]))
    );

    res.status(200).send({
      response
    });
});



正确的解决方案

据我了解,您希望获取用户表中的帖子数据并更新记录。

那么为什么不只在一个查询中这样做呢?

尝试这种方式:

const updateUser = (userId, data) => {
  return Promise((resolve, reject) => {
    const query = 'UPDATE users SET ? WHERE id = ?';
    connection.query(query, [data, userId], (error) => {
      if (error) return reject(error);
      resolve();
    });
  });
};

router.post('/user/:id', async (req, res) => {
  try {
    const userId = req.params.id;
    const data = req.body;
    await updateUser(userId, data);
    res.status(200).send({
      message: 'User account successfully updated'
    })
  }
  catch (error) {
    console.error(error);
    res.status(500).send({
      message: 'Failed update user account'
    });
  }
});



但是最好考虑使用ORM,即Sequelize来提高安全性,验证等功能,从而简化开发人员的工作。