Sha256: f70bbd0879540ff9e0f1c67a41b13949627bd1ca7588368582660c9ae048a5b5
Contents?: true
Size: 1.87 KB
Versions: 2
Compression:
Stored size: 1.87 KB
Contents
# frozen_string_literal: true module RuboCop module Cop module Performance # This cop identifies the use of `Regexp#match` or `String#match`, which # returns `#<MatchData>`/`nil`. The return value of `=~` is an integral # index/`nil` and is more performant. # # @example # # bad # do_something if str.match(/regex/) # while regex.match('str') # do_something # end # # # good # method(str =~ /regex/) # return value unless regex =~ 'str' class RedundantMatch < Base extend AutoCorrector MSG = 'Use `=~` in places where the `MatchData` returned by ' \ '`#match` will not be used.' # 'match' is a fairly generic name, so we don't flag it unless we see # a string or regexp literal on one side or the other def_node_matcher :match_call?, <<~PATTERN {(send {str regexp} :match _) (send !nil? :match {str regexp})} PATTERN def_node_matcher :only_truthiness_matters?, <<~PATTERN ^({if while until case while_post until_post} equal?(%0) ...) PATTERN def on_send(node) return unless match_call?(node) && (!node.value_used? || only_truthiness_matters?(node)) && !(node.parent && node.parent.block_type?) add_offense(node) do |corrector| autocorrect(corrector, node) end end private def autocorrect(corrector, node) # Regexp#match can take a second argument, but this cop doesn't # register an offense in that case return unless node.first_argument.regexp_type? new_source = "#{node.receiver.source} =~ #{node.first_argument.source}" corrector.replace(node.source_range, new_source) end end end end end
Version data entries
2 entries across 2 versions & 1 rubygems
Version | Path |
---|---|
rubocop-performance-1.8.1 | lib/rubocop/cop/performance/redundant_match.rb |
rubocop-performance-1.8.0 | lib/rubocop/cop/performance/redundant_match.rb |