итерация по массиву в jquery

Question

Я пытаюсь сделать некоторые проверки для пользователя, который выбирает элементы из списка. Я хочу убедиться, что элемент не добавляется дважды, проверяя, находится ли <li> в массиве. Это то, что я пытаюсь и не работает.

$(".List").on("click", "li", function() { 
var i = 0; 
var checkArr = []; 

var div = $("#AddedItems"); 
var parent = $(this).closest("ul"); 
var itemtoadd = parent.find("[data-id]").attr("data-id"); 
var name = parent.find("[data-name]").attr("data-name"); 

alert(itemtoadd + name);//checking 

var itemtoadd = ("<li id = " + itemtoadd + " class = \"itemAdd\">" + name + "</li>"); 

checkArr.push(itemtoadd); //put one in to check against? 
checkArr.forEach(item) 
{ 
    if (item == itemtoadd) 
     alert("this item has already been added"); 
    else { 
     checkArr.push(itemtoadd); 
     alert(itemtoadd); 
     $(itemtoadd).appendTo(div); 
    } 
} 
// div.html(itemtoadd); 


}); 

Answer 1

У вас есть по крайней мере три проблемы здесь:

1. Вы не используете Array.forEach правильно - он принимает функцию, которая принимает элемент.
2. Сразу перед тем, как сделать чек, вы добавляете товар, который ищете. Вы всегда будете попадать в корпус alert.
3. Вы используете checkArr в качестве локальной переменной - вы получаете пустой массив при каждом входе в функцию.

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

if ($('#' + itemtoadd, div).length == 0) { 
    itemtoadd = ("<li id = " + itemtoadd + " class = \"itemAdd\">" + name + "</li>"); 
    div.append(itemtoadd); 
} 
else { 
    alert("this item has already been added"); 
}