@KhusanJuraev

ForEach возвращает null как исправить?

const engWord = document.getElementById("eng"),
    rusWord = document.getElementById("rus"),
    inputs =  document.getElementsByClassName("input"),
    addButton = document.getElementById("add-word-btn"),
    table = document.getElementById('table');

let words,
    btnsdelete;

localStorage.length < 1 ? words = [] : words = JSON.parse(localStorage.getItem('words'));

const addWordToTable = index => {
    table.innerHTML +=
        `<tr class="tr">
        <td class="eng-word">${words[index].english}</td>
        <td class="rus-word">${words[index].russian}</td>
        <td>
           <button class="btn-delete">X</button>
        </td>
        <tr> `
};

words.forEach((element, i) => {
    addWordToTable(i);
});

function CreateWord(english, russian) {
    this.english = english;
    this.russian = russian;
}

addButton.addEventListener('click', () => {
    if (
        engWord.value.length < 1 ||
        rusWord.value.length < 1 ||
        !isNaN(engWord.value) ||
        !isNaN(rusWord.value)
    ) {
        for (let key of Array.from(inputs)) {
            key.classList.add("error")
        }
    } else {
        for (let key of Array.from(inputs)) {
            key.classList.remove("error")
        }
        words.push(new CreateWord(engWord.value, rusWord.value));
        localStorage.setItem("words", JSON.stringify(words));
        addWordToTable(words.length - 1);
        engWord.value = null;
        rusWord.value = null;
        addEventDelete();
    }
});

const deleteWord = e => {
    const rowIndex = e.target.parentNode.parentNode.parentNode.rowIndex;
    e.target.parentNode.parentNode.parentNode.remove();
    words.splice(rowIndex, 1);
    localStorage.removeItem('words');
    localStorage.setItem('words', JSON.stringify(words));
};

const addEventDelete = () => {
    if(words.length > 0) {
        btnsdelete = document.querySelectorAll('.btn-delete');
        for(let btn of Array.from(btnsdelete)) {
            btn.addEventListener('click', e => {
                deleteWord(e);
            })
        }
    }
};

addEventDelete();
  • Вопрос задан
  • 124 просмотра
Решения вопроса 1
ertaquo
@ertaquo
Это возможно, если в localStorage есть какие-то элементы, но нет элемента "words".
В любом случае, для несуществующего элемента localStorage.getItem вернет null, а JSON.parse(null) === null. Поэтому лучше написать как-то так:
words = JSON.parse(localStorage.getItem('words')) || [];
Ответ написан
Пригласить эксперта
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы