Sha256: 02340c67142245517b72c61853a4119b7543d070f5a80cf01691551df8162234
Contents?: true
Size: 1.87 KB
Versions: 4
Compression:
Stored size: 1.87 KB
Contents
# frozen_string_literal: true module RuboCop module Cop module Performance # 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 class TimesMap < Cop MESSAGE = 'Use `Array.new(%<count>s)` with a block ' \ 'instead of `.times.%<map_or_collect>s`' MESSAGE_ONLY_IF = 'only if `%<count>s` is always 0 or more' def on_send(node) check(node) end def on_block(node) check(node) end 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 private def check(node) times_map_call(node) do |map_or_collect, count| add_offense(node, message: message(map_or_collect, count)) end end 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 def_node_matcher :times_map_call, <<~PATTERN {(block $(send (send $!nil? :times) {:map :collect}) ...) $(send (send $!nil? :times) {:map :collect} (block_pass ...))} PATTERN end end end end
Version data entries
4 entries across 4 versions & 1 rubygems