“错误”：“无法读取未定义的属性‘findOne’”

Question

Hello everybody !! Here I have a big problem, I would like to do a registration in back with Node.js sequelize and mySql. I looked here and there but I did not find the answer to my problem so I came to ask you for help. With mongoDb, it was easier but I admit that I am going in circles.

Here is my code:

 // Importation : // Bcrypt: const bcrypt = require("bcrypt"); // Jsonwebtoken d'authentification: const jwt = require("jsonwebtoken"); // Import du models user: const models = require("../models/user") ////////////////////////////////////////////////////////////////////////////////////////////// // Fonction/ // Incription: exports.signup = (req, res) => { const username = req.body.username; const email = req.body.email; const password = req.body.password; const bio = req.body.bio; const admin = req.body.admin; console.log(req.body) try { models.User.findOne({ attributes: ['email'], where: { email: email } }) .then((userFound => { if (!userFound) { bcrypt.hash(password, 10, function (err, bcryptPassword) { const newUser = models.User.create({ username : username, email : email, password : bcryptPassword, bio : bio, admin : false }) .then(newUser => { res.status(201).json({ 'userId': newUser.id }) }) .catch(err => { res.status(500).json({ 'error': 'Impossible d\\'ajouter un utilisateur' }) }) }) } else { return res.status(409).json({ error: 'Ce compte existe déjà ' }) } }) .catch((err) => res.status(500).json({ 'err': err + 'Impossible de vérifier l\\'utilisateur', }) ) ) }catch (error) { res.status(400).json({ error: error.message }); } }

And the model User:

 'use strict' const { db } = require('../config/connexion') const { Sequelize, DataTypes } = require('sequelize') const user = db.define('User', { // Model attributes are defined here username: DataTypes.STRING, email: DataTypes.STRING, password: DataTypes.STRING, bio: DataTypes.TEXT, admin: DataTypes.BOOLEAN, }) module.exports = user

and connexion.js:

 // Connexion de sequelize à mysql: const { Sequelize } = require('sequelize') const db = new Sequelize( process.env.NAMEDB, process.env.USERDB, process.env.PASSWORDDB, { host: process.env.HOSTDB, dialect: process.env.DIALECTDB, pool: { min: 0, // nombre minimum de connexion dans le pool max: 5, // nombre maximum de connexion dans le pool acquire: 30000, // durée maximale, en millisecondes, pendant laquelle ce pool essaiera d'obtenir la connexion avant de lancer une erreur idle: 10000, // temps maximum, en millisecondes, pendant lequel une connexion peut être inactive avant d'être libérée }, } ) ////////////////////////////////////////////////////////////////////////////////////////////// // Etablit la connexion à mysql: const dbConnect = async (db) => { await db .authenticate() .then(() => { db.sync() console.log('Connecté à la base de données MySQL!') }) .catch((err) => { console.error('error: ' + err.message) setTimeout(() => { dbConnection(db) }, 5000) }) } ////////////////////////////////////////////////////////////////////////////////////////////// // Exportation: module.exports = { db, dbConnect, }

Certainly there is still a lot to do, but being a beginner I improve as I go.

Do not be angry with me if my English is not at the top, I admit that it is not my strong point.

Thanking you in advance for all the help provided.

Answer 1

You are directly setting the export object equal to the user object.

When you do this const models = require("../models/user") , models is equal to the user value directly.

You can directly use models.findOne . Read this

Answer 2

Check your "model" file to see if the "User" model exists there. Also check if you are using the 'module.exports'

Answer 3

You are exporting the User model directly, but calling it like it is in an object property named User . You can either access it directly, changing:

models.User.findOne

to:

models.findOne

and then you'd probably want to rename models to User .

Or change your export to:

module.exports = { User: user };

Answer 4

You are setting the user variable to be the export of the file

module.exports = user

You then import the user variable as models .

const models = require("../models/user")

this means that you do not need to access user as a property. Instead use:

models.findOne({ // Changed from models.User to models
    attributes: ["email"],
    where: {
        email: email,
    },
});

This should stop your current error, but you will keep on getting errors until you change all instances of models.User to models .

Your main file should end up looking like this:

 // Importation : // Bcrypt: const bcrypt = require("bcrypt"); // Jsonwebtoken d'authentification: const jwt = require("jsonwebtoken"); // Import du models user: const models = require("../models/user"); ////////////////////////////////////////////////////////////////////////////////////////////// // Fonction/ // Incription: exports.signup = (req, res) => { const username = req.body.username; const email = req.body.email; const password = req.body.password; const bio = req.body.bio; const admin = req.body.admin; console.log(req.body); try { models .findOne({ attributes: ["email"], where: { email: email, }, }) .then( ((userFound) => { if (!userFound) { bcrypt.hash(password, 10, function (err, bcryptPassword) { const newUser = models .create({ username: username, email: email, password: bcryptPassword, bio: bio, admin: false, }) .then((newUser) => { res.status(201).json({ userId: newUser.id, }); }) .catch((err) => { res.status(500).json({ error: "Impossible d'ajouter un utilisateur", }); }); }); } else { return res.status(409).json({ error: "Ce compte existe déjà ", }); } }).catch((err) => res.status(500).json({ err: err + "Impossible de vérifier l'utilisateur", }) ) ); } catch (error) { res.status(400).json({ error: error.message, }); } };