NodeJS -> Error Cannot read property 'length' of undefined

Question

I wanted to made a query to the mysql database if the username already exists he should throw me a message and if not, show me the new id. I wanted to check if the query result.lenght is higher than 0. however it throws the error "Cannot read property 'length' of undefined"

Service file

const connect = () =>
    mysql.createConnection({
        host: process.env.MYSQL_HOST,
        user: process.env.MYSQL_USER,
        password: process.env.MYSQL_PASSWORD,
        database: process.env.MYSQL_DATABASE
    });
const insert = (conn, user) => {
    return conn.query(`insert into user(username, firstName, lastName, password)   
    values(?,?,?,?)`,
        [
            user.username,
            user.firstName,
            user.lastName,
            user.password
        ]
    )
}
const getByUsername = username => {
    return connect()
        .then(conn => {
            conn.query( 
                `select id, username, firstName, lastName
                from user
                where username = ?`,
                [username]  
            );
        })
}

const create = user => {

    return getByUsername(user.username)
        .then(result => {
            if (result.length > 0) {
                throw new Error(`User "${user.username}" already exists`);
            }
        })
        .then(() => connect())
        .then(conn => insert(conn, user))    
        .then(result =>
            ({
                id: result.insertId
            }));
};

Controller file

const create = (req, res) => {

    userService.create(req.body)
        .then(result => {
            res.status(200);
            res.json(result);
        })
        .catch(err => {
            res.status(500);
            res.send('Error ' + err.message);
        })
}

I tried to define lenght but I think this isnt the issue. I make the query with postman and i use mysql as database. I also tried to replace the result.length with username.length but it thorws "Error username is not defined"...

What is your npm mysql version?

Gokulakannan T
– Gokulakannan T

2020-02-18 12:45:00 +00:00
Commented Feb 18, 2020 at 12:45 — Gokulakannan T
– Gokulakannan T, Commented Feb 18, 2020 at 12:45
I got version 2.18.1

Dariun
– Dariun

2020-02-18 12:51:54 +00:00
Commented Feb 18, 2020 at 12:51 — Dariun
– Dariun, Commented Feb 18, 2020 at 12:51

Gokulakannan T · Accepted Answer · 2020-02-18 12:52:27Z

The result might be undefined there. Because we don't return proper query results. That why we getting error.

Solution:

const getByUsername = username => {
    return new Promise((resolve, reject) => {
        connect()
            .then(conn => {
                conn.query( 
                    `select id, username, firstName, lastName
                    from user
                    where username = ?`,
                    [username],
                    function(err, results) {
                        if (err) {
                            return reject(err);
                        }
                        return resolve(results)
                    }  
                );
            })
    });
}

Explanation:

As per mysql2 documentation, the conn.query will not return the results. The conn.query method has a third argument which is a callback method. Here we can get the SQL query error and query results.

By the use of javascript Promise API, we can decide to reject or resolve based upon the SQL query result.

Collectives™ on Stack Overflow

NodeJS -> Error Cannot read property 'length' of undefined

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related