Sha256: 21427af9c87b4f31fd9baa5e50f8715eaa693d079df067e23a4172dd684b6a83
Contents?: true
Size: 1.85 KB
Versions: 17
Compression:
Stored size: 1.85 KB
Contents
# frozen_string_literal: true
module RuboCop
module Cop
module Style
# This cop (by default) checks for uses of methods Hash#has_key? and
# Hash#has_value? where it enforces Hash#key? and Hash#value?
# It is configurable to enforce the inverse, using `verbose` method
# names also.
#
# @example EnforcedStyle: short (default)
# # bad
# Hash#has_key?
# Hash#has_value?
#
# # good
# Hash#key?
# Hash#value?
#
# @example EnforcedStyle: verbose
# # bad
# Hash#key?
# Hash#value?
#
# # good
# Hash#has_key?
# Hash#has_value?
class PreferredHashMethods < Base
include ConfigurableEnforcedStyle
extend AutoCorrector
MSG = 'Use `Hash#%<prefer>s` instead of `Hash#%<current>s`.'
OFFENDING_SELECTORS = { short: %i[has_key? has_value?], verbose: %i[key? value?] }.freeze
RESTRICT_ON_SEND = OFFENDING_SELECTORS.values.flatten.freeze
def on_send(node)
return unless node.arguments.one? && offending_selector?(node.method_name)
message = message(node.method_name)
add_offense(node.loc.selector, message: message) do |corrector|
corrector.replace(node.loc.selector, proper_method_name(node.loc.selector.source))
end
end
alias on_csend on_send
private
def message(method_name)
format(MSG, prefer: proper_method_name(method_name), current: method_name)
end
def proper_method_name(method_name)
if style == :verbose
"has_#{method_name}"
else
method_name.to_s.sub(/has_/, '')
end
end
def offending_selector?(method_name)
OFFENDING_SELECTORS[style].include?(method_name)
end
end
end
end
end
Version data entries
17 entries across 17 versions & 3 rubygems