Home All Groups Group Topic Archive Search About
Author
24 Nov 2007 7:40 PM
Paul Jones
Does anyone know if there is a C# code checker that will spot any (or
preferably all) of the following:
- Unused parameters in functions;
- Unused functions;
- Wrong scope (of functions, variables, enums, etc.,) e.g. can their scope
be reduced from public to protected or private.

Thanks, Paul

Author
24 Nov 2007 7:47 PM
Jon Skeet [C# MVP]
Paul Jones <non***@nomail.com> wrote:
> Does anyone know if there is a C# code checker that will spot any (or
> preferably all) of the following:
> - Unused parameters in functions;
> - Unused functions;
> - Wrong scope (of functions, variables, enums, etc.,) e.g. can their scope
> be reduced from public to protected or private.

FXCop may well do all of those - I can't say I use it myself.

ReSharper (jetbrains.com) is excellent at spotting this kind of thing
within the IDE, but I don't know whether you can tell it to perform the
check outside the IDE and produce a report, for instance.

--
Jon Skeet - <sk***@pobox.com>
http://www.pobox.com/~skeet   Blog: http://www.msmvps.com/jon.skeet
World class .NET training in the UK: http://iterativetraining.co.uk
Author
24 Nov 2007 9:00 PM
Chris
"Paul Jones" <non***@nomail.com> wrote in message
news:07%1j.1989$8k2.1465@newsfe4-win.ntli.net...
> Does anyone know if there is a C# code checker that will spot any (or
> preferably all) of the following:
> - Unused parameters in functions;
> - Unused functions;
> - Wrong scope (of functions, variables, enums, etc.,) e.g. can their scope
> be reduced from public to protected or private.

ReSharper, as John recommended, does all of those things well. Unfortunately
I had to stop using it because of a few silly things, like it made my
context menu ridiculously long.

Chris
Author
24 Nov 2007 11:07 PM
Hilton
Paul,

I've been tempted to develop something along those lines - even something
for my own use.  There are many opportunities here such as the one you've
mentioned, plus others such as (for example) passing OK, Cancel, Retry to a
MessageBox on the Smartphone - that parameter is always invalid, and
changing method to private in a post-op, and notifying you of recursion
especially in "public int Age { get { this.Age; } }".

I use FxCop regularly and it does help, I wish they would update it
occasionally though - FxCop was release 6/2006.

Hilton


Show quote
"Paul Jones" <non***@nomail.com> wrote in message
news:07%1j.1989$8k2.1465@newsfe4-win.ntli.net...
> Does anyone know if there is a C# code checker that will spot any (or
> preferably all) of the following:
> - Unused parameters in functions;
> - Unused functions;
> - Wrong scope (of functions, variables, enums, etc.,) e.g. can their scope
> be reduced from public to protected or private.
>
> Thanks, Paul
>
Author
24 Nov 2007 11:17 PM
Peter Bromberg [C# MVP]
FxCop with "a grain of salt", please. If you run it against the Microsoft
Enterprise Library you'll get a list of errors that will keep you busy for
days...

--Peter
"Inside every large program, there is a small program trying to get out."
http://www.eggheadcafe.com
http://petesbloggerama.blogspot.com
http://www.blogmetafinder.com



Show quote
"Hilton" wrote:

> Paul,
>
> I've been tempted to develop something along those lines - even something
> for my own use.  There are many opportunities here such as the one you've
> mentioned, plus others such as (for example) passing OK, Cancel, Retry to a
> MessageBox on the Smartphone - that parameter is always invalid, and
> changing method to private in a post-op, and notifying you of recursion
> especially in "public int Age { get { this.Age; } }".
>
> I use FxCop regularly and it does help, I wish they would update it
> occasionally though - FxCop was release 6/2006.
>
> Hilton
>
>
> "Paul Jones" <non***@nomail.com> wrote in message
> news:07%1j.1989$8k2.1465@newsfe4-win.ntli.net...
> > Does anyone know if there is a C# code checker that will spot any (or
> > preferably all) of the following:
> > - Unused parameters in functions;
> > - Unused functions;
> > - Wrong scope (of functions, variables, enums, etc.,) e.g. can their scope
> > be reduced from public to protected or private.
> >
> > Thanks, Paul
> >
>
>
>
Author
26 Nov 2007 7:16 AM
Hilton
Peter,

I make sure to only enable the rules I care about.  Do you know if MSFT
plans on updating FxCop?  Do you know of any better tools out there worth
trying?

Thanks,

Hilton


Show quote
"Peter Bromberg [C# MVP]" <pbromberg@yahoo.NoSpamMaam.com> wrote in message
news:62390D19-8705-46B9-B834-5727D3801AB6@microsoft.com...
> FxCop with "a grain of salt", please. If you run it against the Microsoft
> Enterprise Library you'll get a list of errors that will keep you busy for
> days...
>
> --Peter
> "Inside every large program, there is a small program trying to get out."
> http://www.eggheadcafe.com
> http://petesbloggerama.blogspot.com
> http://www.blogmetafinder.com
>
>
>
> "Hilton" wrote:
>
>> Paul,
>>
>> I've been tempted to develop something along those lines - even something
>> for my own use.  There are many opportunities here such as the one you've
>> mentioned, plus others such as (for example) passing OK, Cancel, Retry to
>> a
>> MessageBox on the Smartphone - that parameter is always invalid, and
>> changing method to private in a post-op, and notifying you of recursion
>> especially in "public int Age { get { this.Age; } }".
>>
>> I use FxCop regularly and it does help, I wish they would update it
>> occasionally though - FxCop was release 6/2006.
>>
>> Hilton
>>
>>
>> "Paul Jones" <non***@nomail.com> wrote in message
>> news:07%1j.1989$8k2.1465@newsfe4-win.ntli.net...
>> > Does anyone know if there is a C# code checker that will spot any (or
>> > preferably all) of the following:
>> > - Unused parameters in functions;
>> > - Unused functions;
>> > - Wrong scope (of functions, variables, enums, etc.,) e.g. can their
>> > scope
>> > be reduced from public to protected or private.
>> >
>> > Thanks, Paul
>> >
>>
>>
>>
Author
26 Nov 2007 7:24 AM
Bob Powell [MVP]
If you want serious code analysis get a copy of NDepend. It has a code query
language (CQL) that if the stuff you're looking for isn't already defined
you can use to create a query that will extract whatever information you
like.

www.ndepend.com

--
--
Bob Powell [MVP]
Visual C#, System.Drawing

Ramuseco Limited .NET consulting
http://www.ramuseco.com

Find great Windows Forms articles in Windows Forms Tips and Tricks
http://www.bobpowell.net/tipstricks.htm

Answer those GDI+ questions with the GDI+ FAQ
http://www.bobpowell.net/faqmain.htm

All new articles provide code in C# and VB.NET.
Subscribe to the RSS feeds provided and never miss a new article.


Show quote
"Paul Jones" <non***@nomail.com> wrote in message
news:07%1j.1989$8k2.1465@newsfe4-win.ntli.net...
> Does anyone know if there is a C# code checker that will spot any (or
> preferably all) of the following:
> - Unused parameters in functions;
> - Unused functions;
> - Wrong scope (of functions, variables, enums, etc.,) e.g. can their scope
> be reduced from public to protected or private.
>
> Thanks, Paul
>

AddThis Social Bookmark Button