Sha256: bd25cdff79bf7ca99bd0197108465a6de2f376e469be323de5be94d203d74137
Contents?: true
Size: 1.48 KB
Versions: 11
Compression:
Stored size: 1.48 KB
Contents
require_relative 'smell_detector'
require_relative 'smell_warning'
module Reek
module Smells
#
# A Large Class is a class or module that has a large number of
# instance variables, methods or lines of code.
#
# +TooManyInstanceVariables' reports classes having more than a
# configurable number of instance variables.
#
# See {file:docs/Too-Many-Instance-Variables.md} for details.
class TooManyInstanceVariables < SmellDetector
# The name of the config field that sets the maximum number of instance
# variables permitted in a class.
MAX_ALLOWED_IVARS_KEY = 'max_instance_variables'
DEFAULT_MAX_IVARS = 4
def self.smell_category
'LargeClass'
end
def self.contexts
[:class]
end
def self.default_config
super.merge(
MAX_ALLOWED_IVARS_KEY => DEFAULT_MAX_IVARS,
EXCLUDE_KEY => []
)
end
#
# Checks +klass+ for too many instance variables.
#
# @return [Array<SmellWarning>]
#
def inspect(ctx)
max_allowed_ivars = value(MAX_ALLOWED_IVARS_KEY, ctx, DEFAULT_MAX_IVARS)
count = ctx.local_nodes(:ivasgn).map { |ivasgn| ivasgn.children.first }.uniq.length
return [] if count <= max_allowed_ivars
[smell_warning(
context: ctx,
lines: [ctx.exp.line],
message: "has at least #{count} instance variables",
parameters: { count: count })]
end
end
end
end
Version data entries
11 entries across 11 versions & 1 rubygems