exercism / exercism/elixir-analyzer

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

オープン
#433 コメント 2 件 リアクション 4 件 担当者 0 名 GitHub で見る
help wanted 🤝
主要言語
Elixir
スター
33
フォーク
33
PR マージ指標
30日以内にマージされた PR はありません

説明

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".

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。