views:

828

answers:

5

I got a C# application with Visual Studio 2005. The code has some 300KLOC code, which has been raped over 3 years.

When I was going through code found out that lot of unused functions/methods/properties. Its not possible to clean the code manually (Requires checking each member and if found no references delete it.)

I am looking to automate this process, by VS macro, which will walk through each member in code, if it does not have any references it should delete it, if found any references, check its calling member's references if calling member, does not have any reference it should delete both and so on.

I am sure some one would have cracked it earlier.

+12  A: 

Resharper has a Clean Code function and gives pretty good indication of which methods/classes aren't being used.

Ruben Steins
I'm not sure if it's automated or not... but Resharper indicates whether a method has any references.
Chuck Conway
+3  A: 

You can query your code base with NDepend using CQL to find out which methods and classes are not being used.

idursun
NDepend will help you even further in understanding the codebase by providing you info about most used types, dependencies between types, direct and indirect dependencies, measured complexity of types and methods, etc.
flq
A: 

1st use re-sharper as other have suggested.

Resharper assumes public methods are used somewhere by an external assembly, even if they are not, so search and replace "public " with "private " and recompile.

MatthewMartin
+3  A: 

Be careful of code that is invoked via reflection. A lot of refactoring tools will flag this code as not being accessed when in fact they are.

The safest is to run your unit tests (you do have these already right?) before and after the refactoring to ensure that everything still works.

rein
A: 

I have used Gendarme (like FxCop) to determine which code wasn't called. AFAIK it can't be automated, but at least you don't need to go line by line.

Nathan Koop