Running a function in a for Node.js

Asked May 28 '18 at 14:53

Active May 28 '18 at 14:53

Viewed 17 times

I'm trying to run a function in a for loop, the problem is that it takes only the last 'key' from the loop and runs the code for it.

Here's my code:

/*Global variable*/
var PItem = "Item";
for(var key in Prices){
    PItem = key;
    var sql = 'SELECT EXISTS(SELECT * FROM `items` WHERE `name` = "'+PItem+'")';
    con.query(sql , function(err, result){
        if(err) return console.log(err);
        for (var key6 in result){
            for(var key7 in result[key6]) var result2 = result[key6][key7];
        }
        if(result2 == 0){
            var sql = 'INSERT INTO `items` (`name`, `buy`, `sell`, `stock`, `maxStock`) VALUES ("'+PItem+'", '+Prices[PItem].buy+', '+Prices[PItem].sell+', '+Prices[PItem].stock+', '+Prices[PItem].maxstock+')';
            con.query(sql, function(err, result){
                if(err) return console.log(err);
            });
        }
    });
}

It adds to the MySql database several rows with the same values of the last 'key' from the loop.

asked May 28 '18 at 14:53

Nikita Ivanov

Don't use `var`, use `let`. – Marcos Casagrande May 28 '18 at 14:54
@MarcosCasagrande , Could you please explain me what it means? the 'let' variable? – Nikita Ivanov May 28 '18 at 14:57
Check the question I linked, and https://stackoverflow.com/questions/762011/whats-the-difference-between-using-let-and-var-to-declare-a-variable-in-jav – Marcos Casagrande May 28 '18 at 14:58
@MarcosCasagrande I've changed 'var' to 'let' for both, PItem and key, didn't work for me.. – Nikita Ivanov May 28 '18 at 15:04
1

You're not using `key`, you're using `PItem` which is declared on the outer scope, remove `PItem`, just use `key`. – Marcos Casagrande May 28 '18 at 15:07
@MarcosCasagrande Thank you very much! – Nikita Ivanov May 28 '18 at 15:11

Running a function in a for Node.js

0 Answers0