Class: WhyClasses::Rules::SingleAttributeReader

Inherits:
WhyClasses::Rule show all
Defined in:
lib/why_classes/rules/single_attribute_reader.rb

Overview

Decoupled polymorphism: an instance method that only reads a single field could be a freestanding function that works on anything responding to that field. Off by default -- it is a stylistic nudge and can be noisy.

Constant Summary

Constants inherited from WhyClasses::Rule

WhyClasses::Rule::H

Class Method Summary collapse

Instance Method Summary collapse

Methods inherited from WhyClasses::Rule

autocorrectable?, #call, inherited, #initialize, rule_name

Constructor Details

This class inherits a constructor from WhyClasses::Rule

Class Method Details

.description ⇒ Object 



12
13
14
 
# File 'lib/why_classes/rules/single_attribute_reader.rb', line 12

def self.description
  "Instance method that only reads one field -> consider a freestanding polymorphic function."
end

.tier ⇒ Object 



16
17
18
 
# File 'lib/why_classes/rules/single_attribute_reader.rb', line 16

def self.tier
  :unsafe
end

Instance Method Details

#on_class(node) ⇒ Object 



20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
 
# File 'lib/why_classes/rules/single_attribute_reader.rb', line 20

def on_class(node)
  s = shape(node)
  return if s.name.nil?
  return if s.uses_metaprogramming?

  s.behavior_methods.each do |method|
    read = H.ivars_read(method)
    written = H.ivars_assigned(method)
    next unless written.empty?
    next unless read.size == 1

    field = read.first.to_s.delete_prefix("@")
    add_offense(
      method,
      message: "#{s.name}##{method.children[0]} only reads @#{field}; it isn't tied to #{s.name}.",
      suggestion: suggestion(method, field),
      corrector: build_corrector(node, s, method)
    )
  end
end