1

Добавляю в массив элементы страницы (td), вывожу(console.log(arr);)(пример):

(2) [jQuery.fn.init(1), jQuery.fn.init(1)]
   0: jQuery.fn.init [tr.1539953619.hovered.on.used]
   1: jQuery.fn.init [tr.1539949435.hovered.on.used]
   length: 2
   __proto__: Array(0)

то есть, элементы добавляются правильно, дальше когда использую поиск элементов:

console.log(arr.indexOf($(this))); //$(this) потому что при клике если элемент не выбран (отсутствует класс used) добавляет элемент (arr[list.length] = $(this);) если нет - должно выводить его индекс

всегда выводит -1, хотя элементы там есть. Почему может не работать и как исправить?

5
  • list и arr - это, видимо, одно и то же? В jQuery индекс ищется с помощью метода index
    – Regent
    19 окт 2018 в 14:28
  • @Regent, это другой index :)
    – Grundy
    19 окт 2018 в 14:29
  • @Grundy мм, другой - это какой? Я написал из расчета, что вместо массива jQuery объектов стоит использовать один, содержащий все элементы, после чего спокойно использовать .index
    – Regent
    19 окт 2018 в 14:41
  • @Regent да, это один и тот же массив, просто не повсюду заменил на arr) чтоб тут наглядней было 19 окт 2018 в 14:42
  • @shumik_UA если нужно обязательно через список/массив (например, если он часто читается), то я за вариант из ответа Igor. Если список нужен всего один или несколько раз, то, в качестве альтернативы, факт нажатия можно хранить в самом элементе data-атрибутом, а при получении списка брать все элементы и отфильтровывать ненужные по значению data-атрибута
    – Regent
    19 окт 2018 в 14:56

2 ответа 2

1

$(this) - вызов функции $/jQuery, который возвращает новый/свежесозданный объект-обертку вокруг набора DOM-элементов, в данном случае - одного элемента this. Такого объекта-ссылки не может существовать в массиве.


Храните в массиве ссылки на DOM-элементы:

list.push(document.getElementById("test"));
...
console.log(list.indexOf(this));
8
  • При первом нажатии на элемент - он добавляется в массив, при повторном нажатии - удаляется. При добавлении берется $(this) и при удалении тоже 19 окт 2018 в 14:31
  • @shumik_UA Это Вы с кем разговариваете? Это описание чего?
    – user176262
    19 окт 2018 в 14:32
  • ну с вами, объясняю что хочу сделать. Есть массив, и в нем выбранные элементы - как выбрать - первый раз нажал - выбрал - добавился в массив, второй раз нажал - удалил из массива, элемент не выбран 19 окт 2018 в 14:35
  • а разве document.getElementById("test") не равно в jquery $(this) (если имеется ввиду один и тот же элемент) 19 окт 2018 в 14:40
  • @shumik_UA Нет, не равно. Попробуйте еще раз медленно прочитать мой ответ ).
    – user176262
    19 окт 2018 в 14:42
1

Если вы пишете с помощью jQuery, то не нужно смешивать его с чистым js. Для сравление jQuery элементов можно использовать is:

element1.is(element2)

Тогда для проверки есть ли элемент в массиве можно использовать что-то типа

arrayItems.some((element) => { return element.is(element2); }

Где element - это массива, а element2 - то что вы пытаетесь найти в массиве

А для того, чтобы найти элемент в массиве

let founded = arrayItems.filter((element) {
  return element.is(element2);
});

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.