来自多个函数的回调

时间:2018-02-05 10:31:23

标签: node.js callback

我是nodejs的新手,我对回调很头疼。
我想编写一个应用程序,允许用户选择和查询服务器的某些设备状态(例如ping,cpu,内存等)。要查询的项目将通过URL发送到后端服务器。我使用for循环和一个开关来调用执行shell命令的函数(例如ssh,ping等)来获取服务器状态。最后我想把所有结果(res.write)写到前端 我没能使它工作,要么返回null,要么在结束前写入'错误...请帮助我。 这是我的代码,我试图简化它。

app.get('/check', function(req, res, next) {
    var selecteditems = req.query.selecteditems; //ping,memory,disks,etc. 
    checkcon(selecteditems, function(results) {
        res.write(JSON.stringify(results), function(err) {
            res.end();
        });
    });
});

var checkcon = function(selecteditems, callback) {
    var selecteditems = selecteditems.split(',');
    var allresults = [];
    selecteditems.forEach(function(entry) {
        switch (entry) {
            case "ping":
                checkping(function(results) {
                    allresults.push(results);
                });

                break;
            case "memory":
                checkmemory(function(results) {
                    allresults.push(results);
                });
                break;
            case "disks":
                checkdisks(function(results) {
                    allresults.push(results);
                });
                break;
            case "disks":
                break
            default:
        }
        callback(allresults);
    });
}

shell命令的一个函数示例:

var checkping = function(callback) {
    var pingstat = [];
        const child = exec('ping -c 1 -W 1 10.102.12.2',
            (error, stdout, stderr) => {
                if (error !== null) {
                    console.log(`exec error: ${error}`);
                    pingstat.push("Unable to ping machine");
                } else {
                    pingstat.push("OK");
                }
                callback(pingstat);

            });
}

2 个答案:

答案 0 :(得分:1)

您可以在async lib中使用此方法: https://caolan.github.io/async/docs.html#reduce

答案 1 :(得分:1)

使用async eachOf,以循环数组并对每个元素应用异步函数:

var selecteditems   = selecteditems.split(',');
var allresults      = [];

async.eachOf(selecteditems, function(item, index, cb){

     switch (item) {
        case "ping":
            checkping(function(results) {
                allresults.push(results);
                return cb();
            });

            break;
        case "memory":
            checkmemory(function(results) {
                allresults.push(results);
                return cb();
            });
            break;
        case "disks":
            checkdisks(function(results) {
                allresults.push(results);
                return cb();
            });
            break;
        case "disks":
            return cb();
            break
        default:
            return cb();
     }

}, function(err){
   if(err)
   {
        // do stg
    }
    else
    {
         callback(allresults);
    }
})