Search for commented-out code across files in Eclipse

Question

Is there a quick way to find all the commented-out code across Java files in Eclipse? Any option in Search, perhaps, or any add-on that can do this?

It should be able to

Answer 1

Sonar can do it: http://www.sonarsource.org/commented-out-code-eradication-with-sonar/

Answer 2

You can do a search in Eclipse.

All you need to search for is /* and //

However, you will only find the files which contain that expression, and not the actual content which I believe you are after.

However, if you are using Linux you can easily get all the comments with a one liner.

Answer 3

If the problem is to find commented-out code, what is needed is a way to find comments, and way to decide if a comment might contain code.

A simple way to do this is to search for comment that contain code-like things. I'd be tempted to hunt for comments containing a ";" character (or some other rare indicator such as "="); it will be pretty hard to have any interesting commented code that doesn't contain this and in my experience with comments, I don't see many that people write that contain this. A regexp search for this should be pretty straightforward, even if it picked up a few addtional false positives (e.g. // in a string literal).

A more sophisticated way to accomplish this is to use a Java lexer or parser. If you have a lexer that returns comments at tokens (not all of them do, Java compilers aren't interested in comments), then you can simply scan the lexemes for a comment and do the semicolon check I described above. You won't get any false positives hits for comment like things in string literals with this approach.

If you have a re-engineering parser that captures comments as part of the AST ( such as our SD Java Front End), you can mechanically scan the parse tree for comments, feed the comment context back to the parser to see if the content is code like, and report any that passes that test modulo some size-depedent error rate (10 errors in 15 characters implies "really is a comment"). Now the "code-like" test requires the reengineering parser be willing to recognize any substring of the (Java) language. Our DMS Software Reengineering Toolkit underlying the Java Front End can actually do that, using access to the grammar buried in the front end, as it is willing to start a parse for any language (non)terminal, and this question is "can you find a sequuence of (non)terminals that consumes the string?".

The lexer and parser approaches are small and big sledgehammers respectively. If OP is going to do this just once, he can stick to the manual regex search. If the problem is to vet the code base repeatedly (needed in big organizations), he'd want a tool that can be run on regular basis.

Answer 4

@Jorn said:

I think [the OP] wants to find code that is commented out, not code that has a comment.

If the intention is to find commented out code, then I don't think it is possible in general. The problem is that it is impossible to distinguish between comments that were written as code or pseudo-code, and code that is commented out. Making that distinction requires human intelligence.

Now IDE's typically have a "toggle comments" function that comments out code in a particular way. It would be feasible to write a tool / plugin that matches the style produced by a particular IDE. But that's probably not good enough, especially since reformatting the code typically gets rid of the characteristics that made the commented out code recognizable.

Answer 5

You can mark your own commented code with a task tag. You can create your own task tags in Eclipse.

From the menu, go to Window -> Preferences. In the Preferences dialog, go to General -> Editors -> Structured Text Editors -> Task Tags.

Add an appropriate task tag, like COMMENTED. Set the priority to Low.

Then, any code you comment out, you can mark with the COMMENTED task tag. A list of these task tags, along with their locations, appears in the Tasks view.

Answer 6

In Eclipse, I just do a file search with the regular expression checkbox turned on:

(/\*.*;.*\*/)|(//.*;)

It will find semicolons in

// These;

and /* these; */

Works for me.