как исправить 'Ошибка: Ошибка типа: не удается прочитать свойство' catch 'из неопределенного' при сохранении пользователя - PullRequest
Новая
       33

как исправить 'Ошибка: Ошибка типа: не удается прочитать свойство' catch 'из неопределенного' при сохранении пользователя

0 голосов
/ 04 июня 2019

Есть модель пользователя и app.js,

в app.js у нас есть перехват для user.save () , который выдает ошибку в консоли: 

TypeError: Cannot read property 'catch' of undefined

модель пользователя: 

const Schema = mongoose.Schema

const UserSchema = new Schema({
    email: {
        type: String,
        required: true,
        unique: true,
        trim: true,
        minLength: 3        
    },
    password: {
        type: String,
        required: true,
        minLength: 5
    }

})

module.exports = mongoose.model( 'users', UserSchema )

файл app.js: 

const mongoose = require( 'mongoose')
mongoose.Promise = global.Promise

const bcrypt = require( 'bcryptjs')
const User = require('./models/User')
const express = require('express')
const app = express()

const bodyParser = require('body-parser')
app.use(bodyParser.json())
app.use(bodyParser.urlencoded({ extended: true }))



app.listen( 4111, () => {
    console.log( 'listening on 4111 ...' )
})

mongoose.connect( 'mongodb://127.0.0.1/login', () => {
    console.log( 'Connected ! ')
})

app.post('/register', ( req, res ) => {
    let newUser = new User()
    newUser.email = req.body.email || 'gmail'
    newUser.password = req.body.password || 1234
    // res.send( newUser )
    bcrypt.genSalt( 10,  salt => {
        bcrypt.hash( req.body.password, salt, ( err, hash ) => {
            if( err ) return err 
            newUser.password = hash
            newUser.save().then( savedUser => {
                res.send( 'The user saved: ', savedUser )
             }).catch( (err) => {
                res.send( 'The user was not saved : ', err )
            })
        })
    } ).catch( err => {
        res.send( 'gen salt err :', err )
    })
})

Я видел здесь похожий вопрос, но он был не таким, как этот.

1 Ответ

0 голосов
/ 04 июня 2019

.catch можно использовать по обещанию. так как вы не возвращаете никаких обещаний (или вообще ничего), все равно, что использовать .catch для undefined. Вместо этого используйте try catch 

app.post('/register', ( req, res ) => {
let newUser = new User()
newUser.email = req.body.email || 'gmail'
newUser.password = req.body.password || 1234
// res.send( newUser )
try{
  bcrypt.genSalt( 10,  salt => {
      bcrypt.hash( req.body.password, salt, ( err, hash ) => {
         if( err ) return err 
         newUser.password = hash
         newUser.save().then( savedUser => {
             res.send( 'The user saved: ', savedUser )
          }).catch( (err) => {
             res.send( 'The user was not saved : ', err )
         })
       })
  })
 } catch(err){
     res.send( 'gen salt err :', err )    
  }
})
...