Sonar Plugins

Rule: Do not use more than one "break" or "continue" statement in loops

Details

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

Issue Links

is related to

Improvement - An improvement or enhancement to an existing feature or task. SONARPLUGINS-2764 The rule "Do not use more than one 'break' or 'continue' statement in loops" should feed the rule_failures.cost column

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.

Task - A task that needs to be done. SONARPLUGINS-2743 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
Freddy Mallet added a comment -

Works well Evgeny, could you just update :

  • the description
    • The 'break' and 'continue' branching statements increase the essential complexity of the source code by removing the ability to linearly read the flow of statements. Using one 'break' or 'continue' statement in a loop should be highly motivated but is acceptable since this allows optimal coding in some specific cases. However using more than one of those statements in a loop leads to unmaintainable source code.
Show
Freddy Mallet added a comment - Works well Evgeny, could you just update : the description The 'break' and 'continue' branching statements increase the essential complexity of the source code by removing the ability to linearly read the flow of statements. Using one 'break' or 'continue' statement in a loop should be highly motivated but is acceptable since this allows optimal coding in some specific cases. However using more than one of those statements in a loop leads to unmaintainable source code.
Hide
Permalink
Evgeny Mandrikov added a comment -

Integration tests were updated.

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

Done.

Show
Evgeny Mandrikov added a comment - Done.
Hide
Permalink
Freddy Mallet added a comment -

Manually tested !

Show
Freddy Mallet added a comment - Manually tested !

People

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

Dates

  • Created:
    Updated:
    Resolved: