Во первых не понимаю почему Вы пишете
splice(index-1, 1)
- Так вы удаляете элемент, предшествующий тому на котором был клик.
А если кликать по нулевому то удалится элемент с конца (
Первый аргумент splice).
Во вторых дело в том - что Вы жёстко закрепляете индекс у элементов. Т.е:
Есть 10 элементов с индексами 0,1,2,3,4,5,6,7,8,9
Вы удаляете 0 элемент и теперь массив у Вас на 9(Индексы с 0 по 8) элементов
Теперь мы пробуем удалить элемент с индексом 9 и понимаем что нету у нас в массиве такого. Он стал 8ым
Конечно такой splice не сработает и элементов в массиве меньше не станет.
Более того после первого удаления, когда массив сдвинется Вы уже не сможете понять что именно удалилось.