Sha256: f6f8b38198953a4ad7d2c83c9647d0ada69c865d94072ad95ff53d54a4f107d1
Contents?: true
Size: 1.45 KB
Versions: 14
Compression:
Stored size: 1.45 KB
Contents
# frozen_string_literal: true
module RuboCop
module Cop
module Style
# This cop checks for loops which iterate a constant number of times,
# using a Range literal and `#each`. This can be done more readably using
# `Integer#times`.
#
# This check only applies if the block takes no parameters.
#
# @example
# # bad
# (1..5).each { }
#
# # good
# 5.times { }
#
# @example
# # bad
# (0...10).each {}
#
# # good
# 10.times {}
class EachForSimpleLoop < Cop
MSG = 'Use `Integer#times` for a simple loop which iterates a fixed ' \
'number of times.'
def on_block(node)
return unless offending_each_range(node)
send_node = node.send_node
range = send_node.receiver.source_range.join(send_node.loc.selector)
add_offense(node, location: range)
end
def autocorrect(node)
lambda do |corrector|
range_type, min, max = offending_each_range(node)
max += 1 if range_type == :irange
corrector.replace(node.send_node.source_range,
"#{max - min}.times")
end
end
private
def_node_matcher :offending_each_range, <<-PATTERN
(block (send (begin (${irange erange} (int $_) (int $_))) :each) (args) ...)
PATTERN
end
end
end
end
Version data entries
14 entries across 14 versions & 2 rubygems