Форум

DenisReva · #1 (**permalink**) 11.12.2017, 23:13

День добрый. Подскажите, где ошибка в коде условия?

Цель:
При соответствии url выводится баннер в определенной товарной категории, в сайдбаре. То есть, к примеру если url страницы содержит директорию /zakrepochnye-mashiny, то для div с id="TestTest", менять display: none на display: block.

Код ниже выполняет замену display:none на block, но во всех категориях, то есть баннер сквозной получается

<div id="TestTest" style="display: none; background-color: #333; height: 200px">
</div>
<script>
if(window.location.pathname.indexOf('zakrepochnye-mashiny')) {
var foo = document.getElementById("TestTest");
if(foo) {
foo.style.display = 'block';
}
}
</script>

void() · #2 (**permalink**) 12.12.2017, 07:46

Ошибка в том, что блоки у вас отражаются при условии, что существует блок с ID "TestTest". А так как он, очевидно, всегда существует, то и блок будет отображаться тоже всегда.
Так будет правильнее:

<div id="TestTest" style="display: none; background-color: #333; height: 200px">
</div>
<script>
if(window.location.pathname.indexOf('zakrepochnye-mashiny')) {
var foo = document.getElementById("TestTest");
foo.style.display = 'block';
}
</script>

DenisReva · #3 (**permalink**) 12.12.2017, 08:48

Чет все равно, не хочет работать как надо

void() · #4 (**permalink**) 12.12.2017, 08:54

DenisReva,

<!DOCTYPE html>
<html lang="en">
<head>
	<meta charset="UTF-8">
	<title>Document</title>
</head>
<body>
	<div id="TestTest" style="display: none; background-color: #333; height: 200px">
</div>
<script>
if(window.location.pathname.indexOf('TestTest') > -1) {
var foo = document.getElementById("TestTest");
foo.style.display = 'block';
}
</script>

</body>
</html>

DenisReva · #5 (**permalink**) 12.12.2017, 08:56

Условие проверки наличия директории в URL всегда true, почему то

void() · #6 (**permalink**) 12.12.2017, 08:58

DenisReva,
Хм, у меня работает корректно, не получается получить такую проблему... Протестировал в Хроме и Мозилла.

DenisReva · #7 (**permalink**) 12.12.2017, 11:25

c > -1, работает) правда не понял в чем смысл, длина строки больше -1?

Dilettante_Pro · #8 (**permalink**) 12.12.2017, 12:38

DenisReva,
indexOf возвращает -1, если значение не найдено.
indexOf

DenisReva · #9 (**permalink**) 12.12.2017, 14:33

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

DenisReva,
indexOf возвращает -1, если значение не найдено.
indexOf

Ок

Форум

Справочник

Discord чат

Курсы javascript