Sonar Plugins

Rule: Do not use Array and Object constructors

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

Manually tested, could you just update the description as follows :

<p>There are multiple ways to create objects or arrays in JavaScript. Perhaps the more traditional method is to use the "new" constructor, like so:</p>

<pre>
var o = new Object();  
o.name = 'Jeffrey';  
o.lastName = 'Way';  
}  
</pre>

and

<pre>
var a = new Array();  
a[0] = "Joe";  
a[1] = 'Plumber'; 
</pre>

<p>In both cases, it's recommended to use the much more robust object and array literal methods :</p>

<pre>
var o = {  
   name: 'Jeffrey',  
   lastName = 'Way'   
};
</pre>

and

<pre>
var a = ['Joe','Plumber']; 
</pre>
Show
Freddy Mallet added a comment - - edited Manually tested, could you just update the description as follows : <p>There are multiple ways to create objects or arrays in JavaScript. Perhaps the more traditional method is to use the "new" constructor, like so:</p> <pre> var o = new Object(); o.name = 'Jeffrey'; o.lastName = 'Way'; } </pre> and <pre> var a = new Array(); a[0] = "Joe"; a[1] = 'Plumber'; </pre> <p>In both cases, it's recommended to use the much more robust object and array literal methods :</p> <pre> var o = { name: 'Jeffrey', lastName = 'Way' }; </pre> and <pre> var a = ['Joe','Plumber']; </pre>
Hide
Permalink
Evgeny Mandrikov added a comment -

In fact I believe that description should be like http://google-styleguide.googlecode.com/svn/trunk/javascriptguide.xml#Array_and_Object_literals

Show
Evgeny Mandrikov added a comment - In fact I believe that description should be like http://google-styleguide.googlecode.com/svn/trunk/javascriptguide.xml#Array_and_Object_literals
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: