0

我要做的就是运行一个查询，检查一个电子邮件地址是否已经注册，如果是则拒绝请求，如果没有运行一些验证，然后在我的用户表中创建用户。这有点像壳，代码最初基于https://github.com/vitaly-t/pg-promise-demo，代码很棒，但我不是。承诺对我来说完全是荒谬的，我不知道我应该返回什么或在哪里返回。

这段代码运行并实现了它应该做的，但随后它遇到了问题并发回“无法读取未定义的属性'then'”。当我尝试链接第二个 .then 以使其不在第一个 .then() 中时，我收到错误，我试图在 res 已发送后更改它。

有人可以帮我理解这里出了什么问题以及如何解决吗？

useradd: function (req, res) {
    db.users.findbyemail(req.body.email)
        .then(function(data){
            if( typeof data === undefined || data === null) {
                 db.users.add({
                    provider: 'local',
                    email: req.body.email, 
                    password: req.body.password, 
                    salt: 'salt', 
                    displayName: req.body.displayName })
                    .then(function(data){
                        res.json({
                            success: true,
                            data: "Account created"
                        })
                    }) 
            } else {
                 res.json({
                    success: false,
                    error: "Email address " + req.body.email + " is already registered on this site"
                })
            }
        })    

        .catch(function(error){
            res.json({
                success: false,
                error: error.message || error
            })
        })
}
4

1 回答 1

0

我最终得到了一些工作，应用 Bergi 的规则确实将我推向了正确的方向，我还决定重构以使用 throws，因此只有一个成功路径，所有其他路径都使用 .catch 从而避免在发送后更改 res 问题。主代码现在如下所示：

useradd: function (req, res) {
    return db.users.findbyemail(req.body.email)
       .then(function(data){
           return module.exports.validateUser(data, req);
        })
        .then(function(){
            return db.users.add({
                provider: 'local',
                email: req.body.email, 
                password: req.body.password, 
                salt: 'salt', 
                displayName: req.body.displayName })  
        })
        .then(function(){
            res.json({
                success: true,
                data: "Account created"
            })
            return "acct created";
        }) 
        .catch(function(error){
            res.json({
                success: false,
                error: error.message || error
            })
        })
},
validateUser: function(data, req) {
    console.log("got to validateUser, data " + data + " req " + req);
    if (typeof data !== 'undefined' && data !== null) {
        throw new Error("Email address " + req.body.email + " is already registered on this site");
    } else {
        //validation goes here
        return null;
}
},

//with the db.users.add function modified from the version in my comment to:
add: (values)  => {    
        var reponeres = rep.one(sql.add, values);
        return reponeres; 
    },
于 2016-09-09T12:05:14.253 回答