0

我试图在我的模型用户上调用这个函数（我正在使用猫鼬）。像这样：

UserSchema.statics.exists = function exists(req,email, callback) {
    this.findOne({
        email : email
    }, function(err, user,callback) {
        if(err) {
            console.error(err);
            return callback(err);
        }
        if(!user) {
            // console.log("Not user");
            return callback(null, false);// produce error
        }
        if(!user.valid) {
            console.log("User invalid");
            var hostname = req.headers.host;
            // hostname = 'localhost:8080'
            //var pathname = url.parse(req.url).pathname; // pathname = '/MyApp'

            var base_url = 'http://' + hostname + '/activation?key=' + user.account_no;
            user.verifyEmail(base_url, user, function(err, result) {
                if(err) {
                    console.error(err);
                return  callback(err);
                } else {
                    //if(email sent)
                    if(result) {
                    return  callback("Please check your email to activate your account");
                    } else {
                    return  callback("Activation error please contact WOWITO support");
                    }
                }
            });
        }
        return callback(null, user);
    });
}

但后来我收到以下错误：

node.js:201 抛出 e；// process.nextTick 错误，或第一次滴答时的 'error' 事件 ^ TypeError: undefined is not a function

我做错什么了？

谢谢，

4

1 回答 1

4

您callback目前有 2 个不同的变量：

UserSchema.statics.exists = function exists(req, email, callback) { // 1st
    this.findOne({
        email : email
    }, function(err, user, callback) { // 2nd
    // ...

由于它们共享相同的标识符，第二个将“遮蔽”第一个，使匿名函数中的第一个不可访问。

要使用第一个，您必须重命名其中一个 - 也许，existsCallback和/或findOneCallback.

您也可以完全删除第二个，因为undefined无论如何它似乎是：

UserSchema.statics.exists = function exists(req, email, callback) {
    this.findOne({
        email : email
    }, function(err, user) {
    // ...

您还假设正在为 传递一个值callback，而 JavaScript 实际上并不要求或强制执行该值。

您可以通过在调用之前测试一个值来解决此问题：

if (callback) callback(...);

或者在未定义时将其设置为“无操作”功能：

callback = callback || function() { return true; };
//...
callback(...);
于 2012-05-02T18:57:38.813 回答