Estoy tratando de hacer que las películas sean agradables una vez almacenándolas en una matriz como esta:
let likedMovies = []; const movieLike = document.getElementById("likeButton"); likedMovies = JSON.parse(localStorage.getItem("likedMovies")); movieLike.addEventListener("click", () => { if (likedMovies.indexOf(allMovies[i].name) === -1) { confirm("Movie liked"); axios.patch(`url`); likedMovies.push(allMovies[i].name); localStorage.setItem("likedMovies", JSON.stringify(likedMovies)); return true; } confirm("Movie already liked"); return false; });estaba funcionando esta tarde, pero ahora la consola muestra un registro de error que dice esto: "script.js: 75 TypeError no detectado: no se pueden leer las propiedades de nulo (leyendo 'indexOf') en HTMLDivElement". y cuando registro mi matriz likeMovies, dice que las matrices son "nulas", ¿podría ayudarme a entender por qué no funciona?
El problema es que está reasignando likedMovies con el resultado de JSON.parse que es nulo si no puede analizarlo.
let likedMovies = JSON.parse(localStorage.getItem("likedMovies")) || [];Esto debería resolver el error, pero también puede optimizarlo almacenándolo como un objeto.
let likedMovies = JSON.parse(localStorage.getItem("likedMovies")) || {}; const movieLike = document.getElementById("likeButton"); movieLike.addEventListener("click", () => { if (likedMovies[allMovies[i].name]) { confirm("Movie liked"); axios.patch(`url`); likedMovies[allMovies[i].name] = true; localStorage.setItem("likedMovies", JSON.stringify(likedMovies)); return true; } confirm("Movie already liked"); return false; });Ahora su búsqueda ocurrirá no en tiempo lineal que es O(n) sino en constante O(1). Pequeña mejora en el rendimiento.