0

$arr - массив с четырьмя DOM элементами <a>, к которым будем применять hover, для прослушки событий при наведении на эти элементы.


Первый вариант:

for (var $i_nav = 0; $i_nav < 4 ; $i_nav++) {
    $arr[$i_nav] = $nav_conteiner.children('li').children('a').eq($i_nav).attr('id', 'id_nav' + $i_nav);

    $($arr[$i_nav]).hover(
         function () {

         },
         function () {    
         }
   );

}

Второй вариант:

$("#id_a_element_1").hover(
         function () {
         },
         function () {    
         }
   );

$("#id_a_element_2").hover(
         function () {
         },
         function () {    
         }
   );

$("#id_a_element_3").hover(
         function () {
         },
         function () {    
         }
   );

$("#id_a_element_4").hover(
         function () {
         },
         function () {    
         }
   );

Правильно ли будет использовать первый вариант? Не противоречит ли он высшим законам силы природы?

12
  • 3
    а если элементов будет 1000, представьте какой длины будет код из второго варианта, а так абсолютно все равно как вешать обработчик, результат будет один
    – ordman
    Commented 14 февр. 2016 в 6:39
  • 5
    У меня встречный вопрос... а почему используется запись id_a_element_1, id_a_element_2 и т.д.... если проще сделать данным элементам class Commented 14 февр. 2016 в 8:40
  • @Алексей Шиманский Проще с точки зрения производительности? Дело в том что их нужно уникально идентифицировать. Но, в первом примере,так идентифицировать мы не можем как во втором варианте, поэтому к первому варианту мы применяем фильтр, который я сюда не вставил. Фильтр помогает идентифицировать элемент. Commented 14 февр. 2016 в 18:01
  • @ЮрийСветлов проще в плане "никаких извращенских манипуляций".... согласитесь написать $(".elementsForHover").hover() гораздо проще,чем циклы с непонятным поиском городить или миллионы строк с идентификаторами......между прочим в цикле строка $arr[$i_nav] = $nav_conteiner.children('li').children('a').eq($i_nav).attr('id', 'id_nav' + $i_nav); совсем бессмысленна.. так как можно просто написать $('#id_a_element_' + i).hover(); если по порядку цикл...или сделать просто find один раз, а потом также через foerach опять же написать $('#id_a_element_' + i).hover();. Commented 14 февр. 2016 в 19:04
  • @Алексей Шиманский По большому счету использовать class и использовать как в примере for, где мы устанавливаем селектор hover , одинаковые операции, это да. Но во втором примере, если мы будем использовать один класс на все элементы, для установки селектора hover, нам придется еще писать фильтр что бы идентифицировать элемент. Commented 14 февр. 2016 в 19:30

1 ответ 1

1

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

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

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