Не работает определение индекса элемента в обработчике

@Rhoxolan · Регистрация: 16.12.2020

Author24 — интернет-сервис помощи студентам

Здравствуйте. Вот страница:

HTML5

<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <meta http-equiv="X-UA-Compatible" content="IE=edge">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>Star Rating</title>
    <link rel="stylesheet" href="/Task2/rating_stars_styles.css">
</head>
<body>
    <div id="centralDiv">
        <h2>
            Star Rating:
        </h2>
        <div class="starDiv"></div>
        <div class="starDiv"></div>
        <div class="starDiv"></div>
        <div class="starDiv"></div>
        <div class="starDiv"></div>
    </div>
    <script src="/jquery-3.6.3.js"></script>
    <script src="/Task2/rating_stars.js"></script>
</body>
</html>

Даны несколько дивов класса starDiv, которые описаны внутри дива centralDiv.
Текущая задача - с помощью jQuery определить, по какому по счету "starDiv" было произведено нажатие.

Написал этот обработчик, но он не работает, все время выводит -1:

Javascript

$(".starDiv").click(() => {
    var set = $('.starDiv');
    alert(set.index(this));
});

А этот обработчик я нашел в интернете, и он отлично работает:

Javascript

$('div').on('click', '.starDiv', function () {
    var set = $('.starDiv');
    alert(set.index(this));
});

Кто-то может объяснить, почему не работает мой вариант и почему работает вариант с интернета?
Что вижу я - в примере с интернета не используется метод click(), а используется метод on(). Как я знаю, on(), как и delegate() используется в тех случая, когда нужно добавить обработчики к новым элементам страницы, которых не было изначально. Но все мои дивы прописаны в разметке, никуда не удаляются и не добавляются. Так в чем причина?

@brian778 · 04.02.2023, 12:29

Если грубо говорить, то в первом случае вы ссылаетесь на

HTML5

1	<div class="starDiv">

Во втором, на

HTML5

1	<div id="centralDiv">

@Rhoxolan · 04.02.2023, 12:43 **[ТС]**

brian778, спасибо, да, это так.

Но, я только что разобрался - причина была в другом. В моём варианте "this" у меня почему-то представлялся окном, а не элементом, инициировавшим событие. Поменял на "$(e.target)" - все заработало.

Javascript

$(".starDiv").click((e) => {
    var set = $('.starDiv');
    alert(set.index($(e.target)));
});

@brian778 · 04.02.2023, 12:59

Сообщение от Rhoxolan

причина была в другом

Не прав был. Это изза Arrow function

Добавлено через 1 минуту
Пример

Javascript

$(".starDiv").click(function() {
    var set = $('.starDiv');
    alert(set.index(this));
});

@Rhoxolan · 04.02.2023, 13:03 **[ТС]**

brian778, Да, проверил, это действительно так, так тоже работает. Спасибо.

@xkkx · 07.02.2023, 17:08

Сообщение от brian778

Пример

Плохой пример

Javascript

1
2
3

$(".starDiv").click(function() {
    let index = $(this).index();
});

@brian778 · 07.02.2023, 17:12

xkkx, А я тут причём? Это же не мой код

@xkkx · 07.02.2023, 17:24

Сообщение от brian778

А я тут причём? Это же не мой код

Просто сказал)Вообще плохая затея таким способом на блоки ставить обработчики, говорю с большим опытом. Я на практике делаю вот так

PHP

$(document).on('click', '.starDiv', function() {
    let MainBlock =  $(this).closest('#centralDiv'); // лучше заменить на .centralDiv 
    let h2 = MainBlock.find('h2');
    let index = $(this).index();
    h2.html('Star Rating: ' + index);
})

Это хорошо помогает когда на странице много динамических элементов, и вечно обработчики обновлять не имеет никакого смысла. В добавок некоторые блоки имеют свойства на странице повторяться, и что бы не прописывать для каждого блока свой скрипт, легче делать всё в таком стиле

@Rhoxolan 13 / 13 / 7 Регистрация: 16.12.2020 Сообщений: 247
	04.02.2023, 13:03 [ТС]	5
	brian778, Да, проверил, это действительно так, так тоже работает. Спасибо. 0

@brian778 418 / 310 / 107 Регистрация: 30.08.2022 Сообщений: 1,195
	07.02.2023, 17:12	7
	xkkx, А я тут причём? Это же не мой код 1

	07.02.2023, 17:24

Не работает определение индекса элемента в обработчике

Решение