chore(02): correction step 2 - algorithmics

[fabernovel-fnguyen]

No description provided.

[michaelantounfabernovel]

Pour éviter le as unknown as GetPokemonTypeResponse ici, on peut mettre le type dans le .get 🙂

    const pokemonWithCommonTypes = await Promise.all(
      types.map((pokemonType) =>
        this.httpClient
          .get<GetPokemonTypeResponse>(`type/${pokemonType}`)
          .then((response) => response.body),
      ),
    );

et on peut extraire le body directement dans le then 🙂

          .then(({ body }) => body),

[michaelantounfabernovel]

Question: Pourquoi pokemon dans pokemon ? 🤔 Est-ce qu'on ne peut pas faire autrement ? 🤔

Ce serait bien si on pouvait avoir:

    type GetPokemonTypeResponse = {
      pokemon: {
        name: string;
        url: string;
      }[];
    };

Si on change dans cette PR, il faudra modifier aussi dans l'énoncé de la 1ère PR d'algo 🙂

[michaelantounfabernovel]

Might have: On pourrait remplacer le map par un flatMap ici, pour éviter de faire le flat plus bas 🙂

[michaelantounfabernovel]

là aussi, on peut extraire stats de pokemon pour réduire le nombre de pokemon.stats. plus bas 🙂

(acc: PokemonStats, { stats }) => ({