lib/rubocop/cop/style/redundant_conditional.rb
# frozen_string_literal: true
module RuboCop
module Cop
module Style
# Checks for redundant returning of true/false in conditionals.
#
# @example
# # bad
# x == y ? true : false
#
# # bad
# if x == y
# true
# else
# false
# end
#
# # good
# x == y
#
# # bad
# x == y ? false : true
#
# # good
# x != y
class RedundantConditional < Base
include Alignment
extend AutoCorrector
operators = RuboCop::AST::Node::COMPARISON_OPERATORS.to_a
COMPARISON_OPERATOR_MATCHER = "{:#{operators.join(' :')}}"
MSG = 'This conditional expression can just be replaced by `%<msg>s`.'
def on_if(node)
return unless offense?(node)
message = message(node)
add_offense(node, message: message) do |corrector|
corrector.replace(node, replacement_condition(node))
end
end
private
def message(node)
replacement = replacement_condition(node)
msg = node.elsif? ? "\n#{replacement}" : replacement
format(MSG, msg: msg)
end
# @!method redundant_condition?(node)
def_node_matcher :redundant_condition?, <<~RUBY
(if (send _ #{COMPARISON_OPERATOR_MATCHER} _) true false)
RUBY
# @!method redundant_condition_inverted?(node)
def_node_matcher :redundant_condition_inverted?, <<~RUBY
(if (send _ #{COMPARISON_OPERATOR_MATCHER} _) false true)
RUBY
def offense?(node)
return false if node.modifier_form?
redundant_condition?(node) || redundant_condition_inverted?(node)
end
def replacement_condition(node)
condition = node.condition.source
expression = redundant_condition_inverted?(node) ? "!(#{condition})" : condition
node.elsif? ? indented_else_node(expression, node) : expression
end
def indented_else_node(expression, node)
"else\n#{indentation(node)}#{expression}"
end
end
end
end
end