Почему данные из localStorage обновляются при перезагрузки страницы?

Question

[Валентин Попов]

Я пытаюсь реализовать так, чтобы при перезагрузки страницы даты оставались изначальными. Но на данный момент они у меня каждый раз обновляются при перезагрузки страницы.
Я генерирую рандомную дату и затем сохраняю ее в localStorage и затем достаю из него, но при обновлении все равно даты в дом дереве обновляются, хотя должны быть из хранилища. Вот что мне удалось сделать. Может кто поможет и увидит ошибку в моем коде

window.addEventListener('load', function() {
  const dates = document.querySelectorAll('.review-list__date');
 
 for(let i = 0; i < dates.length; i++) {
  const randomDate = moment().add(-Math.random() * 10, 'days').add(-Math.random() * 5, 'hours').format('DD.MM.YY | HH:mm');
  localStorage.setItem('date', randomDate);
  dates[i].textContent = localStorage.getItem('date');
 }
});

Comments

[iBird Rose]

тк ты при каждом обновлении страницы сначала записываешь дату, а потом ее пытаешься брать. логично, что при каждом обновлении там будет новая дата.
вставь туда условие на наличие ключа date. т.е. если date есть, то ничего не записываем, а сразу читаем. а записывай только если нет date

[Валентин Попов]

iBird Rose, типо этого

if (localStorage.getItem('date')) {
    dates[i].textContent = localStorage.getItem('date');
  }

[iBird Rose]

Валентин Попов, я бы проверял через
if (localStorage.hasOwnProperty('date')) {

[Валентин Попов]

iBird Rose, сейчас вообще дата не записывается) и в локальном хранилище лежит одна дата, но и она не записывается. Он же должен цикл проходить и записывать 4 даты

[Валентин Попов]

iBird Rose, вот так сделал и все равно данные обновляются при обновлении страницы

window.addEventListener('load', function() {
  const dates = document.querySelectorAll('.review-list__date');
 
 for(let i = 0; i < dates.length; i++) {
  const randomDate = moment().add(-Math.random() * 10, 'days').add(-Math.random() * 5, 'hours').format('DD.MM.YY | HH:mm');

  localStorage.setItem('date', randomDate);
  
  if (localStorage.hasOwnProperty('date')) {
    dates[i].textContent = localStorage.getItem('date');
  }
 }
});

[iBird Rose]

Валентин Попов, зачем ты в if обернул чтение? тебе запись нужно в нее оборачивать. чтобы, он не перезаписывал данные, если они уже есть

[Валентин Попов]

iBird Rose, я не понимаю, можете поправить мой код. Я проверяю если есть что то в хранилище то записывай его.

[iBird Rose]

Валентин Попов, зачем тебе записывать, если там уже записано? именно от этого в вопросе ты и хотел избавиться не?

window.addEventListener('load', function() {
  const dates = document.querySelectorAll('.review-list__date');
 
 for(let i = 0; i < dates.length; i++) {
  const randomDate = moment().add(-Math.random() * 10, 'days').add(-Math.random() * 5, 'hours').format('DD.MM.YY | HH:mm');

if (!localStorage.hasOwnProperty('date')) {
  localStorage.setItem('date', randomDate);
  }
  
    dates[i].textContent = localStorage.getItem('date');
  
 }
});

[Валентин Попов]

iBird Rose, Так у меня сейчас одно и тоже значение стоит в полях, а должны быть разные и при обновлении страницы они не должны обновляться.

[iBird Rose]

Валентин Попов, ответом написал

Answer 1

[iBird Rose]

Comments

[Валентин Попов]

Спасибо!