Class: RuboCop::Cop::Lint::EmptyConditionalBody
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::CommentsHelp ,
::RuboCop::Cop::Base ,
::RuboCop::Cop::AutocorrectLogic ,
::RuboCop::Cop::IgnoredNode ,
::RuboCop::Util ,
RuboCop::AST::Sexp
|
|
Inherits: |
RuboCop::Cop::Base
|
Defined in: | lib/rubocop/cop/lint/empty_conditional_body.rb |
Overview
Checks for the presence of if
, elsif
and unless
branches without a body.
Note
|
empty else branches are handled by Style/EmptyElse .
|
Constant Summary
-
MSG =
# File 'lib/rubocop/cop/lint/empty_conditional_body.rb', line 69'Avoid `%<keyword>s` branches without a body.'
::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
::RuboCop::Cop::Base
- Inherited
::RuboCop::Cop::AutocorrectLogic
- Included
Instance Method Summary
- #on_if(node)
- #branch_range(node) private
- #can_simplify_conditional?(node) ⇒ Boolean private
- #deletion_range(range) private
- #else_branch?(node) ⇒ Boolean private
- #empty_if_branch?(node) ⇒ Boolean private
- #flip_orphaned_else(corrector, node) private
- #offense_range(node) private
- #remove_empty_branch(corrector, node) private
::RuboCop::Cop::CommentsHelp
- Included
#comments_contain_disables?, #comments_in_range, #contains_comments?, #source_range_with_comment, #begin_pos_with_comment, #buffer, #end_position_for, | |
#find_end_line | Returns the end line of a node, which might be a comment and not part of the AST End line is considered either the line at which another node starts, or the line at which the parent node ends. |
#start_line_position |
::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_gem_version | Returns a gems locked versions (i.e. |
#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 Method Details
#branch_range(node) (private)
[ GitHub ]# File 'lib/rubocop/cop/lint/empty_conditional_body.rb', line 125
def branch_range(node) if empty_if_branch?(node) && else_branch?(node) node.source_range.with(end_pos: node.loc.else.begin_pos) elsif node.loc.else node.source_range.with(end_pos: node.condition.source_range.end_pos) end end
#can_simplify_conditional?(node) ⇒ Boolean
(private)
# File 'lib/rubocop/cop/lint/empty_conditional_body.rb', line 94
def can_simplify_conditional?(node) node.else_branch && node.loc.else.source == 'else' end
#deletion_range(range) (private)
[ GitHub ]# File 'lib/rubocop/cop/lint/empty_conditional_body.rb', line 133
def deletion_range(range) # Collect a range between the start of the `if` node and the next relevant node, # including final new line. # Based on `RangeHelp#range_by_whole_lines` but allows the `if` to not start # on the first column. buffer = @processed_source.buffer last_line = buffer.source_line(range.last_line) end_offset = last_line.length - range.last_column + 1 range.adjust(end_pos: end_offset).intersect(buffer.source_range) end
#else_branch?(node) ⇒ Boolean
(private)
# File 'lib/rubocop/cop/lint/empty_conditional_body.rb', line 121
def else_branch?(node) node.else_branch && !node.else_branch.if_type? end
#empty_if_branch?(node) ⇒ Boolean
(private)
# File 'lib/rubocop/cop/lint/empty_conditional_body.rb', line 113
def empty_if_branch?(node) return false unless (parent = node.parent) return true unless parent.if_type? return true unless (if_branch = parent.if_branch) if_branch.if_type? && !if_branch.body end
#flip_orphaned_else(corrector, node) (private)
[ GitHub ]# File 'lib/rubocop/cop/lint/empty_conditional_body.rb', line 108
def flip_orphaned_else(corrector, node) corrector.replace(node.loc.else, "#{node.inverse_keyword} #{node.condition.source}") remove_empty_branch(corrector, node) end
#offense_range(node) (private)
[ GitHub ]# File 'lib/rubocop/cop/lint/empty_conditional_body.rb', line 86
def offense_range(node) if node.loc.else node.source_range.begin.join(node.loc.else.begin) else node.source_range end end
#on_if(node)
[ GitHub ]# File 'lib/rubocop/cop/lint/empty_conditional_body.rb', line 71
def on_if(node) return if node.body || same_line?(node.loc.begin, node.loc.end) return if cop_config['AllowComments'] && contains_comments?(node) range = offense_range(node) add_offense(range, message: format(MSG, keyword: node.keyword)) do |corrector| next unless can_simplify_conditional?(node) flip_orphaned_else(corrector, node) end end
#remove_empty_branch(corrector, node) (private)
[ GitHub ]# File 'lib/rubocop/cop/lint/empty_conditional_body.rb', line 98
def remove_empty_branch(corrector, node) range = if empty_if_branch?(node) && else_branch?(node) branch_range(node) else deletion_range(branch_range(node)) end corrector.remove(range) end