se ordenan funciones

Author: IsaMonsalves

src/data.js

@@ -1,10 +1,12 @@
 import data from "./data/ghibli/ghibli.js";
+
 const allMovies = document.getElementById("allMovies");
 const gallery = document.getElementById("gallery");
 const text = document.getElementById("search-input");
 const idDirector = document.getElementById("directores");
 const select = document.getElementById("filters");
 
+//Se llaman películas más vistas al inicio
 export const mostViewed = () => {
   const allPosters = data.films.filter((movies) => movies.most_viewed);
 
@@ -33,13 +35,11 @@ export const mostViewed = () => {
     gallery.appendChild(images[i]);
   }
 };
-
+//Se llaman todas las películas en menú películas
 export const movies = () => {
   //esto deja un array de string por lo que se crea una etiqueta imagen para poder insertar el poster
   const imagesMovies = data.films.map((item) => {
     const img = document.createElement("img");
-    // const div = document.createElement("div");
-    // div.classList.add("posters-container");
     img.classList.add("posters");
     img.src = item.poster;
     img.alt = item.title;
@@ -50,17 +50,14 @@ export const movies = () => {
   });
   imagesMovies.map((image) => allMovies.appendChild(image));
 
-
   allMovies.querySelectorAll(".posters").forEach((pelicula) => {
-    pelicula.addEventListener('click', (e) => {
-      const movieObj = data.films.find(movie => movie.title === e.target.alt)
+    pelicula.addEventListener("click", (e) => {
+      const movieObj = data.films.find((movie) => movie.title === e.target.alt);
       console.log(movieObj);
-    })
+    });
   });
-
-
 };
-
+//Optimización del despliegue de películas cada vez que se ejecute una de las funciones
 const _forceRender = (valueToRender) => {
   if (!valueToRender && valueToRender.length) return;
 
@@ -77,40 +74,26 @@ const _forceRender = (valueToRender) => {
   allMovies.innerHTML = "";
   imagesMovies.map((image) => allMovies.appendChild(image));
 };
-
+//Buscador de películas por componente del título
 export const searchMovies = () => {
   if (!text && !text.value) return;
   const searchedMovies = data.films.filter((item) =>
-    //se cambia el star with por includes(busca si está en el array y da un boolean)
     item.title.toLowerCase().includes(text.value.toLowerCase())
   );
 
   _forceRender(searchedMovies);
 };
-
+//Filtro de películas según el director
 export const searchMoviesByDirector = () => {
   const selectedDirector = idDirector.options[idDirector.selectedIndex].value;
 
   const searchMoviesByDirector = data.films.filter(
     (item) => item.director === selectedDirector
   );
 
-  // console.log(searchMoviesByDirector);
-  // const imagesMovies = searchMoviesByDirector.map((item) => {
-  //   const img = document.createElement("img");
-  //   img.classList.add("posters");
-  //   img.src = item.poster;
-  //   img.width = 100;
-  //   img.heigth = 100;
-  //   //return finaliza el callback
-  //   return img;
-  // });
-  // allMovies.innerHTML = "";
-  // imagesMovies.map((image) => allMovies.appendChild(image));
-
   _forceRender(searchMoviesByDirector);
 };
-
+//Orden de películas
 export const sortByYear = () => {
   const value = select.options[select.selectedIndex].value;
 
@@ -130,7 +113,7 @@ export const sortByYear = () => {
   // }
 
   if (value === "year") {
-    films.sort((a, b) => (a.release_date).localeCompare(b.release_date));
+    films.sort((a, b) => a.release_date.localeCompare(b.release_date));
   }
 
   if (value === "rt-score") {

src/main.js

@@ -21,5 +21,3 @@ if (selectDirector) {
 }
 
 filters && filters.addEventListener("change", sortByYear);
-
-// showMovies();