Не получается скрыть блок в зависимости от месяца

@Kazimirskiy · Регистрация: 26.01.2016

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

Добрый день!
Подскажите где ошибка.
Нужно скрыть блок в зависимости от месяца, но не полуается

Javascript

var date=new Date();
var time=date.getMonth();
if(time=4)
{
document.getElementById("m1").style.display="none";
}

Заранее спасибо за помощь

@msheal · 03.05.2018, 15:46

time = 4 -- присваивание
time == 4 -- не строгое сравнение

Вам нужно второе. В остальном код рабочий при условии что элемент с таким айди существует и скрипт подключен после создания этого html элемента

@Kazimirskiy · 03.05.2018, 15:53 **[ТС]**

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

time = 4 -- присваивание
time == 4 -- не строгое сравнение
Вам нужно второе. В остальном код рабочий при условии что элемент с таким айди существует и скрипт подключен после создания этого html элемента

Угу. Не поверите вот только что додумался до этого. Но спасибо за помощь.

Но у меня есть еще вопрос.

У меня 12 блоков по месяцам. Попытался в одном скрипте разместить несколько условий... не вышло. Если на каждый блок в отдельности скрипт писать, то все работает.

Javascript

]var date=new Date();
var time=date.getMonth();
if(time==4)
{
document.getElementById("m1").style.display="none";
}
if(time==5)
{
document.getElementById("m2").style.display="none";
}
else
{
document.getElementById("m2").style.display="block";
document.getElementById("m1").style.display="block";
}

@msheal · 03.05.2018, 16:06

Kazimirskiy, блок прячется, но тут же появляется, потому-что вы малость неправильно записали условие. Нужно так:

Javascript

var date=new Date();
var time=date.getMonth();
 
if(time==4){
  document.getElementById("m1").style.display="none";
} else if(time==5){
  document.getElementById("m2").style.display="none";
} else {
document.getElementById("m2").style.display="block";
document.getElementById("m1").style.display="block";
}

Однако я предлагаю сделать код чище, избавившись от лишних переменных и тонны if/else:

Javascript

switch ( new Date().getMonth() ){
  case 4:
    document.getElementById("m1").style.display="none";
    break;
  case 5:
    document.getElementById("m2").style.display="none";
    break;
  default:
    document.getElementById("m1").style.display="block";
    document.getElementById("m2").style.display="block";
}

@Kazimirskiy · 03.05.2018, 16:10 **[ТС]**

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

Однако я предлагаю сделать код чище, избавившись от лишних переменных и тонны if/else

Спасибо

@msheal 707 / 309 / 191 Регистрация: 05.03.2015 Сообщений: 822
	03.05.2018, 15:46	2
	Сообщение было отмечено Kazimirskiy как решение Решение time = 4 -- присваивание time == 4 -- не строгое сравнение Вам нужно второе. В остальном код рабочий при условии что элемент с таким айди существует и скрипт подключен после создания этого html элемента 2

@Kazimirskiy 3 / 3 / 0 Регистрация: 26.01.2016 Сообщений: 62
	03.05.2018, 16:10 [ТС]	5
	Сообщение от msheal Однако я предлагаю сделать код чище, избавившись от лишних переменных и тонны if/else Спасибо 0

Опции темы

Не получается скрыть блок в зависимости от месяца

Решение