class RuboCop::Cop::Lint::NextWithoutAccumulator

Don’t omit the accumulator when calling ‘next` in a `reduce` block.

@example

# bad
result = (1..4).reduce(0) do |acc, i|
  next if i.odd?
  acc + i
end

# good
result = (1..4).reduce(0) do |acc, i|
  next acc if i.odd?
  acc + i
end

Constants

MSG

Public Instance Methods

on_block(node) click to toggle source
# File lib/rubocop/cop/lint/next_without_accumulator.rb, line 24
def on_block(node)
  on_block_body_of_reduce(node) do |body|
    void_next = body.each_node(:next).find do |n|
      n.children.empty? && parent_block_node(n) == node
    end

    add_offense(void_next) if void_next
  end
end
Also aliased as: on_numblock
on_numblock(node)
Alias for: on_block

Private Instance Methods

parent_block_node(node) click to toggle source
# File lib/rubocop/cop/lint/next_without_accumulator.rb, line 45
def parent_block_node(node)
  node.each_ancestor(:block, :numblock).first
end