Friday, June 8, 2012

Are there any suggestions for developing a C# coding standards / best practices document?


I'm a recent AI graduate (circa 2 years) working for a modest operation. It has fallen to me (primarily as I'm the first 'adopter' in the department) to create a basic (read useful?) C# coding standards document.



I think I should explain that I'm probably the most junior software engineer going, but I'm looking forward to this task as hopefully I might actually be able to produce something half usable. I've done a pretty extensive search of the Internet and read articles on what a coding standards document should / should not contain. This seems like a good as place as any to ask for some suggestions.



I realise that I am potentially opening a door to a whole world of disagreement about 'the best way to do things'. I both understand and respect the undeniable fact that each programmer has a preferred method of solving each individual task, as a result I'm not looking to write anything so draconianly proscriptive as to stifle personal flair but to try and get a general methodology and agreed standards (e.g. naming conventions) to help make individuals code more readable.



So here goes .... any suggestions? Any at all?


Source: Tips4all

15 comments:

  1. We start with


    Microsoft's .NET guidelines: http://msdn.microsoft.com/en-us/library/ms229042.aspx (link updated for .NET 4)
    Microsoft's C# guidelines: http://blogs.msdn.com/brada/articles/361363.aspx.


    and then document the differences from and additions to that baseline.

    ReplyDelete
  2. IDesign has a C# coding standards document that is commonly used. Also see the Framework Design Guidelines 2nd Ed.

    ReplyDelete
  3. Ironically setting the actual standards are likely to be the easy part.

    My first suggestion would be to elicit suggestions from the other engineers about what they feel should be covered, and what guidelines they feel are important. Enforcing any kind of guidelines requires a degree of buy-in from people. If you suddenly drop a document on them that specifies how to write code you'll encounter resistance, whether you're the most junior or senior guy.

    After you have a set of proposals then send them out to the team for feedback and review. Again, get people to all buy into them.

    There may already be informal coding practices that are adopted (e.g prefixing member variables, camelcase function names). If this exists, and most code conforms to it, then it will pay to formalize its use. Adopting a contrary standard is going to cause more grief than it's worth, even if it's something generally recommended.

    It's also worth considering refactoring existing code to meet the new coding-standards. This can seem like a waste of time, but having code that does not meet the standards can be counter-productive as you will have a mish-mash of different styles. It also leaves people in a dilemma whether code in a certain module should conform to the new standard or follow the existing code style.

    ReplyDelete
  4. I have always used Juval Lowy's pdf as a reference when doing coding standards / best practices internally. It follows very close to FxCop/Source Analysis, which is another invaluable tool to make sure that the standard is being followed. Between these tools and references, you should be able to come up with a nice standard that all your developers won't mind following and be able to enforce them.

    ReplyDelete
  5. The other posters have pointed you at the baseline, all I would add is make your document short, sweet, and to the point, employing a heavy dose of Strunk and White to distinguish the "must haves" from the "it would be nice ifs".

    The problem with coding standards documents is that nobody really reads them like they should, and when they do read them, they don't follow them. The likelihood of such a document being read and followed varies inversely with its length.

    I agree FxCop is a good tool but too much of this can take all the fun right out of programming, so be careful.

    ReplyDelete
  6. Never write your own coding standards use the MS ones (or the Sun ones or ... as appropriate for your language). The clue is in the word standard, the world would be a much easier place to code in if each organization hadn't decided to write their own. Who really thinks learning a new set of 'standards' each time you change teams/projects/roles is a good use of anyone's time.
    The most you should ever do is summarize the critical points but I'd advise against doing even that because what is critical varies from person to person.
    Two other points I'd like to make on coding standards


    Close is good enough - Changing code to follow coding standards to the letter is a waste of time as long as the code is close enough.
    If you're changing code you didn't write follow the 'local coding standards', i.e. make your new code look like the surrounding code.


    These two points are the reality to my wish that everybody would write code that looked the same.

    ReplyDelete
  7. I found the following documentation very helpful and concise. It comes from the idesign.net site and it is authored by Juval Lowy

    C# Coding Standard

    ReplyDelete
  8. I would be tempted to enforce Microsoft's StyleCop as the standard. It can be enforced at the build time. but if you have legacy code then just enforce using StyleCop on new code.

    http://code.msdn.microsoft.com/sourceanalysis

    Eventually it will have a refactor option to cleanup code.

    http://blogs.msdn.com/sourceanalysis/

    ReplyDelete
  9. I would add Code Complete 2 to the list (I know Jeff is kind of a fan here)... if you are a junior developer the book comes handy to setup ur mind in a way that sets the foundation in ur mind for the best code writing practices and software building there is.

    I got to say that I came a bit late in my carreer to it, but it rules a lot of the ways I think about coding and framework development in my professional life.

    Worth check it out ;)

    ReplyDelete
  10. Personally I like the one that IDesign has put together. But that's not why I'm posting...

    The tricky bit at my company was taking all the different languages into account. And I know my company isn't alone on this. We use C#, C, assembly (we make devices), SQL, XAML, etc. Although there will be some similarities in standards, each is usually handled differently.

    Also, I believe that higher level standards have a greater impact on the quality of the final product. For example: how and when to use comments, when exceptions are mandatory (e.g. user initiated events), whether (or when) to use exceptions vs. return values, what is the objective way to determine what should be controller code vs presentation code, etc. Don't get me wrong, low level standards are also needed (formatting is important to readability!) I just have a bias towards overall structure.

    Another piece to keep in mind is buy-in and enforcement. Coding standards are great. But if nobody agrees with them and (probably more importantly) no one enforces them then it's all for naught.

    ReplyDelete
  11. Microsoft's own rules are an excellent starting point. You can enforce them with FxCop.

    ReplyDelete
  12. I've just started at a place where the coding standards mandate the use of m_ for member variables, p_ for parameters and prefixes for types, such as 'str' for strings.
    So, you might have something like this jouer au site de casinos virtuels in the body of a method:

    m_strName = p_strName;

    Horrible. Really horrible.

    ReplyDelete
  13. SSW Rules

    It includes some C# standards + a whole lot more.... primarily focused at Microsoft developers

    ReplyDelete
  14. Why not use .NET standards?

    http://www.irritatedvowel.com/Programming/Standards.aspx

    ReplyDelete
  15. You are most likely being set up to fail. Welcome to the industry.


    I disagree - so long as he creates the document, the worst that can happen is that it gets forgotten by everyone.

    If other people have issues with the content, then you can ask them to update it to show what they'd prefer. That way it's off your plate, and the others have the responsibility to justify their changes.

    ReplyDelete