Async.waterfall， foreach 和解析 JSON Node.js.需要一些建议

Question

我的任务是解析最后 100 场 dota2 游戏。我使用dota 2 API和jiiin的小库（https://github.com/jiin/dota2api）

我所拥有的，这是一个async.waterfall调用。喜欢这个：

exports.get = function(req, res, next) {
var playerID = +req.params.id;
var playerCounter = [];
var playerInfo = {
    kills: [],
    deaths: [],
    assists: [],
    last_hits: [],
    denies: [],
    hero_damage: [],
    hero_healing: [],
    gold_spent: [],
    kills_number: 0,
    deaths_number: 0,
    assists_number: 0
};
async.waterfall([
    function getDota2Json(callback) {
        dota.getByAccountID(playerID, function (err, result) {
            callback(err, result);
        });
    },
    function getMatches(result, callback) {
        result.matches.forEach(function (match) {
            callback(null, match.match_id);
        });
    },
    function getMatchInfo(matchID, callback) {
        dota.getMatchDetails(matchID, function (err, result) {
            callback(err, result.players);
        });
    },
    function getCurrentPlayer(players, callback) {
        players.forEach(function (player) {
            if (player.account_id === playerID) {
                callback(null, player);
            }
        });
    },
    function getDamage(player, callback) {
        callback(null,
                player.kills,
                player.deaths,
                player.assists,
                player.last_hits,
                player.denies,
                player.hero_damage,
                player.hero_healing,
                player.gold_spent,
                '1');
    }
], function (err, kills, deaths, assists, last_hits, denies, hero_damage, hero_healing, gold_spent, counter) {
    playerCounter.push(counter);
    playerInfo.kills.push(kills);
    playerInfo.deaths.push(deaths);
    playerInfo.assists.push(assists);
    console.log(playerCounter.length);
    if (playerCounter.length === 100) {
        playerInfo.kills.forEach(function (val) {
            playerInfo.kills_number += val;
        });
        console.log('Last 100 K ' + playerInfo.kills_number);
        playerInfo.deaths.forEach(function (val) {
            playerInfo.deaths_number += val;
        });
        console.log('Last 100 D ' + playerInfo.deaths_number);
        playerInfo.assists.forEach(function (val) {
            playerInfo.assists_number += val;
        });
        console.log('Last 100 A ' + playerInfo.assists_number);

        var magickOpts = [
            "-background", "grey60",
            "-bordercolor", "snow",
            "-border", "6",
            "-fill", "black",
            "-pointsize", "50",
            "label: Dota 2 LAST 100 n K - D - A " + playerInfo.kills_number + ' - ' + playerInfo.deaths_number + ' - ' + playerInfo.assists_number,
            ""+playerID+".png"
        ];
        var im = spawn('convert', magickOpts);
    }
});
res.end('rdy');

};

我认为这不是好的代码，但我找不到替代方案。Dota API 在 JSON 中为我提供了 100 款游戏，但我需要用 game_id "forEach"了解详细信息，player_id "forEach"了解当前玩家的详细信息 request.params.id。毕竟，我再次使用 forEach 作为摘要数据。这是正确的方式？还是我傻了？

也许您有一些想法可以更快地做到这一点？可能需要使用MongoDB（？）或其他东西。

最后，我生成带有统计信息的简单图像。

谢谢！

Answer 1

我觉得你可以使用async.each来简化你的一些forEach调用...

这可能是一个小的吹毛求疵，但要获得数字数组的总和，您可以使用数组的reduce方法。

playerInfo.kills_number = playerInfo.kills.reduce(function(a, b) {
  return a + b;
});