class RuboCop::Cop::Performance::TimesMap

This cop checks for .times.map calls. In most cases such calls can be replaced with an explicit array creation.

@example

@bad
9.times.map do |i|
  i.to_s
end

@good
Array.new(9) do |i|
  i.to_s
end

Constants

MESSAGE
MESSAGE_ONLY_IF

Public Instance Methods

on_block(node) click to toggle source
# File lib/rubocop/cop/performance/times_map.rb, line 30
def on_block(node)
  check(node)
end
on_send(node) click to toggle source
# File lib/rubocop/cop/performance/times_map.rb, line 26
def on_send(node)
  check(node)
end

Private Instance Methods

autocorrect(node) click to toggle source
# File lib/rubocop/cop/performance/times_map.rb, line 58
def autocorrect(node)
  map_or_collect, count = times_map_call(node)

  replacement =
    "Array.new(#{count.source}" \
    "#{map_or_collect.arguments.map { |arg| ", #{arg.source}" }.join})"

  lambda do |corrector|
    corrector.replace(map_or_collect.loc.expression, replacement)
  end
end
check(node) click to toggle source
# File lib/rubocop/cop/performance/times_map.rb, line 36
def check(node)
  times_map_call(node) do |map_or_collect, count|
    add_offense(node, message: message(map_or_collect, count))
  end
end
message(map_or_collect, count) click to toggle source
# File lib/rubocop/cop/performance/times_map.rb, line 42
def message(map_or_collect, count)
  template = if count.literal?
               MESSAGE + '.'
             else
               "#{MESSAGE} #{MESSAGE_ONLY_IF}."
             end
  format(template,
         count: count.source,
         map_or_collect: map_or_collect.method_name)
end