|
There may have been a better place to post this - but the intent is good
I think 'C# style guides' are worthwhile, but, you need to have a living document, and you may well change your stance on something over time - I also think you'd have to provide examples - for example 'Big O Notation' - not many [cough] younger programmers these days I suspect would know or care what that means.
You could also include - use of a code repository, and give basic instructions on maintaining a project within a repository, AND, you could also include suggested directory structure for a project, especially where you have sub projects, 3rd party libraries, shared utility libraries of your own, etc
'g'
|
|
|
|
|
Garth,
I believe you are right or have a good point in everything you have mentioned. I will explain motivations not because I am defensive, but because it adds details to the discussion.
I do agree about needing a living document; I do have some problems finding the right technology however. Once I have (more) coworkers, their input is a must; for a limited time PDF works fine.
About changing the stance, I not only agree, but can admit that it already has happened a few times (E.g. what it says about lambdas). Big O notation: I do not want to belittle younger guys who haven't heard about it. In some sense this style is an example of mild project-dependence: In my particular application not being carefull with Big O notation would doom it pretty fast. And, on project dependency: It is somewhat hurtfull for me to be quiet on scheduling multiple processes, however my time limits and the specific purpose of the guide lead me to ommit features which are not relevant in my particular context. I found that admitting and even using some project dependencies made it much easier to come up with a useful guide.
Code repositary, directory structures, maintenance policy are all out for 2 simple reasons:
1) I'm not experienced enough to know the best ways myself. 2) Once decided I think they will not be part of a style guide; they will be part of mandatory project rules.
About a better place to post: I did some research to find a good place; I apologize if I wasn't thorough enough and used a wrong place. If people who know the right structure for this website can improve the placement, I am thankful.
By the way if that other guy calls it spam, he is not completely off, just a little impolite:
1) I do have a spam component: Attracting the right guys around me was part of my goal...
2) I hate spam so much that I am willing to forgive some knee-jerk reactions against something he considers spam.
Chris
|
|
|
|
|
BTW : I've sent a note to the 'Hamsters' requesting this be moved somewhere 'more appropriate'
|
|
|
|
|
Guide reads: We [...] don't want any rules which makes their life harder.
So, the code is automatically formatted on each checkin?
Guide reads: We prefer standard SQL. **
Which one? I prefer the 92 standard
Bastard Programmer from Hell 
If you can't read my code, try converting it here[^]
|
|
|
|
|
You hit a slightly sour point in my standard:
a) I think most rules are on a level a little bit higher then what automatic formatters can do.
b) Fixing output of automatic formatters is painful. However, readability is so important to me that I pay the price.
I am takeing your word on SQL, at least until a coworker will bring up that issue again. That choice may be important in a particular project, but I don't think the choice adds much value to a somewhat more generic style guide discussion.
(I saw your )
|
|
|
|
|
Chris Jacobi wrote: You hit a slightly sour point in my standard:
I know.
Chris Jacobi wrote: I think most rules are on a level a little bit higher then what automatic formatters can do.
I would not waste any time on formatting to someones personal preference. Yes, it is important, but it is not a religion.
Chris Jacobi wrote: However, readability is so important to me that I pay the price
Readability is the most important thing to keep a project alive, I'll agree with that.
Don't get me wrong; we're doing a lot in a similar way. Then again, the fact that one does not comment the obvious should not be part of a style-guide - it should be part of the education. Same goes for the SOLID principles. No, I don't like it as a compulsory style guide; it'd take too damn long to verify each rule against the code I wrote to see if there's a violation.
Bastard Programmer from Hell
If you can't read my code, try converting it here[^]
|
|
|
|
|
Wordy, verbose, you should use your own style guide when writing the document (as in, liberal use of whitespace), abstract, vague, treads on far too many topics having nothing to do with style...I could go on. This needs some serious polishing. A good style guide is short, to the point, and provides a simple "good" and "bad" example so the desired style is clear -- no wiggle room for interpretation.
Chris Jacobi wrote: Alternatively, would it deter first class programmers because they might oppose using a somewhat unusual style?
From what I read through there's nothing particularly unusual about what you write -- most of it is common sense. But who in the world is going to police an 11 page document?
And what in the world does "Agile" have to do with "style?"
My advice: cut the document by replacing 90% of your words with short "do/don't do" examples. Imagine something that can fit on a 4x6 two-sided notecard.
Marc
|
|
|
|
|
Marc,
thanks for your comment. I have never written a style guide for the English language. Only a fool or an elementary school teacher would make the effort to read what I have to say about the English language; I certainly cannot refute your statement on polish.
I disagree on whatever is the proper length. When I researched style guides I found some very short and to the point guides; I ignored many simply because I couldn't see the reasoning behind what they were saying. There is a 350 page book "Framework Design Guidelines" (mostly C#) [Google]. I think it is well written and has interesting inserts from different personalities. Nevertheless, I ignored most of it, mostly because it doesn't respect the limits of my attention span. Somewhat because is too general and doesn't contain what is specially important for my startup. And lastly, they didn't dare to disagree loudly enough with other standards.
I think having some wiggle room is treating my cowerkers as professional adults. For the same reason it has very little trivial content. Examples would be nice, but most is for the benefit of programming-in-the-large, where I find it difficult to make reasonable examples. "Policing" is not something I like to do; The people I want to work with earn respect and absence of being policed.
If you think there is nothing unusual and most is common sense, I think that this is a compliment to me, but in my opinion it also compliments on you.
What has "Agile" have to do with "style?" Good question, but I am taking some wiggle room myself defining what is style. In what other document could I ramble about obsessive agile-ness in a reasonable way?
And for anybody who has read till here: Google Marc on CodeProject; he has written some nice articles.
Chris
www.whitehawksoftware.com
Edited to spell Marc's name corrctly.
modified 19-Feb-14 15:49pm.
|
|
|
|
|
Please take the opportunity to spell Marc's name properly. I know you don't mean it, but getting someone's name wrong is very disrespectful.
|
|
|
|
|
Pete O'Hanlon wrote: but getting someone's name wrong is very disrespectful.
Making a mistake is never disrespectful.
Bastard Programmer from Hell
If you can't read my code, try converting it here[^]
|
|
|
|
|
Doing it twice is, especially when you can see their name right there in front of you.
|
|
|
|
|
Pete O'Hanlon wrote: Doing it twice is, especially when you can see their name right there in front of you.
Call it a bug; one can overlook a simple typo quickly - let's not suggest that it has anything to do with respect.
Bastard Programmer from Hell
If you can't read my code, try converting it here[^]
|
|
|
|
|
Marc doesn't need any help defending himself. He's got cred.
If your actions inspire others to dream more, learn more, do more and become more, you are a leader.-John Q. Adams
You must accept one of two basic premises: Either we are alone in the universe, or we are not alone in the universe. And either way, the implications are staggering.-Wernher von Braun
Only two things are infinite, the universe and human stupidity, and I'm not sure about the former.-Albert Einstein
|
|
|
|
|
He's also got me. I'll go all Krav Maga on anyone who disses my main man.
|
|
|
|
|
Shields down! Deactivate waepons! At ease! Sounds like you need extended shore leave.
If your actions inspire others to dream more, learn more, do more and become more, you are a leader.-John Q. Adams
You must accept one of two basic premises: Either we are alone in the universe, or we are not alone in the universe. And either way, the implications are staggering.-Wernher von Braun
Only two things are infinite, the universe and human stupidity, and I'm not sure about the former.-Albert Einstein
|
|
|
|
|
Sounds like a plan to me.
|
|
|
|
|
I mostly agree with Marc that the baseline of the style guide should be short and to the point. Supplemental material explaining why can be helpful; but being able to fit the essentials onto at most a single sheet of paper so it can be stuck to the wall for easy reference by anyone new to the team is more useful.
If you want to keep it in PDF form, the best option is probably just to add a 1 page short guide that only has the what not the why. If you're willing to switch to HTML format putting the explanation in default collapsed blocks would be better.
Did you ever see history portrayed as an old man with a wise brow and pulseless heart, waging all things in the balance of reason?
Is not rather the genius of history like an eternal, imploring maiden, full of fire, with a burning heart and flaming soul, humanly warm and humanly beautiful?
--Zachris Topelius
Training a telescope on one’s own belly button will only reveal lint. You like that? You go right on staring at it. I prefer looking at galaxies.
-- Sarah Hoyt
|
|
|
|
|
|
Running Windows users with ‘standard’ rather than ‘administrator’ rights would have removed over 90 percent of the risk posed by critical vulnerabilities reported in Microsoft products last year, an analysis by privilege management firm Avecto has found. Taking computers away from users removes 100% of the risk
|
|
|
|
|
If only Microsoft had introduced a way for everyone to run as a standard user, with a prompt to elevate to administrator when required. I'm sure nobody would ever complain about it, or turn it off because it got in the way of them opening the "FluffyBunniesAndTaxRefund.exe" attachment that the tax man just emailed to them from a .cn address. 
"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer
|
|
|
|
|
How surprising. A company that has a product that is used in companies like banks to run applications with elevated privileges is pushing for computers to be run in a state where you need a product that runs applications with elevated privileges. #Whodathunkit.
|
|
|
|
|
After being used to typing "su" on the prompt, one wonders why it took so damn long on Windows 
Bastard Programmer from Hell 
If you can't read my code, try converting it here[^]
|
|
|
|
|
This seems more like an ad disguised as a news article.
If your actions inspire others to dream more, learn more, do more and become more, you are a leader.-John Q. Adams
You must accept one of two basic premises: Either we are alone in the universe, or we are not alone in the universe. And either way, the implications are staggering.-Wernher von Braun
Only two things are infinite, the universe and human stupidity, and I'm not sure about the former.-Albert Einstein
|
|
|
|
|
They are right - but hardly anyone will follow them. Look around among your friends: a single computer for a whole family with one account (which is an admin account of course) - "Dad, why is the computer getting so slow" asked little Jonny after installing a dozen free apps. Or someone considering a password of less than 20 characters to short - of course, he's using an admin account only. It does not matter how easy it is to create "normal" user accounts - people won't do so.
|
|
|
|
|
In late 2013 we made available a prerelease NuGet package which allows running a managed web application directly on top of IIS without going through the normal ASP.NET (System.Web) request processing pipeline. All the fun of ASP.NET, less of the unfun
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
