Sonar Plugins

Rule: Collapsible If statements

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: JAVASCRIPT-1.2
  • Component/s: JavaScript
  • Labels:
    None
  • Number of attachments :
    0

Description

  • Title
    Collapsible If statements
  • Description
    Several 'if' statements can be consolidated by separating their conditions with a boolean short-circuit operator.

Example :

if(condition1) {
  if(condition2){
      doSomething();
  }
}

should be replaced by :

if(condition1 && condition2){
   doSomething();
}
  • Violation message
    Those two 'if' statements can be consolidated.
  • Default severity
    Minor

Part of Sonar Way quality profile

Issue Links

is related to

Task - A task that needs to be done. SONARPLUGINS-2515 Update the SQALE mapping for new rules

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Fabrice Bellingard added a comment -

Manually tested.

Show
Fabrice Bellingard added a comment - Manually tested.
Hide
Permalink
Evgeny Mandrikov added a comment -

Integration tests were updated.

Show
Evgeny Mandrikov added a comment - Integration tests were updated.

People

  • Assignee:
    Evgeny Mandrikov
    Reporter:
    Freddy Mallet
Vote (0)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: