Как сделать проверку?

Question

[tosster22]

Есть небольшой js код. Все работает на странице на которой код должен срабатывать, но на других страницах выдается ошибка. Подскажите, как правильно поставить проверку, чтобы ошибка не появлялась?

window.onload = function() {
	document.querySelector('.block').addEventListener('click', fTabs);

	function fTabs(event) {
		console.log(event);
		if (event.target.className == 'item'){
			var dataTab = event.target.getAttribute('data-tab');
			var tabH = document.getElementsByClassName('items');
			for (var i=0; i < tabH.length; i++){
				tabH[i].classList.remove('active');
			}
			event.target.classList.add('active');
			var tabBody = document.getElementsByClassName('content');
			for (var i=0;  i < tabBody.length; i++) {
				if (dataTab == i) {
					tabBody[i].style.display = 'block';
				}
				}
			}
		}
	}
}

Comments

[Николай Шабалин]

А какую ошибку-то выдает? Нам сложно судить, так как это может быть просто, что он не смог найти элемент .block

[Stalker_RED]

ошибку нужно угадать?

[tosster22]

Stalker_RED, TypeError: document.querySelector(...) is null

Answer 1

[Ankhena]

var sss= document.querySelector(".block");
if (sss!== null) {
ваш код
}

Comments

[Ankhena]

tosster22 но вообще классный способ задавать вопрос. Выложить код, который не имеет отношения к ошибке, но зато не сказать какая ошибка возникает.

[tosster22]

Ankhena, TypeError: document.querySelector(...) is null

[tosster22]

Ankhena, я просто думал, что тут и так понятно что за ошибка тк я же написал, что она на странице где скрипт работать не должен))) я читал что ошибка частая, но как решить так и не дошло))))

[Ankhena]

tosster22, ну теперь точно рассказывайте куда и как вы написали проверку из моего ответа.

[tosster22]

Ankhena, в смысле куда?) я делал так же примерно, как вы показывали, но до меня только не доходило, что после того как я заменил document.querySelector('.block') нужно в функцию addEventListener('click', fTabs);
добавлять и перед ним указывать называние переменной))) все заработало, вам спасибо за ответ)))

Answer 2

[Шохрух Шаймардонов]

Попробуйте так

window.onload = function() {
  document.querySelector('.block').addEventListener('click', event => {
    console.log(event);
    if (event.target.className == 'item'){
      var dataTab = event.target.getAttribute('data-tab');
      var tabH = document.getElementsByClassName('items');
      for (var i=0; i < tabH.length; i++){
        tabH[i].classList.remove('active');
      }
      event.target.classList.add('active');
      var tabBody = document.getElementsByClassName('content');
      for (var i=0;  i < tabBody.length; i++) {
        if (dataTab == i) {
          tabBody[i].style.display = 'block';
        }
        }
      }
    }
  }
})

Comments

[Ankhena]

Ему не поможет onload, на странице вообще нет такого селектора.

[Шохрух Шаймардонов]

Ankhena, понял. Но можно то что я исправил скрестить с вашим условием if. При онлоде сначала по вашему иф пройтись и дальше все то же самое делать