Issue #17165 has been updated by Eregon (Benoit Daloze).


And I'd argue if one wants to do everything in one block, just enjoy the freedom of imperative programming:
```ruby
result = []
array.each do |foo|
  if bar = baz(foo)
    result.concat bar.map{...}
  end
end
```

----------------------------------------
Feature #17165: Add `filter` and `flatten` keywords to `Enumerable#map`
https://bugs.ruby-lang.org/issues/17165#change-87548

* Author: sawa (Tsuyoshi Sawada)
* Status: Open
* Priority: Normal
----------------------------------------
I had a use case to do `map` on an enumerable, with 1-level flattening, while skipping `nil` values.

There are convenient `Enumerable#flat_map` and `Enumerable#filter_map` methods, but the problem is that they cannot be used at the same time. I had to chose to do either of the following:

```ruby
array
.filter_map do |foo|
  bar = baz(foo)
  next unless bar
  bar.map{...}
end
.flatten(1)
```

```ruby
array
.flat_map do |foo|
  bar = baz(foo)
  next unless bar
  bar.map{...}
end
.compact
```

```ruby
array
.flat_map do |foo|
  bar = baz(foo)
  next [] unless bar
  bar.map{...}
end
```

The last one of the above may not look so bad, but it requires an extra consideration, and is a bit hacky. When you are in a hurry, it just might not come to your mind.

This led me to realize that `flat_map` and `filter_map` should not be independent operations, but are rather some different modes of the operation `map`. There is no reason for the modes to be mutually exclusive of one another, and a use case that I mentioned above may arise.

I propose to add `filter` and `flatten` as optional keyword arguments to `Enumerable#map`.


```ruby
array
.map(filter: true, flatten: 1) do |foo|
  bar = baz(foo)
  next unless bar
  bar.map{...}
end
```

In fact, even when the two parameters are not used together, I believe it would be easier to the brain and I would feel much more comfortable to pass `filter: true` or `flatten: 1` to `map` when necessary rather than having to deicide whether to use `map` or `flat_map` or use `map` or `filter_map`.

Furthermore, this would make it possible to do flattening of an arbitrary depth (as specified by the parameter) during map.



-- 
https://bugs.ruby-lang.org/

Unsubscribe: <mailto:ruby-core-request / ruby-lang.org?subject=unsubscribe>
<http://lists.ruby-lang.org/cgi-bin/mailman/options/ruby-core>