Sonar Plugins

Rule: Do not use wrapper objects for primitive types

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

Issue Links

depends upon

New Feature - A new feature of the product, which has yet to be developed. SONARPLUGINS-1788 Provide JavaScript rule engine based on SSLR

  • 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 - - edited

Woks well could you just update the description as follows :

<p>Use of new Number, new String, and new Boolean produce unnecessary object wrappers. Just use simple literals instead.</p>
Show
Freddy Mallet added a comment - - edited Woks well could you just update the description as follows : <p>Use of new Number, new String, and new Boolean produce unnecessary object wrappers. Just use simple literals instead.</p>
Hide
Permalink
Evgeny Mandrikov added a comment -

In fact they not only produce unnecessary object wrapper, but also can be dangerous: http://google-styleguide.googlecode.com/svn/trunk/javascriptguide.xml#Wrapper_objects_for_primitive_types

Show
Evgeny Mandrikov added a comment - In fact they not only produce unnecessary object wrapper, but also can be dangerous: http://google-styleguide.googlecode.com/svn/trunk/javascriptguide.xml#Wrapper_objects_for_primitive_types
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:
    Evgeny Mandrikov
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: