jQuery Grid Plugin - jqGrid - Topic: small improvements of the jqGrid code with respect of ReSharper 6 http://www.trirand.com/blog/?page_id=393/bugs/small-improvements-of-the-jqgrid-code-with-respect-of-resharper-6 Simple:Press Version 5.7.5.3 OlegK on small improvements of the jqGrid code with respect of ReSharper 6 http://www.trirand.com/blog/?page_id=393/bugs/small-improvements-of-the-jqgrid-code-with-respect-of-resharper-6#p25674 Bugs http://www.trirand.com/blog/?page_id=393/bugs/small-improvements-of-the-jqgrid-code-with-respect-of-resharper-6#p25674 Hello Tony,

I use ReSharper 6.1 for my projects and find it's suggestions very clear. It can help in one click to fix the problem. If I open the current code of jqGrid I can see many small problems identivied by ReSharper. It's mostly the following:

  • declared, but not used parameters in the function. It's very common problem in the usage of $.each or in the event handlers. For example, the line
  • value assigned, but not used in any execution path. For example the line.
  • declaration of variable in the function with the same name as in the outer scope. For example, declaration i variable which can be easy renamed in j or k to fix the problem.
  • the usage of variables which start with capical character, undescore (_) or the usage of undescore inside of the name.
  • usage of possibly unassigned value.
  • ...

Description of all the small problems which are not really errors can get many time, but I could make the most clear and transparent changes (for example the first and the second problems from the list above) for beginning and post the results as a pull request. What do you think about?

Best regards
Oleg 

]]> Sun, 15 Jan 2012 23:22:06 +0200 tony on small improvements of the jqGrid code with respect of ReSharper 6 http://www.trirand.com/blog/?page_id=393/bugs/small-improvements-of-the-jqgrid-code-with-respect-of-resharper-6#p24259 Bugs http://www.trirand.com/blog/?page_id=393/bugs/small-improvements-of-the-jqgrid-code-with-respect-of-resharper-6#p24259 Hello Oleg,

Just do these fixes in GitHub. Thanks again. Going to look at the others.

Kind Regards

]]> Sun, 14 Aug 2011 11:30:57 +0300 OlegK on small improvements of the jqGrid code with respect of ReSharper 6 http://www.trirand.com/blog/?page_id=393/bugs/small-improvements-of-the-jqgrid-code-with-respect-of-resharper-6#p24175 Bugs http://www.trirand.com/blog/?page_id=393/bugs/small-improvements-of-the-jqgrid-code-with-respect-of-resharper-6#p24175 It's just an reminder. You promissed to read my suggestions. ReSharper could help to remove unused parameters used in callback functions in defferent places of jqGrid. It point additionally to some places where uninitialized variables could be used. All this is not clitical, but can improve the code.

Please read another bug report too which I resently posted.

Best regards
Oleg

]]> Sat, 06 Aug 2011 18:34:24 +0300 tony on small improvements of the jqGrid code with respect of ReSharper 6 http://www.trirand.com/blog/?page_id=393/bugs/small-improvements-of-the-jqgrid-code-with-respect-of-resharper-6#p24080 Bugs http://www.trirand.com/blog/?page_id=393/bugs/small-improvements-of-the-jqgrid-code-with-respect-of-resharper-6#p24080 Thanks Oleg,

I will check these at end of week. Will let you know

Regards

Tony

]]> Thu, 28 Jul 2011 12:41:12 +0300 OlegK on small improvements of the jqGrid code with respect of ReSharper 6 http://www.trirand.com/blog/?page_id=393/bugs/small-improvements-of-the-jqgrid-code-with-respect-of-resharper-6#p24074 Bugs http://www.trirand.com/blog/?page_id=393/bugs/small-improvements-of-the-jqgrid-code-with-respect-of-resharper-6#p24074 Hello Tony,

The new version of ReSharper (http://www.jetbrains.com/resharper/) gives many hints which halps to improve a litle JavaScript code of jqGrid. If you have an interest I could post you the recomendations one after another or you can buy the tool yourself.

Here is example of recommendation about grid.grouping.js:

1) the line 77 is

$.each(grp.summaryval[0][itm],function(i,n) {

which can be reduced to

$.each(grp.summaryval[0][itm],function() {

because i and n parameters will be never used.

2) In the line 100 the variable collapsed are initialized to false, so the line 130 with the same assignment can be removed.

3) In the line 141 the variable pmrtl will be initialized to "", but the value will be never used. Instead of that in the lines 153-154 the variable pmrtl will be set to grp.plusicon or grp.minusicon depend on grp.groupCollapse. One can for example remove the lines 153-154 and replace in the line 141 (initialization of pmrtl)

pmrtl = ""

 to

pmrtl = grp.groupCollapse? grp.plusicon: grp.minusicon

4) The last parameter current of the function groupingGroupBy defined in the line 217 can be removed, because it will be never used inside.

5) Variable i will be defined in the  function groupingGroupBy twice (in the line 226 and the line 232). Die suggestion is to insert the declaration of i in the line 219 or 223 (which can be concatinated by the way) and renove "var " from the lines 226 and 232.

It is an example of advices which gives ReSharper. If you have an interest to make the corresponding changes in the jqGrid I could post you one after another advices to all other jqGrid modules.

Best regards
Oleg 

]]> Thu, 28 Jul 2011 01:33:51 +0300