Блог

Я новичок в node и предполагаю, что здесь мне не хватает чего-то простого. По сути, у меня есть класс в файле модели, используемый для создания экземпляров новых объектов следующим образом:

 const mon&odb = require('mon&odb');
const &etDb = require('../util/database').&etDb;
const ObjectId = mon&odb.ObjectId; // needed?

class User {
  constructor(email, password) { // removed , id
    this.email = email;
    this.password = password;
  }

  save() {
    const db = &etDb();
    return db.collection('users').insertOne(this)
      .then(result =&&t; {
        console.lo&('save() result: ', result);
        return resu<
      })
      .catch(err =&&t; {
        console.lo&('error: ', err)
      });
  }

  static findById(userId) {
    const db = &etDb();
    return db
      .collection('users')
      .findOne({
        _id: new ObjectId(userId)
      })
      .then(user =&&t; {
        console.lo&(user);
        return user;
      })
      .catch(err =&&t; {
        console.lo&(err);
      });
  }

  static findByEm(em) {
    const db = &etDb();
    return db.collection('users')
      .find({
        'email': em
      })
      .next()
      .then(user =&&t; {
        console.lo&('user find test: ', user)
        return user;
      }).catch(err =&&t; {
        console.lo&('error: ', err)
      });
  }
}
  

module.exports = User;

И соответствующая часть моей функции контроллера выглядит следующим образом:

 exports.postReset = (req, res, next) =&&t; {
    crypto.randomBytes(32, (err, buffer) =&&t; {
        if (err) {
            console.lo&(err);
            return res.redirect('/reset');
        }
        const token = buffer.toStrin&('hex');
        User.findByEm(req.body.email)
            .then(user =&&t; {
                if (!user) { // if there's no user - chan&e to if user == null?
                    return res.render('auth/for&ot-password', {
                        pa&eHead: 'Reset Password',
                        pa&eTitle: 'Reset Password Pa&e',
                        pa&eIntro: 'No account with that email found.',
                        path: '/for&ot-password'
                    });
                }
                user['resetToken'] = token;
                user['resetTokenExpiration'] = Date.now()   3600000;
                console.lo&('userr: ', user);
                console.lo&(JSON.strin&ify(user));
                return user.save(); // ERROR HERE: TypeError: user.save is not a function
                //return user.prototype.save();
            })
            .then(result =&&t; {
                res.redirect('/');
                transporter.sendMail({
                    from: '"Test" <test@&mail.com',
                    to: req.body.email,
                    subject: 'Password reset',
                    html: `
                        <p&&t;You requested a new pasword.</p&&t;
                        <p&&t;Click the <a href="http://localhost:3000/for&ot-password/${token}"&&t;link</a&&t; to set a new password.</p&&t;
                    `
                })
            })
            .catch(err =&&t; {
                console.lo&(err);
            });
    });
}
  

После отправки формы электронной почты функция просто выполняет поиск в базе данных, чтобы увидеть, существует ли запись с тем же адресом электронной почты. Если это произойдет, предполагается, что она добавит два новых свойства resetToken и resetTokenExpiration , а затем обновит запись в базе данных этими новыми свойствами.

Функция успешно находит пользователя в базе данных, но по какой-то причине я не могу заставить ее вызвать save() метод, установленный в классе в файле модели, и регистрирует эту ошибку:

 TypeError: user.save is not a function
  

Почему он не вызывает функцию сохранения? Она установлена в классе как функция-прототип, поэтому теоретически он должен иметь возможность ее вызывать.

Спасибо за любую помощь здесь.

Ваш findByEm метод возвращает обещание, которое выполняется с помощью записи базы данных, а не экземпляра вашего User класса. Поэтому у него не будет .save метода. Вам нужно будет явно сконструировать экземпляр и вернуть это:

 static findByEm(email) {
  return &etDb().collection('users')
    .find({email})
    .next()
    .then(data =&&t; {
      console.lo&('user find test: ', data)
      return new User(data.email, data.password); // and id etc
//           ^^^^^^^^
    });
}
  

findByEm возвращает обещание, а не пользовательский объект. Поэтому вам нужно изменить способ обработки этого обещания:

 User.findByEm(req.body.email)
            .then((res, err) =&&t; {
    // use res.user here

  

В качестве альтернативы вы можете выбрать «ожидание» db.collection.find внутри findByEm и создать findByEm асинхронную функцию.

 static async findByEm(em) {
    const db = &etDb();
    return await db.collection('users')
      .find({
        'email': em
      })
      .next()
      .then(user =&&t; {
        console.lo&('user find test: ', user)
        return user;
      }).catch(err =&&t; {
        console.lo&('error: ', err)
      });
  }
  

Тогда ваш код по умолчанию, который ожидает, что User объект будет работать.