2017-02-23 49 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