cblavier
Enum.map_join with a comprehension
Do you think a join option would be a good idea for comprehensions?
Something like Enum.map_join/2 that could map and join lists in a single pass.
book_ids = for book <- books, join: ",", do: book.id
If it looks like a good idea, I will open a ticket / PR.
Most Liked
Eiji
Well … decision on adding features and enhancements is up to @Elixir-Core-Team. What I can say is that you can already do that in a single pass using reduce option:
# Here is a simplified list
books = [%{id: 1}, %{id: 2}, %{id: 3}, %{id: 4}, %{id: 5}]
# Here is an example comprehension:
book_ids = for book <- books, reduce: "" do
"" -> book.id
acc -> "#{acc}.#{book.id}"
end
Helpful resource:
:reduce option in Kernel.SpecialForms.for/1
adamu
I think he means why doesn’t join take a function that’ll also map it for you, like into, or Map.new does?
gregvaughn
You could write your own Collectable module and use :into to achieve this if you expect to use it often.
Last Post!
sabiwara
My guess would be because there already is a joiner optional argument, adding an extra optional mapper argument would have made it awkward to use?
join(enumerable, joiner \\ "", mapper \\ fn x -> x end)
Popular in Questions
Other popular topics
Categories:
Sub Categories:
Forums
Popular Tags
- #ecto
- #liveview
- #troubleshooting
- #learning-elixir
- #deployment
- #library
- #erlang
- #testing
- #genserver
- #mix
- #absinthe
- #remote-other
- #otp
- #plug
- #how-to-question
- #macros
- #postgres
- #channels
- #elixirconf
- #exunit
- #discussion
- #code-sync
- #javascript
- #podcasts
- #onsite
- #dialyzer
- #docker
- #authentication
- #umbrella
- #full-time-contract
- #podcasts-by-brainlid
- #ecto-query
- #elixir-ls
- #phoenix_html
- #iex
- #blog-post
- #graphql
- #genstage
- #ai
- #websockets
- #supervisor
- #elixirconf-us
- #advent-of-code
- #distillery
- #processes
- #forms
- #api
- #metaprogramming
- #security
- #hex









