Как добавить элемент в существующий массив?

Question

[Darkness]

Знаю, знаю, вопрос из базы, но не могу понять, как мне тут добавить новый элемент. Т.е я уже добавляю элемент методом push.

.then((data) => {
    let selectionItem = []
      selectionItem.push(data.url)
      localStorage.setItem('selectionItem', JSON.stringify(selectionItem))
 })

Но как мне сделать так, что когда снова придет data, к этому массиву добавить её ? Т.е. будет не
0['n']
а уже

0['n']
1['n']

Как я понимаю , нужно пройтись циклом по массиву и сделать push. Но как бы не пытался я сравнить их и добавить новый, он просто перезаписывает [0] итем , прошу помощи!

Answer 1

[Andrey Suha]

selectionItem объявить выше промиса. Вы когда получаете data, то selectionItem перезаписываете как пустой массив

Comments

[Darkness]

Если делать так:

fetch(`https://source.unsplash.com/collection/${collectionID}/${imageWidth}x${imageHeight}/?sig=${randomNumber}`)
     let selectionItem = []
      .then((data) => {
        localStorage.setItem('unsplashWall', data.url)
          selectionItem.push(data.url)
          localStorage.setItem('selectionItem', JSON.stringify(selectionItem))
        })

Консоль говорит [].then не является функцией
Если делать так:

let selectionItem = []
    fetch(`https://source.unsplash.com/collection/${collectionID}/${imageWidth}x${imageHeight}/?sig=${randomNumber}`)
      .then((data) => {
        localStorage.setItem('unsplashWall', data.url)
          selectionItem.push(data.url)
          localStorage.setItem('selectionItem', JSON.stringify(selectionItem))
        })

Он также записывает в [0], т.е. не добавляется в массив, а перезаписывает 0

[Stalker_RED]

Darkness, добавляет. У вас изначально пассив совсем пустой.

[Darkness]

Stalker_RED, окей, но как мне добавить уже к существуещему массиву, мне нужно сделать push в массив, чтобы получить не только 0, но дальше

[Andrey Suha]

Darkness,

Консоль говорит [].then не является

Конечно, then ведь не метод массива.

Он также записывает в [0], т.е. не добавляется в массив, а перезаписывает 0

А приведите весь код, пожалуйста

[RAX7]

Darkness, и все это завернуто в функцию? типа так:

function loadImage(collectionID, imageWidth, imageHeight, randomNumber) {
  let selectionItem = []
  fetch(`https://source.unsplash.com/collection/${collectionID}/${imageWidth}x${imageHeight}/?sig=${randomNumber}`)
    .then((data) => {
      localStorage.setItem('unsplashWall', data.url)
      selectionItem.push(data.url)
      localStorage.setItem('selectionItem', JSON.stringify(selectionItem))
    })
}
loadImage(42, 420, 420, 777)

[Darkness]

RAX7,

Эта вся функция, но нужно просто сохранить новую картинки из fetch еще в массив, т.е. сейчас у меня каждый день просто обновляет массив этот первый итем, а не добавляет к концу его

function runOncePerDayUnsplash() {
  if (!hasOneDayPassedUnsplash()) return false;

  function renderGalleryItem(randomNumber) {
    let selectionItem = []
    fetch(`https://source.unsplash.com/collection/${collectionID}/${imageWidth}x${imageHeight}/?sig=${randomNumber}`)
      .then((data) => {
        localStorage.setItem('unsplashWall', data.url)
          selectionItem.push(data.url)
        })
          // selectionItem.push(data.url)
          // localStorage.setItem('selectionItem', JSON.stringify(selectionItem))
        if ('FirstOpen' in localStorage) {
          document.querySelector('#main').style.backgroundImage = 'url(' + './welcomeInstant.jpg' + ')';
        } else {

          if ('NopeRandomBg' in localStorage) {
            document.querySelector('#main').style.backgroundImage = 'url(' + localStorage.getItem('bgImg') + ')';
          } else {
            localStorage.bgImg = localStorage.unsplashWall;
            document.querySelector('#main').style.backgroundImage = 'url(' + localStorage.getItem('unsplashWall') + ')';
          }
    }
  }

  for (let i = 0; i < numItemsToGenerate; i++) {
    let randomImageIndex = Math.floor(Math.random() * numImagesAvailable);
    renderGalleryItem(randomImageIndex);
  }
}

[RAX7]

function runOncePerDayUnsplash() {
  if (!hasOneDayPassedUnsplash()) return false;
  let selectionItem = [] // <- вот здесь объяви selectionItem

  function renderGalleryItem(randomNumber) {
    
    fetch(`https://source.unsplash.com/collection/${collectionID}/${imageWidth}x${imageHeight}/?sig=${randomNumber}`)
      .then((data) => {
        localStorage.setItem('unsplashWall', data.url)
          selectionItem.push(data.url)
        })
          // selectionItem.push(data.url)
          // localStorage.setItem('selectionItem', JSON.stringify(selectionItem))
        if ('FirstOpen' in localStorage) {
          document.querySelector('#main').style.backgroundImage = 'url(' + './welcomeInstant.jpg' + ')';
        } else {

          if ('NopeRandomBg' in localStorage) {
            document.querySelector('#main').style.backgroundImage = 'url(' + localStorage.getItem('bgImg') + ')';
          } else {
            localStorage.bgImg = localStorage.unsplashWall;
            document.querySelector('#main').style.backgroundImage = 'url(' + localStorage.getItem('unsplashWall') + ')';
          }
    }
  }

  for (let i = 0; i < numItemsToGenerate; i++) {
    let randomImageIndex = Math.floor(Math.random() * numImagesAvailable);
    renderGalleryItem(randomImageIndex);
  }
}

[Darkness]

RAX7, тоже так делал, все также 0 итем переписывает :с

[RAX7]

Darkness, ну так найди место где он у тебя не будет переписываться. Или храни selectionItem в замыкании.

const renderGalleryItem = (function() {
   const selectionItem = []
   
   return function(randomNumber) {
     fetch(`https://source.unsplash.com/collection/${collectionID}/${imageWidth}x${imageHeight}/?sig=${randomNumber}`)
       .then((data) => {
         localStorage.setItem('unsplashWall', data.url)
         selectionItem.push(data.url)
       })
     // selectionItem.push(data.url)
     // localStorage.setItem('selectionItem', JSON.stringify(selectionItem))
     if ('FirstOpen' in localStorage) {
       document.querySelector('#main').style.backgroundImage = 'url(' + './welcomeInstant.jpg' + ')';
     } else {

       if ('NopeRandomBg' in localStorage) {
         document.querySelector('#main').style.backgroundImage = 'url(' + localStorage.getItem('bgImg') + ')';
       } else {
         localStorage.bgImg = localStorage.unsplashWall;
         document.querySelector('#main').style.backgroundImage = 'url(' + localStorage.getItem('unsplashWall') + ')';
       }
     }
   }
 })()

 function runOncePerDayUnsplash() {
   if (!hasOneDayPassedUnsplash()) return false;
   for (let i = 0; i < numItemsToGenerate; i++) {
     let randomImageIndex = Math.floor(Math.random() * numImagesAvailable);
     renderGalleryItem(randomImageIndex);
   }
 }

[Роман]

Stalker_RED, на 146% поддерживаю!!!!

... пассив совсем пустой.

)))

PS: в спб, водку, коньяк и даже пиво сегодня не продают (для меня это печалька, сарказм, и троллинг)))