【问题标题】:Loop closure - Where do I put it?闭环 - 我把它放在哪里?
【发布时间】:2015-06-07 23:52:16
【问题描述】:

关于previous post,我很难确定在哪里放置一个循环闭包,用于匹配 JSON 返回的数组。

以下是我修改后的代码:

$(function()
{
    var $reports = $('#repOut');
    var techCount = 0;
    var repCount = 0;
    techs = ["Name_1", "Name_2"];

    function counts(tech, count)
    {
        this.tech = tech;
        this.count = count;
    }
    $.ajax(
    {
        type: 'GET',
        url: 'https://api.trello.com/1/board/BOARD_ID/checklists?checkItem_fields=name,state&key=MY_KEY&token=MY_TOKEN',
        dataType: 'jsonp',
        success: function(data)
        {
            $.each(data, function(i, repName)
            {
                var items = repName.checkItems;
                for (i = 0; i < items.length; i++)
                {
                    for (var n = 0; n < techs.length; n++)
                    {
                        techName = techs[n];
                    }
                    var rex = new RegExp(techName, "i");
                    var num = /\(\d+\)/;
                    if (rex.test(items[i].name))
                    {
                        var repFull = items[i].name;
                        repName = repFull.replace(/\.*$|-.*$/, "");
                        if (num.test(items[i].name))
                        {
                            var repNum = parseInt(/\d+/.exec(items[i].name), 10);
                            repCount += repNum;
                        }
                    }
                }
            });
            var techCount = new counts(techName, repCount);
            $reports.after("<table border =1 id='reports'><tr><th>Tech</th><th>Count</th></tr><tr><td>" + techCount.tech + "</td><td>" + techCount.count + "</td></td></tr></table>");
        }
    });
});

循环只返回 Name_2JSON 值(但这一次,它被正确添加并显示在我的网页上)。我需要在我的techs 数组中获取 EACH 名称的这些值并输出它们。我已经阅读了许多关于闭包的网站/文章/答案,但我无法理解将它放在哪里/如何将它放在我的代码中!

谁能帮助提供一个示例,说明我的代码在使用闭包时会是什么样子?我什至怀疑需要关闭是对的吗?

【问题讨论】:

  • 您缺少 techsitechNamerepNamevar 关键字,但在 irepName 的情况下,您正在覆盖您的函数参数。故意的?
  • 不,我认为不需要任何关闭。看起来你只是搞砸了循环结构。
  • 感谢格式化。你现在能自己发现错误吗?提示:techs 上的循环...
  • techs 循环的范围可能应该扩大 - 现在,它正在循环并在键 1 处停止,然后继续......?
  • 您总是要将 techName 设置为“Name_2”

标签: javascript arrays json closures


【解决方案1】:

感谢所有反馈!我想我只是需要有人“看着我的肩膀”来帮助我看到明显的问题。下面是我的 cmets 的固定代码。

$(function()
{
    var $reports = $('#repOut');
    var techCount = 0;
    var techs = ["Name_1", "Name_2"];

    function counts(tech, count)
    {
        this.tech = tech;
        this.count = count;
    }
    $.ajax(
    {
        type: 'GET',
        url: 'https://api.trello.com/1/board/BOARD_ID/checklists?checkItem_fields=name,state&key=KEY&token=TOKEN',
        dataType: 'jsonp',
        success: function(data)
        {
            for (var n = 0; n < techs.length; n++) //MOVED THE 'tech' LOOP UP HERE
            {
                var techName = techs[n]; //INCLUDING THE 'techName' VARIABLE DEFINITION
                var repCount = 0;
                $.each(data, function(_, repName)
                {
                    var items = repName.checkItems;
                    for (var i = 0; i < items.length; i++)
                    {
                        var rex = new RegExp(techName, "i");
                        var num = /\(\d+\)/;
                        if (rex.test(items[i].name))
                        {
                            var repFull = items[i].name;
                            var repFullName = repFull.replace(/\.*$|-.*$/, "");
                            if (num.test(items[i].name))
                            {
                                var repNum = parseInt(/\d+/.exec(items[i].name), 10);
                                repCount += repNum;
                            }
                        }
                    }
                })
                //MOVED MY OBJECT DECLARATION UP IN THE LOOP
                var techCount = new counts(techName, repCount); 
                $reports.after("<table border =1 id='reports'><tr><th>Tech</th><th>Count</th><tr><td>" + techCount.tech + "</td><td>" + techCount.count + "</td><br></td></tr></table>");
            }
        }
    });
});

【讨论】:

  • 太棒了!我做了一些进一步的调整。
  • 我还建议您省略多余的 var techCount = 0;var repFull = items[i].name; var repFullName = repFull.replace(/\.*$|-.*$/, ""); 声明,您实际上从未在任何地方使用过。
猜你喜欢
  • 1970-01-01
  • 1970-01-01
  • 2016-01-28
  • 2013-10-04
  • 1970-01-01
  • 2012-10-14
  • 2013-09-21
  • 2020-09-30
  • 1970-01-01
相关资源
最近更新 更多