troessner/reek

View on GitHub
lib/reek/smell_detectors/repeated_conditional.rb

Summary

Maintainability
A
35 mins
Test Coverage
# frozen_string_literal: true

require_relative '../ast/node'
require_relative 'base_detector'

module Reek
  module SmellDetectors
    #
    # Simulated Polymorphism occurs when
    # * code uses a case statement (especially on a type field);
    # * or code has several if statements in a row
    #   (especially if they're comparing against the same value);
    # * or code uses instance_of?, kind_of?, is_a?, or ===
    #   to decide what type it's working with;
    # * or multiple conditionals in different places test the same value.
    #
    # Conditional code is hard to read and understand, because the reader must
    # hold more state in his head.
    # When the same value is tested in multiple places throughout an application,
    # any change to the set of possible values will require many methods and
    # classes to change. Tests for the type of an object may indicate that the
    # abstraction represented by that type is not completely defined (or understood).
    #
    # +RepeatedConditional+ checks for multiple conditionals
    # testing the same value throughout a single class.
    #
    # See {file:docs/Repeated-Conditional.md} for details.
    class RepeatedConditional < BaseDetector
      # The name of the config field that sets the maximum number of
      # identical conditionals permitted within any single class.
      MAX_IDENTICAL_IFS_KEY = 'max_ifs'
      DEFAULT_MAX_IFS = 2

      BLOCK_GIVEN_CONDITION = ::Parser::AST::Node.new(:send, [nil, :block_given?])

      def self.contexts # :nodoc:
        [:class]
      end

      def self.default_config
        super.merge(MAX_IDENTICAL_IFS_KEY => DEFAULT_MAX_IFS)
      end

      #
      # Checks the given class for multiple identical conditional tests.
      #
      # @return [Array<SmellWarning>]
      #
      # @quality :reek:DuplicateMethodCall { max_calls: 2 }
      def sniff
        conditional_counts.select do |_key, lines|
          lines.length > max_identical_ifs
        end.map do |key, lines|
          occurs = lines.length
          expression = key.format_to_ruby
          smell_warning(
            lines: lines,
            message: "tests '#{expression}' at least #{occurs} times",
            parameters: { name: expression, count: occurs })
        end
      end

      private

      def max_identical_ifs
        @max_identical_ifs ||= value(MAX_IDENTICAL_IFS_KEY, context)
      end

      #
      # Returns a Hash listing all of the conditional expressions in
      # the given syntax tree together with the number of times each
      # occurs. Ignores nested classes and modules.
      #
      # @quality :reek:TooManyStatements { max_statements: 9 }
      def conditional_counts
        result = Hash.new { |hash, key| hash[key] = [] }
        collector = proc do |node|
          next unless (condition = node.condition)
          next if condition == BLOCK_GIVEN_CONDITION

          result[condition].push(condition.line)
        end
        [:if, :case].each { |stmt| context.local_nodes(stmt, &collector) }
        result
      end
    end
  end
end