Async.waterfall、foreach 和解析 JSON node.js。需要一些建议答案

【问题标题】：Async.waterfall, foreach and parsing JSON node.js. Need some advicesAsync.waterfall、foreach 和解析 JSON node.js。需要一些建议
【发布时间】：2013-11-02 12:06:38
【问题描述】：

我的任务是解析最近 100 场 dota2 游戏。我使用来自 jiiin (https://github.com/jiin/dota2api) 的 dota 2 API 和小库

我所有的，这是一个 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 场比赛，但我需要在 request.params.id 中使用带有 game_id 的“forEach”获取详细信息，使用 player_id 使用“forEach”获取当前玩家的详细信息。毕竟，我再次将 forEach 用于汇总数据。这是正确的方式？）还是我很愚蠢？）

您是否有一些想法可以更快地做到这一点？可能需要使用 MongoDB (?) 或其他东西。

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

谢谢！

【问题讨论】：

标签： json node.js mongodb foreach

【解决方案1】：

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

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

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

【讨论】：