new Check: commented code

[romani]

SForge issue: 453
Created: 2007-03-07
Creator: caucabin
Private: No

In source code ,always see like this code:

// int size = p_steps.size();
// for (int i = 0; i < size; i++)

 final int size = p_steps.size();
 for (int i = 0; i < size; i++)

The comment part is a history source code, we can get the history code from the version control tool, and don't want to see it in the code.

This already done in Sonar - http://www.sonarqube.org/commented-out-code-eradication-with-sonar/.
We should think about this.

[daniilyar]

@romani how do we distinguish between commented code and comments?

Detection by existence of some symbols specific to the code would be unreliable and I see to other way to distinguish it.

So I believe this one is impossible to implement in Checkstyle (

[romani]

Commented code should be parse able. By wrapping to class or class+method. If it is not parse-able it is a comment that have some code.

Sonar also have problems to detect this properly, there are cases where we need to put code in comment, rare cases but happening.