lib/rubocop/cop/style/return_nil.rb


# frozen_string_literal: true

module RuboCop
  module Cop
    module Style
      # This cop enforces consistency between 'return nil' and 'return'.
      #
      # Supported styles are: return, return_nil.
      #
      # @example EnforcedStyle: return (default)
      #   # bad
      #   def foo(arg)
      #     return nil if arg
      #   end
      #
      #   # good
      #   def foo(arg)
      #     return if arg
      #   end
      #
      # @example EnforcedStyle: return_nil
      #   # bad
      #   def foo(arg)
      #     return if arg
      #   end
      #
      #   # good
      #   def foo(arg)
      #     return nil if arg
      #   end
      class ReturnNil < Cop
        include ConfigurableEnforcedStyle

        RETURN_MSG = 'Use `return` instead of `return nil`.'.freeze
        RETURN_NIL_MSG = 'Use `return nil` instead of `return`.'.freeze

        def_node_matcher :return_node?, '(return)'
        def_node_matcher :return_nil_node?, '(return nil)'

        def on_return(node)
          # Check Lint/NonLocalExitFromIterator first before this cop
          node.each_ancestor(:block, :def, :defs) do |n|
            break if scoped_node?(n)

            send_node, args_node, _body_node = *n

            # if a proc is passed to `Module#define_method` or
            # `Object#define_singleton_method`, `return` will not cause a
            # non-local exit error
            break if define_method?(send_node)

            next if args_node.children.empty?

            return nil if chained_send?(send_node)
          end

          add_offense(node) unless correct_style?(node)
        end

        def autocorrect(node)
          lambda do |corrector|
            corrected = style == :return ? 'return' : 'return nil'
            corrector.replace(node.source_range, corrected)
          end
        end

        private

        def message(_node)
          style == :return ? RETURN_MSG : RETURN_NIL_MSG
        end

        def correct_style?(node)
          style == :return && !return_nil_node?(node) ||
            style == :return_nil && !return_node?(node)
        end

        def scoped_node?(node)
          node.def_type? || node.defs_type? || node.lambda?
        end

        def_node_matcher :chained_send?, '(send !nil? ...)'
        def_node_matcher :define_method?, <<-PATTERN
          (send _ {:define_method :define_singleton_method} _)
        PATTERN
      end
    end
  end
end