如何提高我的代碼來實現'返回false'？

Question

我想在條件語句失敗後返回false。

我

$('#btn').click(function() { 
    $('.title').each(function() { 
     if (id == $(this).attr('id')) { 
      alert('The name already exists.') 
      return false; //I hope my codes would stop here if condition is true 
     } 
    }) 
    // my codes still call the doSomething function even if the conditional   
    //statement is true 
    doSomething(); 
})​ 

我想你所說的doSomething函數僅如果跌破id != $(this).attr('id).

的代碼給我我想要什麼，但它似乎醜陋。

$('#btn').click(function() { 
    var nameExist = false 
    $('.title').each(function() { 
     if (id == $(this).attr('id')) { 
      alert('The name already exists.') 
      nameExist = true; 
      return false; //I hope my codes would stop here if condition is true 
     } 
    }) 
    if (!nameExist) { 
     doSomething(); 
    } 
})​ 

任何人都有更好的方法呢？非常感謝！

Answer 1

你並不需要遍歷thrue元素，您可以通過id和class讓它像， #myId.myClass。

$('#btn').click(function() { 
    if($('#' + id + '.title').length) { 
     alert('The name already exists.'); 
    } else { 
     doSomething(); 
    } 
}); 

Answer 2

我覺得你有什麼是好的，但這樣就避免了額外的條件：

var func = doSomething; 
... 
if (id == $(this).attr('id')) { 
    func = $.noop; 
... 

func(); 

Answer 3

如果你不介意不及早退出循環，你可以使用jQuery filter

$('#btn').click(function(){ 
    var itensWithSameName = $('.title').filter(function(){ 
     return id == $(this).attr('id'); 
    }) 

    if(itensWithSameName.size() > 0) 
     alert('The name already exists.'); 
}); 

Answer 4

切換到一個基本的for循環。

$('#btn').click(function() { 
    var elements = $(".title"); 
    for (var i = 0; i < elements.length; i++) { 
     if (id == elements[i].id) { 
      alert('The name already exists.') 
      return false; //I hope my codes would stop here if condition is true 
     } 
    } 

    doSomething(); 
})​