Почему не срабатывает getDay?

Question

[mrzgt]

Не могу понять почему не срабатывает условие ((currentDate.getDay==1) || (currentDate.getDay==2)) в скрипте

$(function() {
  // получить значение даты из LocalStorage
  var dateModalShow = localStorage.getItem("dateModalShow");
  // получить значение текущей даты
  var currentDate = new Date().toString();
  var period = (((new Date(currentDate)).getTime())- ((new Date(dateModalShow)).getTime()))/60000;
  // если окно не показывалось или с момента прошлого показа прошли сутки и сегодня пон (или вт), то
  if (((dateModalShow==null) || (period>1))  && ((currentDate.getDay==1) || (currentDate.getDay==2)) ) {
    // запустить таймер, который запустит содержимое функции через 5 минут
    setTimeout(function(){
      $("#modal").modal('show');
      localStorage.setItem("dateModalShow", currentDate);
    },60000);
  }
});

Answer 1

[display: block]

У строки (!) currentDate пытаешься использовать свойство (!) getDay.

Comments

[mrzgt]

можете подробнее как исправить

[display: block]

currentDate не преобразуй в строку.
В period не создавай из строки currentDate ещё один объект.
getDay это метод, а не свойство.

[mrzgt]

побывал, даже так не работает

$(function() {
// получить значение даты из LocalStorage
var dateModalShow = localStorage.getItem("dateModalShow");
// получить значение текущей даты
var currentDate = new Date();
if ((currentDate.getDay==0) || (currentDate.getDay==1) || (currentDate.getDay==2) || (currentDate.getDay==3) || (currentDate.getDay==4) || (currentDate.getDay==5) || (currentDate.getDay==6) || (currentDate.getDay==7) ) {
// запустить таймер, который запустит содержимое функции через 5 минут
setTimeout(function(){
$("#mailModal").modal('show');
localStorage.setItem("dateModalShow", currentDate);
},1);
}
});

[display: block]

mrzgt: А если после getDay добавить скобки (как у функции), то получится.

[mrzgt]

вышло, спасибо

[tema_sun]

mrzgt: и кешируйте полученный номер дня, чтобы он не считался каждый раз.

Answer 2

[tema_sun]

После этого
var currentDate = new Date().toString();
currentDate содержит строку с датой. Разумеется у строки нет никакого свойства getDay.
Обратите внимание, что при этом далее вы из строки с этой датой снова делаете объект Date.
И getDay это не свойство, а метод, т.е. вызывать его нужно так currentDate.getDay()

Comments

[mrzgt]

"и кешируйте полученный номер дня, чтобы он не считался каждый раз. "

можете подсказать как, в js не силен

[tema_sun]

mrzgt: добавьте var currentDay = currentDate.getDay(); И потом в if'е делайте if ((currentDay===0) || (currentDay===1)......)