class RuboCop::Cop::Style::GuardClause
end
end
work
if bar? # <- reports an offense
do_something
end
work
if foo?
def test
# bad
end
end
work
if bar?
end
work
if foo?
def test
# good
@example AllowConsecutiveConditionals: true
end
end
work
if bar? # <- reports an offense
end
work
if foo?
def test
# bad
@example AllowConsecutiveConditionals: false (default)
end
work if something
define_method(:test) do
# also good
end
work
return unless something
define_method(:test) do
# good
end
end
work
if something
define_method(:test) do
# bad
ok
foo || raise(‘exception’) if something
# good
end
ok
else
foo || raise(‘exception’)
if something
# bad
ok
raise ‘exception’ if something
# good
end
ok
else
raise ‘exception’
if something
# bad
end
work if something
def test
# also good
end
work
return unless something
def test
# good
end
end
work
if something
def test
# bad
@example
that contain logical operators such as ‘foo || raise(’exception’)‘
NOTE: Autocorrect works in most cases except with if-else statements
in the body of the conditional expression.
one of `return`, `break`, `next`, `raise`, or `fail` is used
A condition with an `elsif` or `else` branch is allowed unless
expression
Use a guard clause instead of wrapping the code inside a conditional
def accepted_form?(node, ending: false)
def accepted_form?(node, ending: false) accepted_if?(node, ending) || node.condition.multiline? || node.parent&.assignment? end
def accepted_if?(node, ending)
def accepted_if?(node, ending) return true if node.modifier_form? || node.ternary? || node.elsif_conditional? || assigned_lvar_used_in_if_branch?(node) if ending node.else? else !node.else? || node.elsif? end end
def allowed_consecutive_conditionals?
def allowed_consecutive_conditionals? cop_config.fetch('AllowConsecutiveConditionals', false) end
def and_or_guard_clause?(guard_clause)
def and_or_guard_clause?(guard_clause) parent = guard_clause.parent parent.operator_keyword? end
def assigned_lvar_used_in_if_branch?(node)
def assigned_lvar_used_in_if_branch?(node) return false unless (if_branch = node.if_branch) assigned_lvars_in_condition = node.condition.each_descendant(:lvasgn).map do |lvasgn| lvar_name, = *lvasgn lvar_name.to_s end used_lvars_in_branch = if_branch.each_descendant(:lvar).map(&:source) || [] (assigned_lvars_in_condition & used_lvars_in_branch).any? end
def autocorrect(corrector, node, condition, replacement, guard)
def autocorrect(corrector, node, condition, replacement, guard) corrector.replace(node.loc.keyword.join(condition.source_range), replacement) if_branch = node.if_branch else_branch = node.else_branch corrector.replace(node.loc.begin, "\n") if node.then? if if_branch&.send_type? && heredoc?(if_branch.last_argument) autocorrect_heredoc_argument(corrector, node, if_branch, else_branch, guard) elsif else_branch&.send_type? && heredoc?(else_branch.last_argument) autocorrect_heredoc_argument(corrector, node, else_branch, if_branch, guard) else corrector.remove(node.loc.end) return unless node.else? corrector.remove(node.loc.else) corrector.remove(range_of_branch_to_remove(node, guard)) end end
def autocorrect_heredoc_argument(corrector, node, heredoc_branch, leave_branch, guard)
def autocorrect_heredoc_argument(corrector, node, heredoc_branch, leave_branch, guard) remove_whole_lines(corrector, node.loc.end) return unless node.else? remove_whole_lines(corrector, leave_branch.source_range) remove_whole_lines(corrector, node.loc.else) remove_whole_lines(corrector, range_of_branch_to_remove(node, guard)) corrector.insert_after( heredoc_branch.last_argument.loc.heredoc_end, "\n#{leave_branch.source}" ) end
def check_ending_body(body)
def check_ending_body(body) return if body.nil? if body.if_type? check_ending_if(body) elsif body.begin_type? final_expression = body.children.last check_ending_if(final_expression) if final_expression&.if_type? end end
def check_ending_if(node)
def check_ending_if(node) return if accepted_form?(node, ending: true) || !min_body_length?(node) return if allowed_consecutive_conditionals? && consecutive_conditionals?(node.parent, node) register_offense(node, 'return', node.inverse_keyword) check_ending_body(node.if_branch) end
def consecutive_conditionals?(parent, node)
def consecutive_conditionals?(parent, node) parent.each_child_node.inject(false) do |if_type, child| break if_type if node == child child.if_type? end end
def guard_clause_source(guard_clause)
def guard_clause_source(guard_clause) if and_or_guard_clause?(guard_clause) guard_clause.parent.source else guard_clause.source end end
def heredoc?(argument)
def heredoc?(argument) argument.respond_to?(:heredoc?) && argument.heredoc? end
def on_block(node)
def on_block(node) return unless node.method?(:define_method) || node.method?(:define_singleton_method) on_def(node) end
def on_def(node)
def on_def(node) body = node.body return unless body check_ending_body(body) end
def on_if(node)
def on_if(node) return if accepted_form?(node) if (guard_clause = node.if_branch&.guard_clause?) kw = node.loc.keyword.source guard = :if elsif (guard_clause = node.else_branch&.guard_clause?) kw = node.inverse_keyword guard = :else else return end guard = nil if and_or_guard_clause?(guard_clause) register_offense(node, guard_clause_source(guard_clause), kw, guard) end
def range_of_branch_to_remove(node, guard)
def range_of_branch_to_remove(node, guard) branch = case guard when :if then node.if_branch when :else then node.else_branch end branch.source_range end
def register_offense(node, scope_exiting_keyword, conditional_keyword, guard = nil)
def register_offense(node, scope_exiting_keyword, conditional_keyword, guard = nil) condition, = node.node_parts example = [scope_exiting_keyword, conditional_keyword, condition.source].join(' ') if too_long_for_single_line?(node, example) return if trivial?(node) example = "#{conditional_keyword} #{condition.source}; #{scope_exiting_keyword}; end" replacement = <<~RUBY.chomp #{conditional_keyword} #{condition.source} #{scope_exiting_keyword} end RUBY end add_offense(node.loc.keyword, message: format(MSG, example: example)) do |corrector| next if node.else? && guard.nil? autocorrect(corrector, node, condition, replacement || example, guard) end end
def remove_whole_lines(corrector, range)
def remove_whole_lines(corrector, range) corrector.remove(range_by_whole_lines(range, include_final_newline: true)) end
def too_long_for_single_line?(node, example)
def too_long_for_single_line?(node, example) max = max_line_length max && node.source_range.column + example.length > max end
def trivial?(node)
def trivial?(node) return false unless node.if_branch node.branches.one? && !node.if_branch.if_type? && !node.if_branch.begin_type? end