Class: RuboCop::Cop::Style::MultipleComparison
Relationships & Source Files | |
Super Chains via Extension / Inclusion / Inheritance | |
Class Chain:
self,
::RuboCop::Cop::AutoCorrector ,
::RuboCop::Cop::Base ,
::RuboCop::ExcludeLimit ,
NodePattern::Macros,
RuboCop::AST::Sexp
|
|
Instance Chain:
self,
::RuboCop::Cop::Base ,
::RuboCop::Cop::AutocorrectLogic ,
::RuboCop::Cop::IgnoredNode ,
::RuboCop::Util ,
RuboCop::AST::Sexp
|
|
Inherits: |
RuboCop::Cop::Base
|
Defined in: | lib/rubocop/cop/style/multiple_comparison.rb |
Overview
Checks against comparing a variable with multiple items, where
Array#include?
, Set#include?
or a case
could be used instead
to avoid code repetition.
It accepts comparisons of multiple method calls to avoid unnecessary method calls
by default. It can be configured by AllowMethodComparison
option.
Constant Summary
-
MSG =
# File 'lib/rubocop/cop/style/multiple_comparison.rb', line 55'Avoid comparing a variable with multiple items ' \ 'in a conditional, use `Array#include?` instead.'
::RuboCop::Cop::Base
- Inherited
Class Attribute Summary
::RuboCop::Cop::AutoCorrector
- Extended
::RuboCop::Cop::Base
- Inherited
.gem_requirements, .lint?, | |
.support_autocorrect? | Returns if class supports autocorrect. |
.support_multiple_source? | Override if your cop should be called repeatedly for multiple investigations Between calls to |
Class Method Summary
::RuboCop::Cop::Base
- Inherited
.autocorrect_incompatible_with | List of cops that should not try to autocorrect at the same time as this cop. |
.badge | Naming. |
.callbacks_needed, .cop_name, .department, | |
.documentation_url | Returns a url to view this cops documentation online. |
.exclude_from_registry | Call for abstract Cop classes. |
.inherited, | |
.joining_forces | Override and return the Force class(es) you need to join. |
.match? | Returns true if the cop name or the cop namespace matches any of the given names. |
.new, | |
.requires_gem | Register a version requirement for the given gem name. |
.restrict_on_send |
::RuboCop::ExcludeLimit
- Extended
exclude_limit | Sets up a configuration option to have an exclude limit tracked. |
transform |
Instance Attribute Summary
- #allow_method_comparison? ⇒ Boolean readonly private
::RuboCop::Cop::Base
- Inherited
::RuboCop::Cop::AutocorrectLogic
- Included
Instance Method Summary
- #on_or(node)
- #comparison?(node) ⇒ Boolean private
- #comparisons_threshold private
-
#find_offending_var(node, variables = Set.new, values = [])
private
Metrics/PerceivedComplexity.
- #nested_comparison?(node) ⇒ Boolean private
-
#offense_range(values)
private
Metrics/PerceivedComplexity.
- #root_of_or_node(or_node) private
- #simple_comparison?(node) ⇒ Boolean private
- #simple_comparison_lhs?(node) private
- #simple_comparison_rhs?(node) private
- #simple_double_comparison?(node) private
- #variable_name(node) private
::RuboCop::Cop::Base
- Inherited
#add_global_offense | Adds an offense that has no particular location. |
#add_offense | Adds an offense on the specified range (or node with an expression) Unless that offense is disabled for this range, a corrector will be yielded to provide the cop the opportunity to autocorrect the offense. |
#begin_investigation | Called before any investigation. |
#callbacks_needed, | |
#cop_config | Configuration Helpers. |
#cop_name, #excluded_file?, | |
#external_dependency_checksum | This method should be overridden when a cop’s behavior depends on state that lives outside of these locations: |
#inspect, | |
#message | Gets called if no message is specified when calling |
#name | Alias for Base#cop_name. |
#offenses, | |
#on_investigation_end | Called after all on_… |
#on_new_investigation | Called before all on_… |
#on_other_file | Called instead of all on_… |
#parse | There should be very limited reasons for a Cop to do it’s own parsing. |
#parser_engine, | |
#ready | Called between investigations. |
#relevant_file?, #target_rails_version, #target_ruby_version, #annotate, #apply_correction, #attempt_correction, | |
#callback_argument | Reserved for Cop::Cop. |
#complete_investigation | Called to complete an investigation. |
#correct, #current_corrector, | |
#current_offense_locations | Reserved for Commissioner: |
#current_offenses, #currently_disabled_lines, #custom_severity, #default_severity, #disable_uncorrectable, #enabled_line?, #file_name_matches_any?, #find_message, #find_severity, #range_for_original, #range_from_node_or_range, | |
#reset_investigation | Actually private methods. |
#use_corrector |
::RuboCop::Cop::AutocorrectLogic
- Included
#disable_offense, #disable_offense_at_end_of_line, #disable_offense_before_and_after, #disable_offense_with_eol_or_surround_comment, #heredoc_range, #max_line_length, #multiline_ranges, #multiline_string?, | |
#range_by_lines | Expand the given range to include all of any lines it covers. |
#range_of_first_line, #range_overlaps_offense?, #string_continuation?, #surrounding_heredoc?, #surrounding_percent_array? |
::RuboCop::Cop::IgnoredNode
- Included
Constructor Details
This class inherits a constructor from RuboCop::Cop::Base
Instance Attribute Details
#allow_method_comparison? ⇒ Boolean
(readonly, private)
[ GitHub ]
# File 'lib/rubocop/cop/style/multiple_comparison.rb', line 148
def allow_method_comparison? cop_config.fetch('AllowMethodComparison', true) end
Instance Method Details
#comparison?(node) ⇒ Boolean
(private)
# File 'lib/rubocop/cop/style/multiple_comparison.rb', line 127
def comparison?(node) simple_comparison?(node) || nested_comparison?(node) end
#comparisons_threshold (private)
[ GitHub ]# File 'lib/rubocop/cop/style/multiple_comparison.rb', line 152
def comparisons_threshold cop_config.fetch('ComparisonsThreshold', 2) end
#find_offending_var(node, variables = Set.new, values = []) (private)
Metrics/PerceivedComplexity
# File 'lib/rubocop/cop/style/multiple_comparison.rb', line 92
def find_offending_var(node, variables = Set.new, values = []) if node.or_type? find_offending_var(node.lhs, variables, values) find_offending_var(node.rhs, variables, values) elsif simple_double_comparison?(node) return elsif (var, obj = simple_comparison?(node)) return if allow_method_comparison? && obj.send_type? variables << var return if variables.size > 1 values << obj end [variables.first, values] if variables.any? end
#nested_comparison?(node) ⇒ Boolean
(private)
# File 'lib/rubocop/cop/style/multiple_comparison.rb', line 119
def nested_comparison?(node) if node.or_type? node.node_parts.all? { |node_part| comparison? node_part } else false end end
#offense_range(values) (private)
Metrics/PerceivedComplexity
# File 'lib/rubocop/cop/style/multiple_comparison.rb', line 111
def offense_range(values) values.first.parent.source_range.begin.join(values.last.parent.source_range.end) end
#on_or(node)
[ GitHub ]# File 'lib/rubocop/cop/style/multiple_comparison.rb', line 58
def on_or(node) root_of_or_node = root_of_or_node(node) return unless node == root_of_or_node return unless nested_comparison?(node) return unless (variable, values = find_offending_var(node)) return if values.size < comparisons_threshold range = offense_range(values) add_offense(range) do |corrector| elements = values.map(&:source).join(', ') prefer_method = "[#{elements}].include?(#{variable_name(variable)})" corrector.replace(range, prefer_method) end end
#root_of_or_node(or_node) (private)
[ GitHub ]# File 'lib/rubocop/cop/style/multiple_comparison.rb', line 138
def root_of_or_node(or_node) return or_node unless or_node.parent if or_node.parent.or_type? root_of_or_node(or_node.parent) else or_node end end
#simple_comparison?(node) ⇒ Boolean
(private)
# File 'lib/rubocop/cop/style/multiple_comparison.rb', line 131
def simple_comparison?(node) if (var, obj = simple_comparison_lhs?(node)) || (obj, var = simple_comparison_rhs?(node)) [var, obj] end end
#simple_comparison_lhs?(node) (private)
[ GitHub ]# File 'lib/rubocop/cop/style/multiple_comparison.rb', line 82
def_node_matcher :simple_comparison_lhs?, <<~PATTERN (send $lvar :== $_) PATTERN
#simple_comparison_rhs?(node) (private)
[ GitHub ]# File 'lib/rubocop/cop/style/multiple_comparison.rb', line 87
def_node_matcher :simple_comparison_rhs?, <<~PATTERN (send $_ :== $lvar) PATTERN
#simple_double_comparison?(node) (private)
[ GitHub ]# File 'lib/rubocop/cop/style/multiple_comparison.rb', line 79
def_node_matcher :simple_double_comparison?, '(send $lvar :== $lvar)'
#variable_name(node) (private)
[ GitHub ]# File 'lib/rubocop/cop/style/multiple_comparison.rb', line 115
def variable_name(node) node.children[0] end