的Javascript返回未打破的功能

Question

我有一個JavaScript函數，它檢查是否在一個XML文件中存在的藝術家：

function artistExists(artist) { 
// get data from artists.xml 
$('.loading').show(); 
$.get(artists_xml, function(xml){ 
    $('.loading').hide(); 
    $(xml).find('artist').each(function(){ 
     if ($(this).find("ar_artist").text() == artist.val()) { 
      alert ('artist exists'); 
      return true; 
     } //end if 
    }); // end each 
    alert ('artist does not exist'); 
    return false; 
}); // end .get function 
} // end of artistExists function 

我是正確的思維是，「返回true」行應該退出執行的功能？我認爲它會，但是在找到一條記錄並運行第一個警報執行後，繼續執行底部的故障警報。

我在做什麼錯誤？謝謝。

Answer 1

返回false而不是true，終止each循環;從the docs：

我們可以通過返回false來停止回​​調函數中的循環。

這隻會終止您的each循環，但不是整體功能。你需要設置一個標誌，以便你知道你是否找到了某些東西，例如像這樣：

function artistExists(artist) { 
// get data from artists.xml 
$('.loading').show(); 
$.get(artists_xml, function(xml){ 
    var found = false; // <== Added 
    $('.loading').hide(); 
    $(xml).find('artist').each(function(){ 
     if ($(this).find("ar_artist").text() == artist.val()) { 
      alert ('artist exists'); 
      found = true; // <== Added 
      return false; // <== Modified 
     } //end if 
    }); // end each 
    if (!found) {   // <== Added 
     alert ('artist does not exist'); 
    }      // <== Added 
    return found;   // <== Modified 
}); // end .get function 
} // end of artistExists function 

Answer 2

是的，它會「退出」執行該功能。問題是，「哪個功能？」在這種情況下，答案應該很清楚：這是傳遞給.each()的函數。

您可以通過返回false而不是true來終止.each()的循環行爲，但仍不會讓您脫離外部函數。你應該考慮的是在外部函數中設置一個局部變量，並設置內部函數，當它找到某些東西（然後打破.each()循環）。然後主函數可以檢查局部變量是否被設置。

這是一個我真的很喜歡用.reduce()或.inject() API的例子，但jQuery沒有它，他們真的反對它。

Answer 3

$.get是一個異步函數，這意味着主函數artistExists將立即返回，並且將發起GET請求。爲了能夠得到結果，你需要一個回調。

function artistExists(artist, cb) { 
    $('.loading').show(); 
    $.get(artists_xml, function(xml) { 

     var found = false; 

     $('.loading').hide(); 

     $(xml).find('artist').each(function(){ 
      if ($(this).find("ar_artist").text() == artist.val()) { 
       found = true; 
       return false; // use return false to stop .each() 
      } 
     }); 

     // the built in action. 
     if (found) { 
      alert ('artist exists'); 
     } else { 
      alert ('artist does not exist'); 
     } 

     // call the callback function 
     cb (found); 

    }); 
} 

然後使用，您需要使用回調函數。從

var isExists = artistExists('lol'); 
// do stuff 

您需要將其更改爲：

artistExists('lol', function(isExists) { 
    // do stuff 
}); 

Answer 4

感謝所有的建議。最後，我決定我需要一個同步調用，所以我做了不用彷徨功能的以下新版本，叫做.sget：

jQuery.extend({ 
sget: function(url, callback, type) { 
     return jQuery.ajax({ 
      type:  "GET", 
      url:  url, 
      success: callback, 
      async:  false, 
      dataType: type 
     }); 
    } 
}); 

的「異步：假」對在「AJAX」選項使呼叫同步。然後我的原始失敗的功​​能如下編輯：

function artistExists(artistname) { 
var found = false; 
console.log("From Input:Artist= " + artistname.val()); 
// get data from artists.xml 
$('.loading').show(); 
$.sget(artists_xml, function(xml){ // new synchronous get 
    $('.loading').hide(); 
    $(xml).find('artist').each(function(){ 
     if ($(this).find("ar_artist").text() == artistname.val()) { 
      console.log('From File:Artist= ' + $(this).find("ar_artist").text()); 
      found = true; 
      console.log("In each loop:Flag= " + found); 
      return; 
     } //end if 
    }); // end each 
}); // end .get function 
console.log("At end:Flag= " + found); 
return found; 

}

的執行console.log線將被刪除。然而，他們表明，事情正在按照我想要的順序發生。因此，如上所述，新的同步.sget函數和'找到'標誌的使用爲我完成了訣竅。不知道爲什麼我不能想到昨天這樣做。

謝謝大家。