class RuboCop::Cop::Lint::RequireParentheses
Checks for expressions where there is a call to a predicate method with at least one argument, where no parentheses are used around the parameter list, and a boolean operator, && or ||, is used in the last argument.
The idea behind warning for these constructs is that the user might be under the impression that the return value from the method call is an operand of &&/||.
@example
# bad if day.is? :tuesday && month == :jan # ... end
@example
# good if day.is?(:tuesday) && month == :jan # ... end
Constants
- MSG
Public Instance Methods
on_send(node)
click to toggle source
# File lib/rubocop/cop/lint/require_parentheses.rb, line 35 def on_send(node) return if !node.arguments? || node.parenthesized? if node.first_argument.if_type? && node.first_argument.ternary? check_ternary(node.first_argument, node) elsif node.predicate_method? check_predicate(node.last_argument, node) end end
Also aliased as: on_csend
Private Instance Methods
check_predicate(predicate, node)
click to toggle source
# File lib/rubocop/cop/lint/require_parentheses.rb, line 56 def check_predicate(predicate, node) return unless predicate.operator_keyword? add_offense(node) end
check_ternary(ternary, node)
click to toggle source
# File lib/rubocop/cop/lint/require_parentheses.rb, line 48 def check_ternary(ternary, node) return if node.method?(:[]) || !ternary.condition.operator_keyword? range = range_between(node.source_range.begin_pos, ternary.condition.source_range.end_pos) add_offense(range) end