Challenge 4 - Githubissues

tanrax / soy-un-buen-programador-js

Juego con retos sencillos de JavaScript para probarte a ti mismo que sabes lo que haces

11 stars 2 forks source link

Challenge 4 #16

Open nestor-almarza opened 2 years ago

nestor-almarza commented 2 years ago

const miLista4 = ["Lisp", "Clojure", "Haskell", "Elm", "Racket", "Swift", "Erlang", "Scala"];

const numberOfLangsThatContainS = (list) => list.map((lang) => lang.toLowerCase()) .filter((lowCaseLang) => lowCaseLang.includes("s")) .reduce((prv, curr) => (curr ? ++prv : prv), 0); ; console.log(numberOfLangsThatContainS(miLista4));

tanrax commented 2 years ago

Primero: buen trabajo. Lo has logrado! Segundo:

Tal vez puedas evitar el map realizando el toLowerCase en el filter. Pero esto va a gustos.
¿por qué usar en el reduce una condicional para verificar que curr existe?
¿por qué usar ++prv? y no un simple prv += 1. Es una tontería y no lo es. Te permite cambiar ese 1 por otro numero.

nestor-almarza commented 2 years ago

const numberOfLangsThatContainS2 = (list) => list .filter((lang) => lang.toLowerCase().includes("s")) .length;

Vaya, no habia caido en que me puedo quitar el .map con el filter. mucho mejor asi!
Pues si, estaba tan enfocado en intentar encajar metodos funcionales que se me olvidaba que un simple .length cumple.
Esto ultimo es simplemnete estetico, me aprece que queda mas guay un ++ que un + 1; (A mi teclado no le funcionan los acentos)

Gracias por el code review!!

tanrax commented 2 years ago

En cada mensaje te vuelves mejor dev 😄