Open Hugo-Hache opened 2 years ago
I don't get this part at all:
# bad - non explicit array name
results = electric_appliance.process!
results.each { _1.taste }
Seems like an OK usage to me, if this has some side effects.
I think the code bellow is OK, even if it'd stick to the older shorthands.
# bad - block can be created from method
waffles.each { _1.taste } # => waffles.each(&:taste)
waffles.each { taste(_1) } # => waffles.each(&method(:taste))
# bad - non explicit array name
results = electric_appliance.process!
results.each { _1.taste }
I view the usage of numbered parameter bad in this case because by removing the variable name, you loose information on what you manipulate (as the array name is not specific enough). Here for instance the code below would have been easier to understand:
results = electric_appliance.process!
results.each { |waffle| waffle.taste }
I understand this part is quite subjective so it would not surprise me it would not fit in the style guide.
Following my PR introducing a cop for numbered parameters, @koic suggested that choosing the default style required a discussion in the style guide.
As a basis, here is what we agreed upon in our team. "Explicit" array name is not per se enforcable with a cop, so I'm not sure how it would fit in the guide.