Add Array#compact_map

[bdunne]

Too many times I find myself enumerating over an array wanting some transformed data and no nils. So, I wind up with something like this:

[1,2,3,4,5].collect { |i| i * 2 if i.odd? }.compact
=> [2,6,10]

Instead, we can have:

[1,2,3,4,5].compact_map { |i| i * 2 if i.odd? }
=> [2,6,10]

This avoids allocating an extra array before the compact

[Fryguy]

Should this be on Enumerable instead?

[Fryguy]

👍 for the idea, but I don't like the name... What are the other methods that do this "dual" thing. Only one I can think of is .flat_map. Perhaps select_and_collect or select_and_map? Otherwise there's .each_with_object and .each_with_index, which can be .each.with_object and .each.with_index. Perhaps this might be more flexible....select.collect?

Additionally, if you have a select, you probably want a reject.

[Fryguy]

Crystal lang uses compact_map, so I suggest going with that. This also then feels analogous to flat_map.

[Fryguy]

Also a few of these rubocops are legit.

[miq-bot]

Checked commit bdunne@ec36c0b with ruby 2.5.5, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
3 files checked, 0 offenses detected
Everything looks fine. ⭐