Class: RuboCop::Cop::Lint::LiteralAsCondition

Inherits:
Base
  • Object
show all
Includes:
RangeHelp
Defined in:
lib/rubocop/cop/lint/literal_as_condition.rb

Overview

This cop checks for literals used as the conditions or as operands in and/or expressions serving as the conditions of if/while/until.

Examples:


# bad
if 20
  do_something
end

# bad
if some_var && true
  do_something
end

# good
if some_var && some_condition
  do_something
end

# good
# When using a boolean value for an infinite loop.
while true
  break if condition
end

Constant Summary collapse

MSG =
'Literal `%<literal>s` appeared as a condition.'

Constants inherited from Base

Base::RESTRICT_ON_SEND

Instance Attribute Summary

Attributes inherited from Base

#config, #processed_source

Instance Method Summary collapse

Methods inherited from Base

#add_global_offense, #add_offense, autocorrect_incompatible_with, badge, #callbacks_needed, callbacks_needed, #config_to_allow_offenses, #config_to_allow_offenses=, #cop_config, #cop_name, cop_name, department, documentation_url, exclude_from_registry, #excluded_file?, #external_dependency_checksum, inherited, #initialize, joining_forces, lint?, match?, #offenses, #on_investigation_end, #on_new_investigation, #on_other_file, #ready, #relevant_file?, support_autocorrect?, support_multiple_source?, #target_rails_version, #target_ruby_version

Methods included from AutocorrectLogic

#autocorrect?, #autocorrect_enabled?, #autocorrect_requested?, #correctable?, #disable_uncorrectable?, #safe_autocorrect?

Methods included from IgnoredNode

#ignore_node, #ignored_node?, #part_of_ignored_node?

Methods included from Util

silence_warnings

Constructor Details

This class inherits a constructor from RuboCop::Cop::Base

Instance Method Details

#message(node) ⇒ Object


76
77
78
# File 'lib/rubocop/cop/lint/literal_as_condition.rb', line 76

def message(node)
  format(MSG, literal: node.source)
end

#on_case(case_node) ⇒ Object


55
56
57
58
59
60
61
62
63
64
65
66
67
68
# File 'lib/rubocop/cop/lint/literal_as_condition.rb', line 55

def on_case(case_node)
  if case_node.condition
    check_case(case_node)
  else
    case_node.each_when do |when_node|
      next unless when_node.conditions.all?(&:literal?)

      range = when_conditions_range(when_node)
      message = message(range)

      add_offense(range, message: message)
    end
  end
end

#on_if(node) ⇒ Object


37
38
39
# File 'lib/rubocop/cop/lint/literal_as_condition.rb', line 37

def on_if(node)
  check_for_literal(node)
end

#on_send(node) ⇒ Object


70
71
72
73
74
# File 'lib/rubocop/cop/lint/literal_as_condition.rb', line 70

def on_send(node)
  return unless node.negation_method?

  check_for_literal(node)
end

#on_until(node) ⇒ Object Also known as: on_until_post


48
49
50
51
52
# File 'lib/rubocop/cop/lint/literal_as_condition.rb', line 48

def on_until(node)
  return if condition(node).false_type?

  check_for_literal(node)
end

#on_while(node) ⇒ Object Also known as: on_while_post


41
42
43
44
45
# File 'lib/rubocop/cop/lint/literal_as_condition.rb', line 41

def on_while(node)
  return if condition(node).true_type?

  check_for_literal(node)
end