Code Reviews.
-
How do you guys do them?
cheers, Chris Maunder The Code Project | Co-founder Microsoft C++ MVP
-
How do you guys do them?
cheers, Chris Maunder The Code Project | Co-founder Microsoft C++ MVP
Like a scene out of Enter The Dragon.
I was brought up to respect my elders. I don't respect many people nowadays.
CodeStash - Online Snippet Management | My blog | MoXAML PowerToys | Mole 2010 - debugging made easier -
How do you guys do them?
cheers, Chris Maunder The Code Project | Co-founder Microsoft C++ MVP
If you're even half good, you place a hand on the machine and feel the code...
Reality is an illusion caused by a lack of alcohol
-
How do you guys do them?
cheers, Chris Maunder The Code Project | Co-founder Microsoft C++ MVP
Kicking and screaming... ;) We have designated peer reviews and follow a review process.
It was broke, so I fixed it.
-
How do you guys do them?
cheers, Chris Maunder The Code Project | Co-founder Microsoft C++ MVP
There aren't very many of us, so sometimes we are on a project by ourselves. If that is the case, then we just pick someone who is good in that "area", or has available time. Once that person starts reviewing a project's code, we like to keep them as the reviewer. After that, it's just sit down and go over the code. We like to do it in person, though sometimes we do it through git without the other person. We explain what was changed, and show the features/stories. We demo the changes, and then go through the code. The boss isn't involved, though he does review the change sets for anything crazy (or so I've noticed). We have full control over the review process. If the reviewer has issue with the code, we try to reach a compromise or a good reason to leave it alone (or change it). If nothing can be resolved we may poll the team, or may move on and come back to it later. It always seems to go really well. Some disagreements have to be shelved , but we usually come to a compromise fairly easily. We've all read the same type of literature (code complete, don't make me think, etc) that are office standards.
If it moves, compile it
-
How do you guys do them?
cheers, Chris Maunder The Code Project | Co-founder Microsoft C++ MVP
We do more informal/peer reviews now instead of hours in a conference room with a huge team. Typically the software leads for the team do the review with other senior devs. We have even done some over Lync remotely with screen sharing and a Lync call.
Steve Maier
-
How do you guys do them?
cheers, Chris Maunder The Code Project | Co-founder Microsoft C++ MVP
Yes. They are mandatory for each unit of work here. The old "system" was just to pick on someone to review and get their comments. We were using a tool called Crucible[^], the UI was a bit unintuitive, but better than the alternatives + it integrates with SVN nicely, to you just open it up and review. Currently we are back to the old system of just finding a random punter to do it, apparently we are looking for a replacement to crucible (we were on a free trial). Before I came here I was against reviews, now I've moved round a bit to a more positive anything more than a sanity check is a waste of time as I'm less likely to wrap my head around the problem a fellow dev has spent time and effort working out. Still getting used to it though, so mod through inexperience on my part.
Sort of a cross between Lawrence of Arabia and Dilbert.[^]
-Or-
A Dead ringer for Kate Winslett[^] -
How do you guys do them?
cheers, Chris Maunder The Code Project | Co-founder Microsoft C++ MVP
As a less frivolous answer, our code reviews are done on an ad-hoc basis, about once a week, and more formally once a month. All code is fair game - my code is as up for review as the newest member of staff. We have a review checklist but, to be honest, we tend to pay more attention to results out of our CI process (where we run FxCop and StyleCop - even though I don't like them), plus NDepend and NCover.
I was brought up to respect my elders. I don't respect many people nowadays.
CodeStash - Online Snippet Management | My blog | MoXAML PowerToys | Mole 2010 - debugging made easier -
How do you guys do them?
cheers, Chris Maunder The Code Project | Co-founder Microsoft C++ MVP
Boss: I just let (Bob) go; we'd better take a look at what he produced, if anything.
-
How do you guys do them?
cheers, Chris Maunder The Code Project | Co-founder Microsoft C++ MVP
Whenever necessary, at least every stable release (Version number X.0.0.0). Rules: Program must work, object orientated programmed and code conventions must have be followed.
cheers Marco Bertschi
Software Developer & Founder SMGT Web-Portal CP Profile | My Articles | Twitter | Facebook | SMGT Web-Portal
-
Yes. They are mandatory for each unit of work here. The old "system" was just to pick on someone to review and get their comments. We were using a tool called Crucible[^], the UI was a bit unintuitive, but better than the alternatives + it integrates with SVN nicely, to you just open it up and review. Currently we are back to the old system of just finding a random punter to do it, apparently we are looking for a replacement to crucible (we were on a free trial). Before I came here I was against reviews, now I've moved round a bit to a more positive anything more than a sanity check is a waste of time as I'm less likely to wrap my head around the problem a fellow dev has spent time and effort working out. Still getting used to it though, so mod through inexperience on my part.
Sort of a cross between Lawrence of Arabia and Dilbert.[^]
-Or-
A Dead ringer for Kate Winslett[^] -
How do you guys do them?
cheers, Chris Maunder The Code Project | Co-founder Microsoft C++ MVP
Three levels: - pass-through: people have to go over the code and send results per mail, everybody on his own. - f2f: everybody sits in a room, odd number of people to get majority if questions, author presents his changes and everybody ... well, reviews. - inspection: (for critical code) just as f2f but with someone conducting the review who is not from the same department as the people authoring it. Findings documented in review minutes, on the fly corrections not allowed.
~RaGE();
I think words like 'destiny' are a way of trying to find order where none exists. - Christian Graus Do not feed the troll ! - Common proverb
-
How do you guys do them?
cheers, Chris Maunder The Code Project | Co-founder Microsoft C++ MVP
I look at it and go "Yeah! That's good code!" One person reviews don't generally catch much... ;)
If you get an email telling you that you can catch Swine Flu from tinned pork then just delete it. It's Spam.
-
Keith Barrow wrote:
Yes
- How do you answer an open question ? - Yes. :rolleyes:
~RaGE();
I think words like 'destiny' are a way of trying to find order where none exists. - Christian Graus Do not feed the troll ! - Common proverb
Manifesto: stop the posting of unhelpful comments pointing out that someone has "got something wrong" where the "something wrong" part is relatively minor, thus causing distraction. Such examples examples include spelling/syntactical/grammatical mistakes where is it pretty easy to work the originator's intention. Why "unhelpful comment"? At best the comment make the poster feel superior to someone else whilst denigrating or undermining the original poster especially when the originator is trying to add something to the debate, at worst it diverts the discussion away from the topic. Some people seem to mistake the Internet for an academic paper. -------------------- If you agree with the sentiment, please feel free to copy and paste this manifesto into your replies, including this comment -------------------- In this case, I misread CM's question as "Do you Guys do them"
Sort of a cross between Lawrence of Arabia and Dilbert.[^]
-Or-
A Dead ringer for Kate Winslett[^] -
How do you guys do them?
cheers, Chris Maunder The Code Project | Co-founder Microsoft C++ MVP
- Formalize them. If they are not formalized, eventually the review process will fade away 2) During the review focus on Code, Logic, and Design; ignore formatting, style, unless such style introduces an error. I could go on for months on this subject but, let's just say every code review I have participated in that was not run by me, focused on Style, Formatting and Coding Standards without a single minute spent on actual errors in code; which do you think is more important, honestly? Style formatting and Standards is a holy war with no winner; logic, functionality, approach and requirements all have the ability to be independently assessed and decrease code entropy. With that said, don't allow some college kid with no sense of style to randomly assign variable names and mix case depending on the mood, consistency per developer is more important than consistency on a team. 3) Don't focus on the easy stuff, if you do it becomes about nits. 4) If you find a teaching moment, use it. Bring in the team. Last year I found a code section that crashed production because of the as operator instead of casting. They had to bring me in to find the bug. 5) Allow yourself to be wrong.
Need custom software developed? I do custom programming based primarily on MS tools with an emphasis on C# development and consulting. "And they, since they Were not the one dead, turned to their affairs" -- Robert Frost "All users always want Excel" --Ennis Lynch
-
I look at it and go "Yeah! That's good code!" One person reviews don't generally catch much... ;)
If you get an email telling you that you can catch Swine Flu from tinned pork then just delete it. It's Spam.
-
Manifesto: stop the posting of unhelpful comments pointing out that someone has "got something wrong" where the "something wrong" part is relatively minor, thus causing distraction. Such examples examples include spelling/syntactical/grammatical mistakes where is it pretty easy to work the originator's intention. Why "unhelpful comment"? At best the comment make the poster feel superior to someone else whilst denigrating or undermining the original poster especially when the originator is trying to add something to the debate, at worst it diverts the discussion away from the topic. Some people seem to mistake the Internet for an academic paper. -------------------- If you agree with the sentiment, please feel free to copy and paste this manifesto into your replies, including this comment -------------------- In this case, I misread CM's question as "Do you Guys do them"
Sort of a cross between Lawrence of Arabia and Dilbert.[^]
-Or-
A Dead ringer for Kate Winslett[^]I agree, except where the error is Ironic, Amusing or of such vast incomprehensibleness as to render the reader speechless and/or catatonic.
--------------------------------- I will never again mention that I was the poster of the One Millionth Lounge Post, nor that it was complete drivel. Dalek Dave CCC Link[^]
-
How do you guys do them?
cheers, Chris Maunder The Code Project | Co-founder Microsoft C++ MVP
Just at this moment I'm managing a new project: We want to define a new QS-Process and for Code Review we want to use a tool. We are testing Jenkins-Server now.
------------------------------ Author of Primary ROleplaying SysTem How do I take my coffee? Black as midnight on a moonless night. War doesn't determine who's right. War determines who's left.
-
Manifesto: stop the posting of unhelpful comments pointing out that someone has "got something wrong" where the "something wrong" part is relatively minor, thus causing distraction. Such examples examples include spelling/syntactical/grammatical mistakes where is it pretty easy to work the originator's intention. Why "unhelpful comment"? At best the comment make the poster feel superior to someone else whilst denigrating or undermining the original poster especially when the originator is trying to add something to the debate, at worst it diverts the discussion away from the topic. Some people seem to mistake the Internet for an academic paper. -------------------- If you agree with the sentiment, please feel free to copy and paste this manifesto into your replies, including this comment -------------------- In this case, I misread CM's question as "Do you Guys do them"
Sort of a cross between Lawrence of Arabia and Dilbert.[^]
-Or-
A Dead ringer for Kate Winslett[^]Historical background: (which I should have explained right away) I answer all my wife's open question with "yes". And find it funny. Sorry for making insider jokes with myself. :) OTOH, that "Yes" sitting alone, nicely separated from the rest of the text, was yielding for a comment. As for the manifesto, you are right, I now feel superior to you and denigrate you, especially since you brought something valuable to the debate. My hobby is to make the Internet a mistake free zone ( but for the ones I am making myself). ;P
~RaGE();
I think words like 'destiny' are a way of trying to find order where none exists. - Christian Graus Do not feed the troll ! - Common proverb
-
I agree, except where the error is Ironic, Amusing or of such vast incomprehensibleness as to render the reader speechless and/or catatonic.
--------------------------------- I will never again mention that I was the poster of the One Millionth Lounge Post, nor that it was complete drivel. Dalek Dave CCC Link[^]
