为什么这段代码总是返回“no_conflict”？ [复制]

Question

我写了一个方法如下：

detectNameConflict: function() {
    var existing_filenames = this.element.find('ul.existing_files > li');
    if (existing_filenames.length > 0) {
        var try_name = this.element.find('div.target_filename').text().trim();
        existing_filenames.each(function(index, el) {
            if ($(el).text() == try_name) {
                return "contain_conflict";
            }
        });
    } else {
      return "no_conflict";
    }
},

此代码不起作用，因为它总是返回“no_conflict”，即使存在命名冲突。

注意：this.element 来自 jQueryUI 小部件工厂。它指的是小部件实例所附加的 DOM 元素。

Answer 1

您可以将 jQuery 集合转换为数组，然后使用 Javascript 的some() 方法来测试它们是否与try_name 匹配。

detectNameConflict: function() {
    var try_name = this.element.find('div.target_filename').text().trim();
    var existing_filenames = this.element.find('ul.existing_files > li').toArray();
    if (existing_filenames.some(function(el) {
        return $(el).text() == try_name;
    })) {
        return "contain_conflict";
    } else {
        return "no_conflict";
    }
}

Answer 2

"contain_conflict" 的 return 语句退出谓词，而不是整个过程，因此每个在检测到冲突后都会继续循环。您需要使用for 循环或基于异常的控制流（因为JavaScript 缺少goto）。

更新

Barman 在上面的 cmets 中指出，您可以通过从谓词中返回 false 来停止循环。更喜欢例外；当我写这个答案时，我不知道这个功能。

结束更新

一个for循环：

detectNameConflict: function()
{
    var existing_filenames = this.element.find('ul.existing_files > li');
    if (existing_filenames.length > 0)
    {
        var try_name = this.element.find('div.target_filename').text().trim();
        for(var filename in existing_filenames)
        {
            if ($(filename).text() == try_name)
            {
                return "contain_conflict";
            }
         }
    }
    return "no_conflict";
}

基于异常的流程：

detectNameConflict: function()
{
    var existing_filenames = this.element.find('ul.existing_files > li'); 
    if (existing_filenames.length > 0)
    {
        var try_name = this.element.find('div.target_filename').text().trim();
        try
        {
            existing_filenames.each(function(index, el)
                {
                    if ($(el).text() == try_name)
                    {
                        throw "contain_conflict";
                    }
                });
        }
        catch(e)
        {
            return e;
        }
    }
    return "no_conflict";
}