Node js execute another function after complete multiple function in loop node js

Question

My code is below-:

function get_btc(address) {
    address_transaction(address, user_id, coin_key, deposite_txn_fee, function (callback) {
        for (var j = 0; j < callback.response.data.txs.length; j++) {
            let user_id = callback.user_id;
            //some code//
        }
    });
}

get_label_info(function (err, data) {
    var promise = [];
    for (var i = 0; i < data.length; i++) {
        let address = data[i].address;
        var test_function = function (done) {
            deposite_model.get_coin_info(function (err, data1) {
                var coin_name = data1[0].coin_code;
                const return_functions = get_switch(coin_name);
                if (return_functions) {
                    obj[return_functions](address);
                }
                done(err, data1);
            });
        };
        promise.push(test_function);
    }
    sample();
});

function sample() {
    console.log('all functions has been completed');
}

By the help of above mentioned code i want to excecute all_completed loop when all functions has been completly done. At the initial start get_label_info function is excuted then controller go on to get_btc function.

Please help me how could i run all_completed functions after all functions completed run.

you are only pushing a single function into the promise array, as the .push is outside of the loop. — George
– George, Commented Mar 20, 2018 at 5:35
then controller go on to get_btc function - no, you don't call get_btc anywhere in your code — Jaromanda X
– Jaromanda X, Commented Mar 20, 2018 at 5:38

Rahul Sharma · Accepted Answer · 2018-03-20 05:42:58Z

1

Try this,

Define test_function outside for loop, It's not the good approach to declare inside the loop.

var test_function = function (address) {
    return new Promise((resolve, reject) => {
        deposite_model.get_coin_info(function (err, data1) {
            if (err)
                return reject(err);
            var coin_name = data1[0].coin_code;
            const return_functions = get_switch(coin_name);
            if (return_functions) {
                obj[return_functions](address);
            }
            return resolve(data1);
        })
    })
}

get_label_info(function (err, data) {
    var promises = [];
    for (var i = 0; i < data.length; i++) {
        promises.push(test_function(address));
    }

    Promise.all(promises).then((data) => {
        sample();
    })
});


function sample() {
    console.log('all functions has been completed');
}

edited Mar 20, 2018 at 5:42

answered Mar 20, 2018 at 5:37

Rahul Sharma

10.1k1 gold badge20 silver badges41 bronze badges

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

9 Comments

Sarus Crane Over a year ago

its works but what happen when controll go into get_btc function

Rahul Sharma Over a year ago

It depends on get_btc content If there is no async call inside that no issue.

Sarus Crane Over a year ago

Sarus Crane Over a year ago

Sarus Crane Over a year ago

cron_model.get_label_info(function(err,data){ if(err){ throw err; }else{ var promises=[]; for (var i=0;i<data.length; i++) { let address=data[i].address; let user_id=data[i].user_id; let coin_key=data[i].coin_key; promises.push(test_function(address,user_id,coin_key)); } Promise.all(promises).then((data)=>{ sample(); }); } }); function sample() { console.log('all functions has been completed'); }

|

Collectives™ on Stack Overflow

Node js execute another function after complete multiple function in loop node js

1 Answer 1

9 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

9 Comments

Your Answer

Sign up or log in

Post as a guest

Related