JustFS / react-basics

35 stars 14 forks source link

on a perdu le range #1

Open lemarcgagnon opened 3 years ago

lemarcgagnon commented 3 years ago

Bonjour Bonjour,

Je reproduisais le code avec la video jusqu'a sort pour voir que cela ne fonctionnait pas pour moi et constater que le code en ligne avait été simplifé avec les .filter ... .sort... .map

mais du coup on a perdu le range, car il ne fonctionne plus avec le nouveau code du github, j'essaie de la figurer mais je dois dire que j'en prends plein la geule. Comment faire au juste svp.

Merci,

Marc Montréal, Canada

J'adore votre méthode pédagogique!!! Bravo!!

lemarcgagnon commented 3 years ago

ha ha je suis fier de moi, il faut juste ajouter le .slice

"

JustFS commented 3 years ago

Oui Slice fonctionne mais moi la version améliorée j'avais fait avec filter : .filter((country, index) => index < rangeValue)

Tous les chemins menent à Rome ;)

On Tue, Jun 29, 2021 at 2:35 PM Marc Gagnon @.***> wrote:

ha ha je suis fier de moi, il faut juste ajouter le .slice

"

{data .filter((country) => country.region.includes(selectedRadio)) .sort((a, b) => b.population - a.population) .slice(0, rangeValue) .map((country) => { return

               })}
       </ul> "

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/JustFS/react-basics/issues/1#issuecomment-870561192, or unsubscribe https://github.com/notifications/unsubscribe-auth/AM6XY7OQF76QVQJJF7JK7R3TVG4XTANCNFSM47P6V7UQ .

ssommelet21 commented 2 years ago

De nos jours (au moment de mon commentaire) :

https://restcountries.eu/rest/v2/all?fields=name;population;region;capital;flag

Ne marche plus, j'ai dû utiliser :

https://restcountries.com/v3.1/all?fields=name,population,region,capital,flags

Effectivement y a d'autres changements à faire ensuite dans le code :

flag devient flags et pour avoir l'image il faut faire : country.flags.svg et country.name ne suffit plus et devient country.name.common et country.capital devient country.capital[0]

Merci JustFS pour ta vidéo super claire ;-)