class RuboCop::Cop::InternalAffairs::SingleLineComparison
node.single_line?
# good
node.first_line == node.last_line
# bad
node.loc.last_line == node.loc.line
# bad
node.loc.line == node.loc.last_line
# bad
node.loc.last_line == node.loc.first_line
# bad
node.loc.first_line == node.loc.last_line
# bad
@example
comparing ‘first_line` and `last_line` for equality.
Enforces the use of `node.single_line?` instead of
def extract_receiver(node)
def extract_receiver(node) node = node.receiver if node.send_type? && %i[loc source_range].include?(node.method_name) node.source end
def on_send(node)
def on_send(node) return unless (receiver = single_line_comparison(node)) bang = node.method?(:!=) ? '!' : '' preferred = "#{bang}#{extract_receiver(receiver)}.single_line?" add_offense(node, message: format(MSG, preferred: preferred)) do |corrector| corrector.replace(node, preferred) end end