Почему эта часть кода всегда возвращает «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" выходит предикат, а не процедура в целом, так что каждый продолжает цикл после того, как обнаруживаются конфликт. Вам нужно либо использовать цикл for, либо поток управления на основе исключений (поскольку JavaScript не хватает goto).

Update

Бармен указал в комментариях выше, что вы можете остановить зацикливание, возвращая false из вашего предиката. Предпочитайте исключения; Я не знал об этой функциональности, когда писал этот ответ.

End Update

для цикла:

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"; 
}