Class: RuboCop::Cop::Lint::DuplicateMethods

Inherits:
Base
  • Object
show all
Defined in:
lib/rubocop/cop/lint/duplicate_methods.rb

Overview

Checks for duplicated instance (or singleton) method definitions.

NOTE: Aliasing a method to itself is allowed, as it indicates that the developer intends to suppress Ruby’s method redefinition warnings. See bugs.ruby-lang.org/issues/13574.

Examples:


# bad
def foo
  1
end

def foo
  2
end

# bad
def foo
  1
end

alias foo bar

# good
def foo
  1
end

def bar
  2
end

# good
def foo
  1
end

alias bar foo

# good
alias foo foo
def foo
  1
end

# good
alias_method :foo, :foo
def foo
  1
end

# bad
class MyClass
  extend Forwardable

  # or with: `def_instance_delegator`, `def_delegators`, `def_instance_delegators`
  def_delegator :delegation_target, :delegated_method_name

  def delegated_method_name
  end
end

# good
class MyClass
  extend Forwardable

  def_delegator :delegation_target, :delegated_method_name

  def non_duplicated_delegated_method_name
  end
end

AllCops:ActiveSupportExtensionsEnabled: false (default)


# good
def foo
  1
end

delegate :foo, to: :bar

AllCops:ActiveSupportExtensionsEnabled: true


# bad
def foo
  1
end

delegate :foo, to: :bar

# good
def foo
  1
end

delegate :baz, to: :bar

# good - delegate with splat arguments is ignored
def foo
  1
end

delegate :foo, **options

# good - delegate inside a condition is ignored
def foo
  1
end

if cond
  delegate :foo, to: :bar
end

Constant Summary collapse

MSG =

rubocop:disable Metrics/ClassLength

'Method `%<method>s` is defined at both %<defined>s and %<current>s.'
RESTRICT_ON_SEND = 
i[alias_method attr_reader attr_writer attr_accessor attr
delegate def_delegator def_instance_delegator def_delegators
def_instance_delegators].freeze

Instance Attribute Summary

Attributes inherited from Base

#config, #processed_source

Instance Method Summary collapse

Methods inherited from Base

#active_support_extensions_enabled?, #add_global_offense, #add_offense, #always_autocorrect?, autocorrect_incompatible_with, badge, #begin_investigation, #callbacks_needed, callbacks_needed, #config_to_allow_offenses, #config_to_allow_offenses=, #contextual_autocorrect?, #cop_config, cop_name, #cop_name, department, documentation_url, exclude_from_registry, #excluded_file?, #external_dependency_checksum, inherited, #inspect, joining_forces, lint?, match?, #message, #offenses, #on_investigation_end, #on_new_investigation, #on_other_file, #parse, #parser_engine, #ready, #relevant_file?, requires_gem, #string_literals_frozen_by_default?, support_autocorrect?, support_multiple_source?, #target_gem_version, #target_rails_version, #target_ruby_version

Methods included from ExcludeLimit

#exclude_limit

Methods included from AutocorrectLogic

#autocorrect?, #autocorrect_enabled?, #autocorrect_requested?, #autocorrect_with_disable_uncorrectable?, #correctable?, #disable_uncorrectable?, #safe_autocorrect?

Methods included from IgnoredNode

#ignore_node, #ignored_node?, #part_of_ignored_node?

Methods included from Util

silence_warnings

Constructor Details

#initialize(config = nil, options = nil) ⇒ DuplicateMethods

Returns a new instance of DuplicateMethods.



127
128
129
130
131
 
# File 'lib/rubocop/cop/lint/duplicate_methods.rb', line 127

def initialize(config = nil, options = nil)
  super
  @definitions = {}
  @scopes = Hash.new { |hash, key| hash[key] = [] }
end

Instance Method Details

#alias_method?(node) ⇒ Object 



167
168
169
 
# File 'lib/rubocop/cop/lint/duplicate_methods.rb', line 167

def_node_matcher :alias_method?, "(send nil? :alias_method (sym $_name) (sym $_original_name))\n"

#delegate_method?(node) ⇒ Object 



172
173
174
175
176
177
 
# File 'lib/rubocop/cop/lint/duplicate_methods.rb', line 172

def_node_matcher :delegate_method?, "(send nil? :delegate\n  ({sym str} $_)+\n  (hash <(pair (sym :to) {sym str}) ...>)\n)\n"

#delegator?(node) ⇒ Object 



180
181
182
183
184
185
186
187
 
# File 'lib/rubocop/cop/lint/duplicate_methods.rb', line 180

def_node_matcher :delegator?, "(send nil? {:def_delegator :def_instance_delegator}\n  {\n    {sym str} ({sym str} $_) |\n    {sym str} {sym str} ({sym str} $_)\n  }\n)\n"

#delegators?(node) ⇒ Object 



190
191
192
193
194
195
 
# File 'lib/rubocop/cop/lint/duplicate_methods.rb', line 190

def_node_matcher :delegators?, "(send nil? {:def_delegators :def_instance_delegators}\n  {sym str}\n  ({sym str} $_)+\n)\n"

#method_alias?(node) ⇒ Object 



153
154
155
 
# File 'lib/rubocop/cop/lint/duplicate_methods.rb', line 153

def_node_matcher :method_alias?, "(alias (sym $_name) (sym $_original_name))\n"

#on_alias(node) ⇒ Object 



157
158
159
160
161
162
163
164
 
# File 'lib/rubocop/cop/lint/duplicate_methods.rb', line 157

def on_alias(node)
  name, original_name = method_alias?(node)
  return unless name && original_name
  return if name == original_name
  return if node.ancestors.any?(&:if_type?)

  found_instance_method(node, name)
end

#on_def(node) ⇒ Object 



133
134
135
136
137
138
139
 
# File 'lib/rubocop/cop/lint/duplicate_methods.rb', line 133

def on_def(node)
  # if a method definition is inside an if, it is very likely
  # that a different definition is used depending on platform, etc.
  return if node.each_ancestor.any?(&:if_type?)

  found_instance_method(node, node.method_name)
end

#on_defs(node) ⇒ Object 



141
142
143
144
145
146
147
148
149
150
 
# File 'lib/rubocop/cop/lint/duplicate_methods.rb', line 141

def on_defs(node)
  return if node.each_ancestor.any?(&:if_type?)

  if node.receiver.const_type?
    _, const_name = *node.receiver
    check_const_receiver(node, node.method_name, const_name)
  elsif node.receiver.self_type?
    check_self_receiver(node, node.method_name)
  end
end

#on_send(node) ⇒ Object

rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/MethodLength, Metrics/PerceivedComplexity



200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
 
# File 'lib/rubocop/cop/lint/duplicate_methods.rb', line 200

def on_send(node) # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/MethodLength, Metrics/PerceivedComplexity
  name, original_name = alias_method?(node)

  if name && original_name
    return if name == original_name
    return if inside_condition?(node)

    found_instance_method(node, name)
  elsif (attr = node.attribute_accessor?)
    on_attr(node, *attr)
  elsif active_support_extensions_enabled? && (names = delegate_method?(node))
    return if inside_condition?(node)

    on_delegate(node, names)
  elsif (name = delegator?(node))
    return if inside_condition?(node)

    found_instance_method(node, name)
  elsif (names = delegators?(node))
    return if inside_condition?(node)

    names.each { |name| found_instance_method(node, name) }
  end
end

#sym_name(node) ⇒ Object 



198
 
# File 'lib/rubocop/cop/lint/duplicate_methods.rb', line 198

def_node_matcher :sym_name, '(sym $_name)'