javascript - 回调 hell ，厄运金字塔，node.js

Question

我是node新手，当我完成我的API后，我意识到它们都是一团糟，并且是一个回调 hell ，这迫使我学习 promise ，现在一切都很好，直到我遇到了一个API条件超过可能的功能，我的问题是如何做嵌套 promise ，我的代码是关于一个父对象，它有一个用户附加到它，当删除路由被调用时有很多条件:

1. 如果这个 parent 有 child 附属于它，它不应该被删除

2. 没有 child

   一个。这个家长在不止一所学校，school_id 从父对象的学校 id 和附加到它的用户对象中删除

   如果这个 parent 没有 child 并且只有 school_id 应该删除它并且附加的用户也应该删除

代码看起来像这样，它是 100% 的功能

router.post('/delete',Validation, function (req, res) {
var school_id = req.body.schoolId;
var parent_id = req.body.selected[0];
/
Student.findOne({parent_ids:parent_id},function(err,parentF){
    if(err){
        console.log(err);
        res.json({status:"error",message:"an error occurred"});
        return
    }else if(parentF){
        res.json({status:"error", message:"you can not delete a parent who has students associated with it"});
        return;
    }else{
        Parent.findOne({_id:parent_id},function(err,parent){
            if(err){
                console.log(err);
                res.json({status:"error",message:"an error occurred"});
                return;
            }else{
                if(parent.school_id.length>1){
                    var a = parent.school_id.indexOf(school_id);
                    parent.school_id.pop(a);
                    parent.save(function(err,parent){
                        if(err){
                            console.log(err);
                            res.json({status:"error",message:"an error occurred"});
                            return;
                        }else{
                            User.findOne({refid:parent_id},function(err,user){
                                if(err){
                                    console.log(err);
                                    res.json({status:"error",message:"an error occurred"});
                                    return;
                                }else {
                                    user.school_id.pop(a);
                                    user.save(function(err) {
                                        if(err){
                                            console.log(err);
                                            res.json({status:"error",message:"an error occurred"});
                                            return;
                                        }else{
                                            res.json({status: "success", message: "parent removed"});
                                            return;
                                        }
                                    });
                                }
                            });
                        }
                    });
                }else{
                    Parent.remove({_id: parent_id}, function (err) {
                        if (err) {
                            res.json({status: "error", message: err.message});
                        } else {
                            User.remove({refid:parent_id},function(err){
                                if (err) {
                                    res.json({status: "error", message: "parent user wasn't deleted"});
                                    return;
                                }else{
                                    res.json({status: "success", message: "parent data successfully deleted"});
                                    return;
                                }
                            });

                        }
                    });
                }
            }
        });
    }
});
});

我正在尝试应用 promises 并使我的代码变得更好。

抱歉这个问题太长了，但我遇到了困难任何建议都表示赞赏

Answer 1

为 findOne 创建一些“promise”风格的函数，删除并保存

let findOneP = (object, params) => new Promise((resolve, reject) => {
    object.findOne(params, (err, data) => {
        if (err) {
            return reject(err);
        }
        resolve(data);
    });
});
let removeP = (object, params) => new Promise((resolve, reject) => {
    object.remove(params, (err, data) => {
        if (err) {
            return reject(err);
        }
        resolve(data);
    });
});
let saveP = (object) => new Promise((resolve, reject) => {
    object.save((err, data) => {
        if (err) {
            return reject(err);
        }
        resolve(data);
    });
});

使用 Bluebird Promisify 可以更轻松地完成上述操作，但是由于您没有指定 bluebird，所以我只是使用了一个快速而肮脏的 promise 包装器

然后使用 promise 链让生活变得轻松

router.post('/delete',Validation, function (req, res) {
    var school_id = req.body.schoolId;
    var parent_id = req.body.selected[0];

    findOneP(Student, {parent_ids: parent_id})
    .then(parentF => {
        if (parentF) {
            throw "you can not delete a parent who has students associated with it";
        }
    })
    .then(() => findOneP(Parent, {_id: parent_id}))
    .then(parent => {
        if (parent.school_id.length > 1) {
            var a = parent.school_id.indexOf(school_id);
            parent.school_id.pop(a);
            return saveP(parent)
                .then(parent => findOneP(User, {refid: parent_id}))
                .then(user => saveP(user))
                .then(() => "parent removed");
        }
        return removeP(Parent,{_id: parent_id})
            .then(() => removeP(User, {refid: parent_id}))
            .then(() => "parent data successfully deleted");
    })
    .then(message => res.json({status: "success",message}))
    .catch(message => res.json({status: "error",message}));
});

我刚读到您使用 Node 4.2 - 您可能需要

1. 通过像babel 这样的转译器运行它
2. 加载一个 Promise 库，例如 bluebird

如果使用 bluebird，请使用它的 promisify 函数而不是“手动”promisifying