2017-02-23 56 views
0

嗨,有人可以看看我的代码吗?我在后端实现身份验证我正在构建一个副项目来提高我对JavaScript的理解。处理难度在javascript中的承诺

我不知道为什么从猫鼬发现的承诺返回undefined。

我有两个功能需要帮助。一个是帮助功能,将传递给控制器​​。

import db from '../models'; 
import bcrypt from 'bcrypt'; 
import validateLogin from '../validations/login'; 
import isEmpty from 'lodash/isEmpty'; 
const loginController = {}; 

loginController.login = function(req,res){ 
    validateLogin(req.body).then(({isValid, errors }) => { 
    // isValid is undefined here 
    // This is the problem 
     if(isValid){ 
      // give token 
      res.status(200).json({ 
       success: true, 
      token: 'here is your token' 
      }); 
     } else { 
      res.status(401).json({ 
      errors 
      });  
     }  
    }).catch(err =>{ 
     console.log(err) 
    });  
}; 

export default loginController; 

另一个是控制器功能本身,将根据助手函数是否返回有效或不响应来使用令牌。

import validator from 'validator'; 
    import db from '../models'; 
    import isEmpty from 'lodash/isEmpty'; 
    import bcrypt from 'bcrypt'; 

    function validateLogin(data){ 
     const { userInput, password } = data; 
     const errors = {}; 
     if(validator.isEmpty(userInput)){ 
     errors.userInput = 'username is required'; 
     } 
     if(validator.isEmpty(password)){ 
     errors.password = 'password is required'; 
     } 
     return db.User.find({$or:[{ username: userInput }, { email: userInput }]}).then(existingUser =>{ 
     if(existingUser.length > 0){ 
      // User exists, check if password matches hash 
      const user = existingUser[0]; 
      bcrypt.compare(password, user.password_digest).then(valid => { 
       if(!valid){ 
        errors.password = 'Invalid Password'; 
       } 
       console.log('from prmomise'); 
       return { 
        isValid: isEmpty(errors), 
        errors 
       }; 
      }).catch(err => console.log(err)); 
      } else { 
      errors.userInput = 'username or email does not exist'; 
      return { 
       isValid: isEmpty(errors), 
       errors 
      }; 
      } 
     }); 
    } 

    export default validateLogin 
+1

*处理难度承诺*在现实生活中也是如此:-p – Rajesh

+1

为什么downvotes? – Nate

+0

承诺是承诺@Rajesh^_^ – Deep

回答

1

它看起来像一个return缺少的bcrypt电话:

return bcrypt.compare(password, user.password_digest).then(valid => { 
    // password check and returning result object 
}).catch(err => console.log(err)); 

没有它,你的validateLogin函数最终返回解析为undefined的承诺。

+0

'最终返回undefined' - 否,'validateLogin'返回一个承诺......当'db.User.find'完成后'.then'完成时,这个承诺将解析为'undefined' :p –

+0

Thx,你说得对,那就是我的意思。我修好了它。 – lex82

+0

太棒了!我的主要失误是什么。为了解决这个问题,我坚持了两个小时 – Nate