class RuboCop::Cop::Performance::ReverseEach
This cop is used to identify usages of `reverse.each` and change them to use `reverse_each` instead.
If the return value is used, it will not be detected because the result will be different.
- source,ruby
- 1, 2, 3].reverse.each {} #=> [3, 2, 1
- 1, 2, 3].reverse_each {} #=> [1, 2, 3
@example
# bad items.reverse.each # good items.reverse_each
- 1, 2, 3].reverse.each {} #=> [3, 2, 1
Constants
- MSG
- RESTRICT_ON_SEND
Public Instance Methods
on_send(node)
click to toggle source
# File lib/rubocop/cop/performance/reverse_each.rb, line 34 def on_send(node) return if use_return_value?(node) reverse_each?(node) do range = offense_range(node) add_offense(range) do |corrector| corrector.replace(range, 'reverse_each') end end end
Private Instance Methods
offense_range(node)
click to toggle source
# File lib/rubocop/cop/performance/reverse_each.rb, line 54 def offense_range(node) range_between(node.children.first.loc.selector.begin_pos, node.loc.selector.end_pos) end
use_return_value?(node)
click to toggle source
# File lib/rubocop/cop/performance/reverse_each.rb, line 48 def use_return_value?(node) !!node.ancestors.detect do |ancestor| ancestor.assignment? || ancestor.send_type? || ancestor.return_type? end end