Избегайте повторных объявлений

Question

Я пытаюсь использовать localStorage для сохранения массива строк. У меня есть функция addToFav(string), которая предназначена для загрузки строкового значения в массив, который содержит имена программ или pNames в моем случае. Я стараюсь следовать этой схеме: How do I store an array in localStorage?

localStorage.arrayNum = 0; 
var pNames=[]; 


function addToFav(string) 
{ 
    pNames[localStorage.arrayNum] = string; 

    if (localStorage.arrayNum) 
    { 
     localStorage.arrayNum=Number(localStorage.arrayNum)+1; 
    } 
    else 
    { 
     localStorage.arrayNum=1; 
    } 

    localStorage['pNames']=JSON.stringify(pNames); 
    var storedNames=JSON.parse(localStorage['pNames']); 

    document.getElementById("testField").innerHTML=storedNames; 

} 

При нажатии на кнопку, вызывающую addToFav(string), строку, которую я переходила в функции отображается в testfield DIV, который OK. Тем не менее, запуск функции снова должен добавить еще один элемент в массив, но это не так, он просто несколько раз добавляет к pNames [0], потому что я думаю, что pNames и arrayNum получают каждый раз. С этим, кто-нибудь знает, как предотвратить эту проблему? Я хотел бы создать массив, который добавляет любимые элементы, когда пользователь вызывает мою функцию.

Спасибо!

Answer 1

Это потому, что вы каждый раз используете pNames как свой ключ.

localStorage['pNames']=JSON.stringify(pNames); 

Вы должны сделать что-то вроде:

localStorage['pNames'+someVal]=JSON.stringify(pNames); 

Answer 2

Никогда доступ LocalStorage как обычный объект! Используйте according functions for that. Это также исправляет вашу ошибку в упомянутом коде ido;)