lista de películas

index.js

const pelis = require("./pelis");

function parsearARGV(argv) {
    const respuesta = {}
    argv.forEach(function (item, indice) {
        if (item.startsWith("--")) {
            const nombreSinGuiones = item.slice(2);
            respuesta[nombreSinGuiones] = argv[indice + 1];
            if (nombreSinGuiones === 'no-format') {
                respuesta['format'] = 'no';
            }
        }
    })
    return respuesta;
}

function main() {
    const comandosAEjecutar = parsearARGV(process.argv.slice(2));

    console.log(pelis.searchByCriteria(comandosAEjecutar));

}

main();

pelis.js

const fs = require("fs");

const getAll = function () {
  return (arrayDePelis = JSON.parse(
    fs.readFileSync(__dirname + "/pelis.json")
  ));
};

const searchBy = function (texto, array) {
  return array.filter(function (element) {
    return element["title"].toLowerCase().indexOf(texto.toLowerCase()) > -1;
  });
};

const searchByTag = function (texto, array) {
  return array.filter(function (element) {
    return element["tags"].indexOf(texto) > -1;
  });
};

const sortBy = function (propiedad, array) {
  return array.sort(function (primero, segundo) {
    if (primero[propiedad] > segundo[propiedad]) {
      return 1;
    }
    if (primero[propiedad] < segundo[propiedad]) {
      return -1;
    }
    return 0;
  });
};

exports.searchByCriteria = function (criterios) {
  let tmp = getAll();

  if (criterios.search) {
    tmp = searchBy(criterios.search, tmp);
  } else {
  }

  if (criterios.tag) {
    tmp = searchByTag(criterios.tag, tmp);
  } else {
  }

  if (criterios.sort) {
    tmp = sortBy(criterios.sort, tmp);
  } else {
  }

  if (criterios.format) {
    return JSON.stringify(tmp);
  }

  return tmp;
};

pelis.json

[
  {
    "title": "El señor de los anillos",
    "rating": 9,
    "tags": ["Fantasía, facinante, literatura fantástica, favorita"]
  },

  {
    "title": "Volver al futuro",
    "rating": 8,
    "tags": ["Ciencia ficcion, aventura"]
  },

  {
    "title": "It",
    "rating": 7,
    "tags": ["Terror, misterio, sobre natural"]
  },

  {
    "title": "Terminator",
    "rating": 6,
    "tags": ["Acción, ciencia ficción, thriller"]
  }
]

Hola mati, funciona todo bien excepto la busqueda por tags. Fijate que cuando parseas te faltan las S en tags. Es decir:
if (criterios.tagS) {
tmp = searchByTag(criterios.tagS, tmp);

Aun asi no va a funcionar, porque en la lógica del método searchByTag estas filtrando por array entero, osea debes escribir tal cual el array completo de tags para que filtre. Fijate una manera para que el método entre a esos array de tags y logre devolver las pelis que tienen cierto tag.
Tambien revee como se utiliza el if else. vos podes hacer una cadena de if sin necesidad de colocar else ya que los estas dejando vacios por cada if. el else funciona como una especie de si no se cumple esta funcion, hace esto (va sin instruccion de los ())

Hola Giuliana, gracias por la devolución. En cuanto pueda lo reviso y lo mando otra vez, verdad? Por más que lo otro este bien, si no funciona todo no paso al otro módulo?. Gracias y buen finde El vie., 1 oct. 2021 14:12, Giuliana Erro ***@***.***> escribió:

…

***@***.**** commented on this gist. ------------------------------ Hola mati, funciona todo bien excepto la busqueda por tags. Fijate que cuando parseas te faltan las S en tags. Es decir: if (criterios.tagS) { tmp = searchByTag(criterios.tagS, tmp); Aun asi no va a funcionar, porque en la lógica del método searchByTag estas filtrando por array entero, osea debes escribir tal cual el array completo de tags para que filtre. Fijate una manera para que el método entre a esos array de tags y logre devolver las pelis que tienen cierto tag. Tambien revee como se utiliza el if else. vos podes hacer una cadena de if sin necesidad de colocar else ya que los estas dejando vacios por cada if. el else funciona como una especie de si no se cumple esta funcion, hace esto (va sin instruccion de los ()) — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <https://gist.github.com/d40da03cb0e1e2f04076eed3809913e4#gistcomment-3913881>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AV3VWPFCIUY6B3OVL4UREX3UEXTX5ANCNFSM5FFEC45Q> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

sisi exactamente, cuando te ande lo de tags volves a mandarlo y ya una vez que funcione todo podes pasar al siguiente modulo. Cualquier duda podes escribir en discord. Buen finde!

Hola Mati! probé el código y note lo mismo que Giuliana, te recomendaría que le pegues una ojeada a los diferentes métodos que hay para ir revisando las posiciones de un array, además de eso, fíjate que los tags los tenes todos adentro de un solo string divididos con comillas, lo que lo hace muy complicado de buscar ya que tenes que si o si que separar cada palabra y después revisarla.

Osea, la lista de tags es un string entero con comas, similar a una frase:

Te aconsejaría, que aparte de lo que te comentan arriba, de ultima guardes los tags en diferentes posiciones de un array y ahí te fijes como buscarlos, ya que te va a ser muchísimo mas sencillo y te va a ahorrar que los separes y sumes mas trabajo al código.

(algo así):

Esperamos esos cambios!

Saludos!