app.post('/register', function(req,res){ console.log("register post got", req.body) if (req.body.username && req.body.password) { db.get("SELECT * FROM Users WHERE username = '" + req.body.username + "' LIMIT 1"); res.end("Account already exists"); var stmt = db.prepare("INSERT INTO user VALUES (?,?)"); stmt.run(req.body.username, req.body.password); res.end("OK"); } else { res.end("username and password are requesiraehri"); } });¿Qué está mal aquí? Si estoy tratando de registrar un usuario, dice que ya existe mientras que la base de datos está completamente vacía. ¿Alguna ayuda?
En todo lo que sigue, supongo que está utilizando esta biblioteca sqlite: https://www.npmjs.com/package/sqlite3
res.send("Account already exists"); se llama incondicionalmente. Olvidó verificar el resultado de su consulta.
Pero hay otros problemas en su código. En primer lugar, no está utilizando funciones asíncronas con devoluciones de llamada. db.get es una función asíncrona y toma una devolución de llamada como segundo argumento, que recibe la primera fila del conjunto de resultados o un error ( consulte la documentación ).
app.post('/register', function(req,res){ console.log("got register post", req.body) if (req.body.username && req.body.password) { db.get("SELECT * FROM Users WHERE username = ? LIMIT 1", req.body.username, function(err, row){ if(row){ res.end("Account already exists"); } else { var stmt = db.prepare("INSERT INTO user VALUES (?,?)"); stmt.run(req.body.username, req.body.password); res.end("OK"); } } } else { res.end("username and password are required"); } }); stmt.run también es asíncrono. Así que tenga en cuenta que res.end('OK') se ejecuta antes de que los datos se inserten en la base de datos. Probablemente sería mejor verificar si la declaración de inserción se ejecuta correctamente antes de enviar la respuesta.
Otro tema importante es que insertes la contraseña en claro en la base de datos. Deberías pensar en encriptarlo. ¡ Nunca almacene contraseñas en texto claro!