Проблема с добавлением обработчика onclick к тегу гиперссылки внутри listitem

StackOverflow https://stackoverflow.com/questions/367752

  •  21-08-2019
  •  | 
  •  

Вопрос

Пытаюсь добавить обработчик onclick на мои вкладки и, похоже, не могу правильно выбрать DOM.Ребята, вы можете помочь?

  <div id="tabstrip">
    <ul>
      <li id="a" class="selected"><a href="#">A</a></li>
      <li id="b"><a href="#">B</a></li>
      <li id="b"><a href="#">C</a></li>
    </ul>
  </div>

function initTabStrip()
{
  var lis = document.getElementById('tabstrip').getElementsByTagName('li');
  for (var i=0;i<items.length;i++)
  {
    var as = items[i].getElementsByTagName('a');
    for (var j=0;j<as.length;j++)
    {
      as[j].onclick=function(){changeTab(items[i].id);return false}
    }
  }
}
Это было полезно?

Решение

Похоже, ваше заключение неверно.Попробуй

as[j].onclick = function(items, i)
{
    return function()
    {
        changeTab(items[i].id);
        return false;
    };
}(items, i);

Если это работает, то вопрос является обманом Замыкания, циклы и события jQuery

Другие советы

Я согласен с RoBorg и предлагаю почитать о областях применения JavaScript.Это многое объяснит.

Первую строку вашего JS вы создаете var lis но затем вы повторяете дальше items.Это действительно то , чего ты хочешь ?

полная функция с изменениями, предложенными Робергом:

function initTabStrip()
{
  var tabstrip = document.getElementById('tabstrip');
  var items = tabstrip.getElementsByTagName('li');
  for (var i=0;i<items.length;i++)
  {
    var as = items[i].getElementsByTagName('a');
    for (var j=0;j<as.length;j++)
    {
      as[j].onclick = function(items, i)
      {
        return function()
        {
          changeTab(items[i].id);
          return false;
        };
      }(items, i);
    }
  }

}

Лицензировано под: CC-BY-SA с атрибуция
Не связан с StackOverflow
scroll top