Contents

require File.join( File.dirname( File.expand_path(__FILE__)), 'smell_detector')
require File.join(File.dirname(File.dirname(File.expand_path(__FILE__))), 'smell_warning')
require File.join(File.dirname(File.dirname(File.expand_path(__FILE__))), 'core', 'smell_configuration')

module Reek
  module Smells

    #
    # A Long Parameter List occurs when a method has more than one
    # or two parameters, or when a method yields more than one or
    # two objects to an associated block.
    #
    # Currently +LongParameterList+ reports any method or block with too
    # many parameters.
    #
    class LongParameterList < SmellDetector

      SMELL_CLASS = self.name.split(/::/)[-1]
      SMELL_SUBCLASS = 'LongParameterList'

      PARAMETER_COUNT_KEY = 'parameter_count'

      # The name of the config field that sets the maximum number of
      # parameters permitted in any method or block.
      MAX_ALLOWED_PARAMS_KEY = 'max_params'

      # The default value of the +MAX_ALLOWED_PARAMS_KEY+ configuration
      # value.
      DEFAULT_MAX_ALLOWED_PARAMS = 3

      def self.default_config
        super.adopt(
          MAX_ALLOWED_PARAMS_KEY => DEFAULT_MAX_ALLOWED_PARAMS,
          Core::SmellConfiguration::OVERRIDES_KEY => {
            "initialize" => {MAX_ALLOWED_PARAMS_KEY => 5}
            }
        )
      end

      def initialize(source, config = LongParameterList.default_config)
        super(source, config)
      end

      #
      # Checks the number of parameters in the given method.
      #
      # @return [Array<SmellWarning>]
      #
      def examine_context(ctx)
        @max_allowed_params = value(MAX_ALLOWED_PARAMS_KEY, ctx, DEFAULT_MAX_ALLOWED_PARAMS)
        num_params = ctx.exp.arg_names.length
        return [] if num_params <= @max_allowed_params
        smell = SmellWarning.new(SMELL_CLASS, ctx.full_name, [ctx.exp.line],
          "has #{num_params} parameters",
          @source, SMELL_SUBCLASS,
          {PARAMETER_COUNT_KEY => num_params})
        [smell]
      end
    end
  end
end

Version data entries

8 entries across 8 versions & 1 rubygems

Version	Path
reek-1.3.1	lib/reek/smells/long_parameter_list.rb
reek-1.3	lib/reek/smells/long_parameter_list.rb
reek-1.2.13	lib/reek/smells/long_parameter_list.rb
reek-1.2.12	lib/reek/smells/long_parameter_list.rb
reek-1.2.11	lib/reek/smells/long_parameter_list.rb
reek-1.2.10	lib/reek/smells/long_parameter_list.rb
reek-1.2.9	lib/reek/smells/long_parameter_list.rb
reek-1.2.8	lib/reek/smells/long_parameter_list.rb