lib/rubocop/cop/style/unless_else.rb
# frozen_string_literal: true
module RuboCop
module Cop
module Style
# Looks for `unless` expressions with `else` clauses.
#
# @example
# # bad
# unless foo_bar.nil?
# # do something...
# else
# # do a different thing...
# end
#
# # good
# if foo_bar.present?
# # do something...
# else
# # do a different thing...
# end
class UnlessElse < Base
include RangeHelp
extend AutoCorrector
MSG = 'Do not use `unless` with `else`. Rewrite these with the positive case first.'
def on_if(node)
return unless node.unless? && node.else?
add_offense(node) do |corrector|
body_range = range_between_condition_and_else(node, node.condition)
else_range = range_between_else_and_end(node)
next if part_of_ignored_node?(node)
corrector.replace(node.loc.keyword, 'if')
corrector.replace(body_range, else_range.source)
corrector.replace(else_range, body_range.source)
end
ignore_node(node)
end
def range_between_condition_and_else(node, condition)
range_between(condition.source_range.end_pos, node.loc.else.begin_pos)
end
def range_between_else_and_end(node)
range_between(node.loc.else.end_pos, node.loc.end.begin_pos)
end
end
end
end
end