Trying to get sqlite data using node

Question

I'm running into a small problem while trying to get data out of a sqlite database to a string. Here is the code:

router.get('/', function(req, res) {
    var db = req.db;
    var thisShouldWork = 'Hmmm';

    db.all('select rowid as id, value from test', function(err, rows) {
        rows.forEach(function(row){
            console.log(row.id + ": " + row.value);
            thisShouldWork += 'Heeee';
        });
    });
    thisShouldWork += 'What?';
    console.log(thisShouldWork.toString());

    res.send(thisShouldWork);
});

The variable 'thisShouldWork' just outputs 'HmmmWhat?' at the end of this code, while it should have a few 'Heeee's in there as well. Also, console.log prints 3 lines of data so the for loop is definitely executing. Am I doing something wrong without realizing or is there a different/better way to achieve the same thing?

CL. · Accepted Answer · 2014-11-23 21:30:13Z

1

The callback is executed asynchronously.

Whatever you want to do afterwards must be moved at the end of the callback:

router.get('/', function(req, res) {
    var db = req.db;
    var thisShouldWork = 'Hmmm';

    db.all('select rowid as id, value from test', function(err, rows) {
        rows.forEach(function(row){
            console.log(row.id + ": " + row.value);
            thisShouldWork += 'Heeee';
        });
        thisShouldWork += 'What?';
        console.log(thisShouldWork.toString());

        res.send(thisShouldWork);
    });
});

answered Nov 23, 2014 at 21:30

CL.

182k18 gold badges241 silver badges282 bronze badges

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

1 Comment

afroze9 Over a year ago

Hi, thanks for the answer! I just tried it and its working perfectly now.

Collectives™ on Stack Overflow

Trying to get sqlite data using node

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related