The amazeui-areaselect from czl032405

amazeui-areaselect's People

Contributors

Stargazers

Watchers

amazeui-areaselect's Issues

for in的一个bug

我说的这个forin循环处理object的bug可能大部分时候不存在，但是一出现就会在选项中增加一个属性

具体是怎么个问题我截图

一共19个元素结果输出了20个适应某个js或者浏览器在处理object的时候给对象加了一些属性。

具体问题代码为

{
            key: "buildList",
            value: function buildList(subArea) {
                var plugin = this;
                var options = this.options;
                var $list = null;
                //查找已经存在的list
                $(".am-areaselect-list", this.$popup).not("am-tmpl").each(function (idx, ele) {
                    if ($(ele).data('subArea') == subArea) {
                        $list = $(ele);
                    }
                });
                //构建list
                if (!$list) {
                    $list = $(".am-areaselect-list.am-tmpl", this.$popup).clone().removeClass("am-tmpl am-hide").data('subArea', subArea);
                    $list.appendTo($(".am-areaselect-lists", this.$popup));
                    for (var i in subArea) {
                        var $item = $(".am-areaselect-item.am-tmpl", $list).clone().removeClass("am-tmpl am-hide").text(subArea[i][options.areaNameKey]).data('value', subArea[i][options.areaValueKey]);
                        $item.appendTo($list);
                        (function (_i) {
                            $item.on("click", function (e) {
                                e.preventDefault();
                                if ($(this).hasClass("am-btn-primary") || $(this).siblings().hasClass("am-btn-primary")) {
                                    plugin.selectedArea.pop();
                                }
                                $(this).addClass("am-btn-primary").siblings().removeClass("am-btn-primary");
                                plugin.select(subArea[_i]);
                            });
                        })(i);
                    }
                }
                $list.siblings(".am-areaselect-list").addClass("am-hide");
            }
        }

将for in 改成$.map就可以很好的解决这个问题，但是这又引入了jquery依赖，我不知道是否有更好的不依赖jquery的方式，所以没有提交pr。