Почему не работает функция JS?

Question

[pet1987]

На сайте есть блок с текстом - Сегодня на нашем сайте заказов 32.
Нужно что бы число с количеством заказов менялось каждую минуту для каждого пользователя.
Думаю нужно делать так:
Считываем данные из хранилища localStorage... Там сумма и дата, когда мы ее объявили...
Далее делаем проверку равенства текущего времени и времени в хранилище localStorage.
Если совпадает, останавливаем функцию. Если не совпадает то записываем в localStorage новое время и выводи на сайт сгенерированное новое количество заказав.
Вот что получилось, но не работает:

<p>Сегодня на нашем сайте заказов - <span id="zn">32</span></p>
<script>
function zakaz() {
var logElem = document.querySelector("#zn");
var znn = 30;
var date = new Date;
var time, znach, a, b;
a = localStorage.setItem (time , date.getMinutes());
b = localStorage.setItem(znach , znn);
if (a == date.getMinutes())
{
return true; 
} else {
a = localStorage.setItem (time , date.getMinutes());
logElem.innerHTML = (Math.floor(Math.random() * 100) + 1);
}
}
</script>

Comments

[ThunderCat]

но не работает

Вы задолбали со своим "не работает". Пишите ошибку или ожидаемое вс действительное поведение!!

[pet1987]

ThunderCat,
Ошибку консоль не выдает.
Ожидаемое подведение должно быть такое что если в localStorage время не совпадает с текущим, т.е. минуты, то генерируется новое число и выводиться на сайт в блок id="zn"

[ProjectSoft]

pet1987,
https://developer.mozilla.org/ru/docs/Web/API/Stor...
Читайте внимательно. Что возвращает метод setItem?
И дальше по коду куча вопросов... Особенно здесь

a = localStorage.setItem (time , date.getMinutes());
b = localStorage.setItem(znach , znn);
if (a == date.getMinutes())
{
return true; 
} else {

Если бы setItem что-то возвращал, то что вы сравнивает? Одно и тоже значение?

[pet1987]

Получается т.к. setItem ничего не возвращает, то за место него использовать нужно getItem

<p>Сегодня на нашем сайте заказов - <span id="zn">32</span></p>
<script>
function zakaz() {
var logElem = document.querySelector("#zn");
var znn = 30;
var date = new Date;
var time, znach, a, b;
a = localStorage.getItem (time , date.getMinutes());
b = localStorage.getItem (znach , znn);
if (a == date.getMinutes())
{
return true; 
} else {
a = localStorage.getItem (time , date.getMinutes());
logElem.innerHTML = (Math.floor(Math.random() * 100) + 1);
}
}
</script>

[ThunderCat]

pet1987,

localStorage.getItem (time , date.getMinutes());

то есть упорно игнорируем наличие документации, да?

Answer 1

[Elvis]

Количество заказов вообще не должно храниться в localstorage. Это значение должно подтягивать я из вашей базы, с сервера. При обновлении страницы и, если уж так нужно, каждую минуту, делая ajax запрос на сервер.

Comments

[ThunderCat]

скорее всего это фейк заглушка, ну типа "закажите срочно, осталось ндцать товаров и 40 минут"

Answer 2

[ThunderCat]

Во первых - где вызов функции то?
Во вторых:

a = localStorage.setItem (time , date.getMinutes());
// https://developer.mozilla.org/ru/docs/Web/API/Storage/setItem#возвращаемые_значения
b = localStorage.setItem(znach , znn); // то же самое, скорее всего задумывалось getItem()
if (a == date.getMinutes()) // логично что здесь фигня будет