Retrieve data from sqlite query Node.js

Question

I'm currently trying to retrieve data from a sqlite query in node.js, the sql function is on a diferent file so i'm exporting it as a module and then call the function from the index.js. But when i try to retrieve the data the function returns a null value.

Here is my code

Index.js

var express = require("express");
var body_parser = require("body-parser");
var app = express();
var db = require('./dbhandler.js');

app.set("view engine", "jade");

app.get("/data",function(req,res){
   let data = db.select();
   res.send(data);
});

app.get("/",function(req,res){
   res.render("index");
});

app.listen(8888);

dbhandler.js

var sqlite3 = require("sqlite3");

const file = "hr";

exports.select = function (){

var lista = [];

var db = new sqlite3.Database(file);

db.all("SELECT * FROM usuarios", function(err,rows){

    let contador = 0;

    rows.forEach(function (row) {
            lista[contador] = row.nombre + ";" + row.cedula + ";" + row.edad + ";" + row.pais;
    });
});

db.close();

return lista;
}

Robert Moskal · Accepted Answer · 2016-06-12 16:34:06Z

11

Node is asynchronous!!!. lista is returned from the module before the db.all function completes.

You either need to pass a callback into the select function or return a promise. The callback approach would look something like this:

 exports.select = function (cb){
    var lista = [];
    var db = new sqlite3.Database(file);           
    db.all("SELECT * FROM usuarios", function(err,rows){
         if(err) return cb(err);
         let contador = 0; 
         rows.forEach(function (row) { 
            lista[contador] = row.nombre + ";" + row.cedula + ";" + row.edad + ";"
    + row.pais; }); 
        db.close();
        return cb(null, lists);
}); 
 }

edited Jun 12, 2016 at 16:34

answered Jun 12, 2016 at 15:56

Robert Moskal

22.7k8 gold badges69 silver badges91 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

A.rodriguez Over a year ago

Thanks it worked. I started learning node about 3 days ago and never tried to learn about callbacks until now so i had no idea how to fix it.

Collectives™ on Stack Overflow

Retrieve data from sqlite query Node.js

1 Answer 1

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related