且构网

分享程序员开发的那些事...
且构网 - 分享程序员编程开发的那些事

环闭合 - 我在哪里可以把它?

更新时间:2022-12-08 13:20:14

感谢所有的反馈!我想我只是需要有人看着我的肩膀帮我看到明显的问题。下面是固定的code。与我的意见。

Thanks for all the feedback! I guess I just needed someone "looking over my shoulder" to help me see the obvious problems. Below is the fixed code with my comments.

$(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>");
            }
        }
    });
});