[ruby-core:95059] [Ruby master Bug#16178] Numbered parameters: _1 should be the same as |x| and _0 should not exist
From:
eregontp@...
Date:
2019-09-24 13:37:30 UTC
List:
ruby-core #95059
Issue #16178 has been updated by Eregon (Benoit Daloze).
nobu (Nobuyoshi Nakada) wrote:
> When `_1` is same as `|x|`, what does `[[1, 2]].map {_1 + _2}` mean?
It means `[[1, 2]].map { |a,b| a + b } # => [3]` of course.
> The meaning of `_1` changes if `_2` is used or not?
Yes, just like `Proc#arity` changes.
```ruby
-> { _1 }.arity #=> 1
-> { _2; _1 }.arity #=> 2
```
It's just consistent.
We have to accept the drawback that numbered parameters change arity, there is no way around that.
Changing arity with named arguments has the same effect (`[[1, 2]].map { |a| a }` to `[[1, 2]].map { |a,b| a + b }`).
My main point is we want `_1` to be the non-dangerous behavior.
Changing arity can break things, that is not new, but indeed using numbered parameters makes the change less obvious in the source. That is intrinsically a drawback of numbered parameters.
----------------------------------------
Bug #16178: Numbered parameters: _1 should be the same as |x| and _0 should not exist
https://bugs.ruby-lang.org/issues/16178#change-81692
* Author: Eregon (Benoit Daloze)
* Status: Open
* Priority: Normal
* Assignee: matz (Yukihiro Matsumoto)
* Target version:
* ruby -v: ruby 2.7.0dev (2019-09-24T12:57:54Z master 0e84eecc17) [x86_64-linux]
* Backport: 2.5: UNKNOWN, 2.6: UNKNOWN
----------------------------------------
Currently on trunk:
```ruby
array = ["string", 42, [1, 2]]
array.map { |x| x * 2 }
# => ["stringstring", 84, [1, 2, 1, 2]]
array.map { _1 * 2 }
# => ["stringstring", 84, 2]
```
Oops, this trivial code just lost data and completely ignored the element class!
This is clearly contrary to intuition and is very dangerous.
Using `_0` instead has the correct behavior but it's clear we use 1-based indexing for numbered parameters,
and it doesn't solve that `_1` has dangerous behavior.
Basically the current behavior is that `_0` is the same as `|x|` and `_1` is the same as `|x,|`.
`|x,|` is almost never used in Ruby, and for good reasons, it just throws away data/information/the class of the object.
Such a dangerous operation should only be done when it's explicit, and the trailing comma in `|x,|` shows that, but `_1` does not.
So let's make `_1` be `|x|` and remove `_0`.
I am going to be harsh, but this discussion has gone too long without any serious written argument for the current behavior:
I believe it's irrational and irresponsible to have `_1` be `|x,|`, it's just going to lead to nasty bugs.
Try to convince me otherwise.
If not, in one week I want to apply this change.
From the discussion in https://bugs.ruby-lang.org/issues/15723#note-127
and in https://bugs.ruby-lang.org/issues/15708
Some reactions to this behavior in https://twitter.com/eregontp/status/1115318993299083265
--
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>