2017-02-23 56 views
0

Привет, может кто-нибудь взглянуть на мой код? Я выполняю аутентификацию на задней панели, которую я создаю как побочный проект, чтобы улучшить мое понимание JavaScript.Сложность обработки Обещания в javascript

Я не уверен, почему обещание от поиска мангусты возвращается неопределенно.

У меня есть две функции, с которыми мне нужна помощь. Одна из них - это вспомогательная функция, которая будет передана контроллеру.

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; 

Другая функция контроллера сама, что будет issuse маркер в зависимости от того, возвращает ли вспомогательная функция действительного или нет ответа.

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

'в конечном итоге возвращает неопределенные' - нет,' validateLogin' возвращает обещание ... это обещание разрешит 'undefined', когда' .then' после 'db.User.find' заканчивается : p –

+0

Thx, вы правы, вот что я имел в виду. Я починил это. – lex82

+0

Замечательно! Что за то, что я сделал. Я просунул твердые два часа, пытаясь исправить это – Nate