Windows LiveWindows Live™
 
 
Claudio's profileClaudio Lassala in Softw...PhotosBlogListsMore Tools Help

Blog
    May 27

    Checking out the Source Analysis for C#

    I've just spent about 30 minutes checking out the recently announced Microsoft Source Analysis for C#. I definitely love the idea, but I'm not sure I'll be using it just yet. Here are some of my impressions (again, I haven't spent a lot of time on it yet)...

    All using directives must be placed inside of the namespace

    Hmm, but VS itself violates that rule on every class we create...!

    Fig01

    The property must not be placed on a single line. The opening and closing curly brackets must each be placed on their own line

    Huh? But that an auto-implemented property. What would we want to split that into multiple lines?!

    Fig02

    Variable names must not start with 'm_'

    Hmpf, that's the standard with have here at the company...

    Fig03

    A closing curly bracket must not be preceded by a blank line

    Ok, this one I like.

    Fig04

    Tabs are not allowed. Use spaces instead

    Huh? Thanks, but no, thanks. I've seen a lot of people complaining online about this one too. What's the problem with using tabs??

    Fig05

    The call ... must begin with 'this.'...

    Ah, I like that one. This is something we've made as part of our guideline here, but couldn't use FxCop to trap for that, since the 'this.' doesn't make into the IL.

    Now, the problem here is that, again, the code that VS generates violates this rule all over the place, such as on the example below:

    Fig06

    I'll continue to use this tool to see if it's going to stick with me. I've tried it on a very small project, and got 246 violations; the vast majority where produced by code generated by VS. On a big project, I'd probably end up with a LOT more violations, so I'm not sure I can live with that. I believe there is a way to disable some of the rules, but haven't looked for it just yet...

    Anyway, I do appreciate Microsoft releasing tools like this one. Hopefully, more and more developers will start paying more attention to how they write code.

    5:24 PM | Blog it

    Comments (8)

    Please wait...
    Sorry, the comment you entered is too long. Please shorten it.
    You didn't enter anything. Please try again.
    Sorry, we can't add your comment right now. Please try again later.
    To add a comment, you need permission from your parent. Ask for permission
    Your parent has turned off comments.
    Sorry, we can't delete your comment right now. Please try again later.
    You've exceeded the maximum number of comments that can be left in one day. Please try again in 24 hours.
    Your account has had the ability to leave comments disabled because our systems indicate that you may be spamming other users. If you believe that your account has been disabled in error please contact Windows Live support.
    Complete the security check below to finish leaving your comment.
    The characters you type in the security check must match the characters in the picture or audio.

    To add a comment, sign in with your Windows Live ID (if you use Hotmail, Messenger, or Xbox LIVE, you have a Windows Live ID). Sign in


    Don't have a Windows Live ID? Sign up
    neslihanwrote:
    very good . Thanks
    July 30
    Lex Liwrote:
    It is rational to use spaces because not everyone uses the same tab settings in his/her IDE. In my case, when I navigate source code files on CodePlex.com, tabs are misaligned (CodePlex web page uses more spaces than 4).
     
    Also I find it a must to rurn off some rules (persoanlly I hate "this", and ReSharper hates it, too) on legacy projects. Otherwise, over 10,000 warnings may be provided by StyleCop.
    July 28
    Alfred Myerswrote:
    As a matter of fact, "this" ALWAYS makes its way into IL.
    private string foo;

    public string Foo

    {

    get

    {

    return foo;

    }

    set

    {

    this.foo = value;

    }

    }

     

    gets translated to

    method public hidebysig specialname instance string get_Foo() cil managed
    {
        .maxstack 1
        .locals init ( [0] string CS$1$0000)
        L_0000: nop
        L_0001: ldarg.0
        L_0002: ldfld string ItauTests.Program::foo
        L_0007: stloc.0
        L_0008: br.s L_000a
        L_000a: ldloc.0
        L_000b: ret
    }
    .method public hidebysig specialname instance void set_Foo(string 'value') cil managed { .maxstack 8 L_0000: nop L_0001: ldarg.0 L_0002: ldarg.1 L_0003: stfld string ItauTests.Program::foo L_0008: ret } arg.0 is "this"If you use .NET Reflector you'll get:public string Foo { get { return this.foo; } set { this.foo = value; } }

     

    July 8
    Janewrote:
    In the Detailed Settings pane you can choose to switch off "Analyze designer files" and "Analyze generated files" which I found help remove the number of warnings. 
    July 8
    Don Awaltwrote:
    Be wary of this add-on. When I added it to my Windows Server 2008 dev system for VS 2008, when I went into Project properties a couple of days later, I received an error message "COM object that has been separated from its underlying RCW cannot be used" and nothing showed up on any of the Properties pages! I un-installed this add-on and the problem went away.

    Not worth the hassle, besides I too got a bunch of really un-useful reports. Very hard to find the gems in all the fluff.
    May 30
    No namewrote:
    "What's the problem with using tabs??"
    You can change how many spaces wide a tab looks in the IDE.  That's the problem.  are they 2 spaces, 4, 8, or what?  Visual Studio defaults them to 4, but meanwhile, most diff tools display them at 8.

    As soon as tabs mix with spaces, then formatting goes to hell.

    That said, that's the only one of these rules that I agree with.
    May 30
    Serge Baranovskywrote:
    Holy crap!
     
    Sorry about the duplicate comments ... don't know how they came up. I only submitted once. Apparently Live Spaces have issues with Firefox... Any way you could delete the duplicate comments? I don't seem to find a way to delete my own.
     
    Sorry about the inconvenience again...
    May 28
    Serge Baranovskywrote:
    Hi Claudio,

    Thank you for the great review. I too advocate for developers to adopting good coding style.

    I wonder if you like our CodeIt.Right tool - http://submain.com/codeit.right - it is more efficient as it can automatically correct violations and code smells for you.

    Please feel free to contact me if you have any questions or are interested in a review copy.

    May 28

    Trackbacks

    The trackback URL for this entry is:
    http://claudiolassala.spaces.live.com/blog/cns!E2A4B22308B39CD2!779.trak
    Weblogs that reference this entry
    • None