express - 如何在Sequelize中更新一对多关系中的行？

Question

我有一个用户表，它与一对多关系中的权限表相关(用户有很多权限)，我想知道更新用户权限的干净和正确的方法。

这是我写的代码，但我觉得这不是正确的方法:

update(req, res) {
  db.sequelize
    .transaction()
    .then(t => {
        Group.findByPk(req.params.id).then(group => {
          if (!group)
            return res.status(404).send({
              error: "Ce groupe n'éxiste pas"
            });

         GroupPrivilege.destroy({
           where: { groupId: req.params.id },
           transaction: t
         }).then(resDestroy => {
           req.body.group_privileges.forEach(element => {
           element.groupId = req.params.id;
         });

        GroupPrivilege.bulkCreate(req.body.group_privileges, {
          transaction: t
        }).then(resCreatePrivileges => {
          delete req.body.group_privileges;
          console.log(req.body);
          Group.update(req.body, {
            where: { id: req.params.id },
            transaction: t
          }).then(resSave => {
            t.commit();
            return res.status(200).send("");
          });
        });
      });
    });
  })
  .catch(err => {
    t.rollback();
    return res.status(400).send(err);
  });
}

Answer 1

这是我写的代码，但我觉得这不是正确的方法:

 update(req, res) {
    db.sequelize
      .transaction()
      .then(t => {
        Group.findByPk(req.params.id).then(group => {
          if (!group)
            return res.status(404).send({
              error: "Ce groupe n'éxiste pas"
            });

          GroupPrivilege.destroy({
            where: { groupId: req.params.id },
            transaction: t
          }).then(resDestroy => {
            req.body.group_privileges.forEach(element => {
              element.groupId = req.params.id;
            });
            GroupPrivilege.bulkCreate(req.body.group_privileges, {
              transaction: t
            }).then(resCreatePrivileges => {
              delete req.body.group_privileges;
              console.log(req.body);
              Group.update(req.body, {
                where: { id: req.params.id },
                transaction: t
              }).then(resSave => {
                t.commit();
                return res.status(200).send("");
              });
            });
          });
        });
      })
      .catch(err => {
        t.rollback();
        return res.status(400).send(err);
      });
  }