Challenge 4 #16

New Issue

2022-03-11T08:04:52+01:00

nestor-almarza commented

2022-03-11 08:04:52 +01:00

(Migrated from github.com)

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));

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

2022-03-11 11:02:23 +01:00

(Migrated from github.com)

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.

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

2022-03-11 11:36:12 +01:00

(Migrated from github.com)

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!!

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!!

🎉 1

tanrax commented

2022-03-11 11:38:19 +01:00

(Migrated from github.com)

En cada mensaje te vuelves mejor dev 😄

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: andros/soy-un-buen-programador-js#16