exercism / exercism/elixir-analyzer

Is Enum.reduce the "best fitting Enum function" for total_quantity in `boutique-inventory`?

Ouverte
#433 2 commentaires 4 réactions 0 personnes assignées Voir sur GitHub
help wanted 🤝
Langage dominant
Elixir
Étoiles
33
Forks
33
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Solution Implementation:

```elixir
def total_quantity(item) do
Map.values(item.quantity_by_size)
|> Enum.sum()
end

```

The solution says:
image

I agree that `reduce` is a good function/concept to practice with for this solution but "best fitting" seems like a stretch. It took me a long time to understand the concept of `reduce` when learning functional programming and this may be a head scratcher for some folks new to Elixir and functional programming togther.

Would it be possible to change that message to say "Use the Enum.reduce function in the total_quantity function to practice."?

This doesn't give folks who can't figure out `reduce` right away the feeling like they are doing it "wrong".

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.