Sha256: 7b5a1fdda3477442d128c79f8276f9387e97b9d38eca5d43726fb84b74db6455
Contents?: true
Size: 1.05 KB
Versions: 6763
Compression:
Stored size: 1.05 KB
Contents
# frozen_string_literal: true
module RuboCop
module Cop
module Lint
# This cop checks to make sure safe navigation isn't used with `empty?` in
# a conditional.
#
# While the safe navigation operator is generally a good idea, when
# checking `foo&.empty?` in a conditional, `foo` being `nil` will actually
# do the opposite of what the author intends.
#
# @example
# # bad
# return if foo&.empty?
# return unless foo&.empty?
#
# # good
# return if foo && foo.empty?
# return unless foo && foo.empty?
#
class SafeNavigationWithEmpty < Cop
MSG = 'Avoid calling `empty?` with the safe navigation operator ' \
'in conditionals.'.freeze
def_node_matcher :safe_navigation_empty_in_conditional?, <<-PATTERN
(if (csend (send ...) :empty?) ...)
PATTERN
def on_if(node)
return unless safe_navigation_empty_in_conditional?(node)
add_offense(node)
end
end
end
end
end
Version data entries
6,763 entries across 6,759 versions & 23 rubygems