SonarQube Plugins

Rule: Detect conditional operators

Details

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

Description

<p>
Detects inline conditionals. An example inline conditional is this:
</p>

<pre>
function sayHello()

{ var a = (a === 'A') ? 'is A' : 'is not A'; }

</pre>

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Eriks Nukis added a comment -

Fixed and pull request https://github.com/SonarCommunity/javascript/pull/1 created

Show
Eriks Nukis added a comment - Fixed and pull request https://github.com/SonarCommunity/javascript/pull/1 created
Hide
Permalink
Evgeny Mandrikov added a comment -

Has been merged.

Show
Evgeny Mandrikov added a comment - Has been merged.
Hide
Permalink
Freddy Mallet added a comment -

Works well but :

  • I guess it's better to replace "Inline conditional" by "Conditional operator"
  • The description should be : Condition operators can quickly become hard to be read. The following code snippet illustrates this rule :"
  • The violation message should be "Replace this conditional operator by a standard if/else control flow statement."
Show
Freddy Mallet added a comment - Works well but : I guess it's better to replace "Inline conditional" by "Conditional operator" The description should be : Condition operators can quickly become hard to be read. The following code snippet illustrates this rule :" The violation message should be "Replace this conditional operator by a standard if/else control flow statement."
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:
    Eriks Nukis
    Reporter:
    Eriks Nukis
Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: