Using FxCop and StyleCop

Question

0.00/5 (No votes)

See more:

Hello all,

I have been reading information and the usage of FxCop and StyleCop for a day or so. But I also would like to have your opinions? You useful and important to use these two programs. I will be glas if you can share your thoughts with me.

Thanks

Posted 24-Jan-12 11:21am

wonder-FOOL

Add a Solution

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Sergey Alexandrovich Kryukov · Accepted Answer · 2012-01-24T16:24:00

Solution 1

First, about FxCop. I think it is a very useful if you take is critically. You certainly need to adjust the rule sets and customize some rule to get a reasonable check. In my opinion, some rules cannot be used because they are simply idiotic. I give some example with argument in my past answer:

FXCop - How seriously do you take it?[^].

Please also see the opinion of OP Joe_Dert and Andreas Gieriet — they are interesting and make sense. Basically, we all agree.

Now, StyleCop. This is a very different thing. I see no reason to recommend it. I tested it just once to through out in disgust. It's much worse than FxCop. How about such "improvement": using clauses are re-sorted alphabetically. I tend to sort group them semantically, from lower level to application level. We human being remember things semantically, positionally, etc. For alphabetic search, there is a Search dialog. Many other stupid things. I cannot imaging how can it help and why. However, I can imagine some people who might like it.

Style can only be as good as it supports maintenance of the code.

Well, see by yourself.

—SA

Posted 24-Jan-12 16:24pm

Sergey Alexandrovich Kryukov

Updated 24-Jan-12 16:30pm

v2

Comments

wonder-FOOL 25-Jan-12 12:44pm

I have voted 5+ thank you very much for the detailed answer I really appreciate it and respect your solutions. Thanks.

Sergey Alexandrovich Kryukov 25-Jan-12 12:55pm

You are welcome.

Good luck, call again.
--SA

Sergey Alexandrovich Kryukov 25-Jan-12 14:33pm

By the way, how about accepting this answer formally (green button) -- thanks.
--SA

wonder-FOOL 25-Jan-12 19:41pm

:) I know how to accept the answer. At first I have 5ed you because I thought some other people might answer it but seems like noone did it so i went ahead and marked it as an answer now.

Sergey Alexandrovich Kryukov 25-Jan-12 20:04pm

Great, thanks.

Best,
--SA

Andreas Gieriet 25-Jan-12 14:57pm

My 5! I fully agree. What concerns FxCop: Use it and use it wisely. I.e. deliberately decide what rules make sense to you. The rest: ignore (turn off). I have no experience with StyleCop. As part of my work, I review a lot of C# code. There are huge differences in coding styles (I.e. how to express the task at hand in a clear and concise way, using the language features in a decent way). I have my doubts if style can be checked by a tool in a useful way - the topic is too much dependent on taste, too: what is "clear", what is "concise", what is "decent"? In the end it's the whole picture that you as a reviewer either understand or that is so confused that you suspect hidden mistakes or errors everywhere... Cheers Andi

Sergey Alexandrovich Kryukov 25-Jan-12 20:04pm

Thank you Andi.
Great points about the taste.

The individual traits are usually highly under-estimated. This is not just about taste which in principle could be wrong, based on misconception (there is a notion of ill taste), but types of memory and cognitive constitutions in different characters have striking differences. Decent styles can be very different but but comparable. At the same time, people understand each other very well if they learn to do so through experience and thinking about it. Lack of flexibility in formal methods can spoil it, compromise performance and drive.

Thank you, Andi,
--SA