Не работает событие onclick

@Юрий 1989 · Регистрация: 17.07.2018

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

Javascript

var p = document.getElementsByTagName('p');
      console.log(p);
 
      p[0].onclick = f1; 
 
 
function f1(){
    alert('Hello');
}

Вот простенький код. Пытаюсь повесить событие на элемент массива (при нажатии выводит Hello). Выдаёт ошибку. Как правильно вешать события на определённый элемент массива ? Данный пример взял с видео уроков по которым учусь.

@j2FunOnly · 26.07.2018, 12:24

Сообщение от Юрий 1989

Выдаёт ошибку

Какую, если не секрет?

@Юрий 1989 · 26.07.2018, 12:28 **[ТС]**

Cannot set property 'onclick' of undefined

@j2FunOnly · 26.07.2018, 12:34

А полностью весь пример с разметкой можете привести?

@Юрий 1989 · 26.07.2018, 12:37 **[ТС]**

HTML5

<!DOCTYPE html>
<html>
<head>
    <title>Работаем с переменными</title>
    <script src="1.js"></script>
</head>
<body>
    <p>Item 1</p>
    <p>Item 2</p>
    <p>Item 3</p>
    <p>Item 4</p>
    <p>Item 5</p>
</body>
</html>

тут нечего особенного нету. просто хочу к параграфам обращаться как к массиву.

@AlexZaw · 26.07.2018, 12:55

Подключите скрипт перед закрывающим тегом body, либо сделайте так,что-бы код запускался после построения DOM

@Юрий 1989 · 26.07.2018, 12:59 **[ТС]**

Спасибо помогло. Подключил скрипт перед закрывающим тегом body.

Добавлено через 2 минуты
Или можно было указать в подключении defer. Совсем из головы вылетело.

@j2FunOnly · 26.07.2018, 13:00

Смотрите какое дело: грубо говоря, браузер "читает" документ сверху вниз и исполняет javascript по мере прочтения, то есть в вашем примере

PHP/HTML

<!DOCTYPE html>
<html>
<head>
  <title>Работаем с переменными</title>
  <script type="text/javascript">
    var p = document.getElementsByTagName('p');
    console.log(p); 
    p[0].onclick = f1; 
    
    function f1 () {
      alert('Hello');
    }
  </script>
</head>
<body>
<p>Item 1</p>
<p>Item 2</p>
<p>Item 3</p>
<p>Item 4</p>
<p>Item 5</p>
</body>
</html>

на момент исполнения скрипта p - будет пустая коллекция.

Возникает вопрос: почему в консоли вы видите массив из пяти элементов? Потому что метод getElementsByTagName() возвращает HTMLCollection, а у этой коллекции есть один нюанс - она т. н. "live" (живая), то есть изменяется по мере изменения документа. Такое поведение может привести к неожиданным результатам:

PHP/HTML

<!DOCTYPE html>
<html>
<head>
  <title>Работаем с переменными</title>
</head>
<body>
<p>Item 1</p>
<p>Item 2</p>
<p>Item 3</p>
<p>Item 4</p>
<p>Item 5</p>
 
<script type="text/javascript">
  var p = document.getElementsByTagName('p');
  
  for (var i = 0; i < p.length; i++) {
    var newP = document.createElement('p');
    newP.textContent = 'New Item ' + i;
    
    document.body.appendChild(newP);
    
    if (i > 50) {
      alert('Горшочек не вари!');
      break;
    }
  }
</script>
 
</body>
</html>

Но что же делать вам?
Поместите скрипт в конец документа, перед закрывающим тегом </body>
Или запустите скрипт по событию браузера load

Javascript

window.onload = function () {
  var p = document.getElementsByTagName('p');
  console.log(p); 
  p[0].onclick = f1; 
  
  function f1 () {
    alert('Hello');
  }
}

@j2FunOnly the hardway first 2434 / 1815 / 898 Регистрация: 05.06.2015 Сообщений: 3,584
	26.07.2018, 12:24	2
	Сообщение от Юрий 1989 Выдаёт ошибку Какую, если не секрет? 0

@Юрий 1989 0 / 0 / 0 Регистрация: 17.07.2018 Сообщений: 31
	26.07.2018, 12:28 [ТС]	3
	Cannot set property 'onclick' of undefined 0

@j2FunOnly the hardway first 2434 / 1815 / 898 Регистрация: 05.06.2015 Сообщений: 3,584
	26.07.2018, 12:34	4
	А полностью весь пример с разметкой можете привести? 0

@AlexZaw Модератор 2172 / 1571 / 626 Регистрация: 07.08.2016 Сообщений: 3,839
	26.07.2018, 12:55	6
	Сообщение было отмечено Юрий 1989 как решение Решение Подключите скрипт перед закрывающим тегом body, либо сделайте так,что-бы код запускался после построения DOM 0

@Юрий 1989 0 / 0 / 0 Регистрация: 17.07.2018 Сообщений: 31
	26.07.2018, 12:59 [ТС]	7
	Спасибо помогло. Подключил скрипт перед закрывающим тегом body. Добавлено через 2 минуты Или можно было указать в подключении defer. Совсем из головы вылетело. 0

Опции темы

Не работает событие onclick

Решение