jQuery renvoie la valeur true même si false avec la liste de sélection dupliquée

Question

Je suis assez nouveau pour jQuery. Et je suis coincé avec cela, j'essaie de trouver des doublons dans une zone de liste de sélection sur un clic de souris. Si vrai, il alertera l'utilisateur, s'il est faux, il ajoutera la nouvelle valeur et le texte. Cependant, la fonction findDuplicates (aVal) à l'appel renvoie toujours true. J'ai essayé d'entrer dans le code, la fonction elle-même fonctionne, mais quand elle est entrée dans le script jQuery (1.7.2), elle devient vraie. Peut-être que je fais quelque chose de mal? Merci d'avance!

Voici le code:

function findDuplicates(aVal) { 
     $("#listBoxAllergy option").each(function (i){ 
      var isDuplicate = false; 
      if ($.trim(aVal) == $(this).val()) { 
       isDuplicate = true; 
      } 
      return isDuplicate; 
     }); 
    }; 

    $("#btnAddAllergy").on("click", function() { 
     var allergy = $("#autoComAllergy").val(); 
     if (findDuplicates(allergy)) 
      alert("The allergy is already in the list"); 
     else 
      $("#listBoxAllergy").append("<option value='" + allergy + "'>" + allergy + "</option>"); 
    }); 

Answer 1

D'abord, il n'y a pas de méthode .value() dans jQuery.

.value() doit être .val().

Deuxièmement, vous ne devriez pas return isDuplicate; dans le rappel .each, déplacer l'extérieur.

function findDuplicates(aVal) { 
    var isDuplicate = false; 
    $("#listBoxAllergy option").each(function (i){ 
     if ($.trim(aVal) == $(this).val()) { 
      isDuplicate = true; 
      return false; // here return false let stop the each. 
     } 
    }); 
    return isDuplicate; 
}; 

Answer 2

Dans votre fonction de comparaison, je vais avec l'égalité stricte (===) au lieu d'égalité (==) bien que ce soit pas la source du problème.

Il n'y a pas de méthode value() dans jQuery, il est val() au lieu

Answer 3

function findDuplicates(aVal) { 
    var isDuplicate = false; 
    $("#listBoxAllergy option").each(function (i){    
     if ($.trim(aVal) == $(this).val()) { 
      isDuplicate = true; 
     }    
    }); 
    return isDuplicate; 
}; 

Essayez celui