Почему .splice всегда удаляет последний элемент?

Question

В javascript есть два массива: tags[] и tags_java[]. Я использую .splice для удаления определенных элементов, которые имеют один и тот же индекс в двух массивах. tags[] отлично работает, но tags_java нет, кажется, всегда удаляется последний элемент.
Вот код и the jsfiddle link.

var tag = $(this).text(); 
var index = $.inArray(tag, tags); 
    tags.splice(index,1); 
    tags_java.splice(index,1); 

Answer 1

Nah, оба не работают, потому что вы на самом деле не находите правильный индекс своего тега.

Почему нет? Поскольку $(this).text() включает добавленную вами метку удаления, × - например. "Утро ×". Так как это не в вашем массиве tags, index будет -1. tags.splice(-1, 1); удалит 1 элемент из конца массива.

В общем, никогда не рекомендуется использовать текст презентации (т. Е. Текст вашего элемента тега) в качестве данных (например, используя этот текст в качестве значения поиска в массиве). Очень вероятно, что он будет сломан, когда что-то изменится в презентации - как здесь. Таким образом, было бы предложение сохранить данные (что вам нужно искать в тегах) как данные - например. используя предоставленный jQuery API data() - даже если он кажется излишним.

Вот быстрый пример - просто добавление/замена двух линий, которые я пометил с комментариями, начиная с «JT»: JSFiddle

Теперь, вместо того, чтобы смотреть вверх $(this).text(), мы смотрим вверх значение данных «tagValue», хранящееся с $(this).data() - таким образом, значение поиска по-прежнему привязано к элементу, но мы не полагаемся на текст презентации.

Answer 2

Если tag не в tags массиве, $.inArray вернется -1, который затем вызывает последний элемент будет удален.

You have to make sure that the item is actually in the array.