News for hackers. Stuff that matters.
Re:Design/code reviews in a startup by edw519 on Thursday March 20, 2008 @01:21PM (#141763)
Good standards make code reviews trivial.

Well defined, zero tolerance, guidelines virtually eliminate the need for code reviews. Variable naming, standard routines, syntax, how to do iteration, branching, common functions, etc., etc. etc., should all be outlined clearly, agreed upon, and adhered to.

I prefer to look at it this way...

Code Review: No one gets out of this room alive until this is right (whatever that means).

Standards Compliance: Wanna play? Good. Follow the rules.

Oh, and please don't give me that age old argument about how standards limit creativity; if this small price for quality software is too limiting for you, then go play by yourself.

Re:Design/code reviews in a startup by twak on Thursday March 20, 2008 @03:18PM (#141856)
Code reviews should be much more to with medium to big ideas, not small details. Small details should be trivial to a good hacker.

so: "You've got that wired in backwards", "Tim built one of those over here", "When we implement foo, that'll bar" & "That's a security hole because ...." are good.

Re:Design/code reviews in a startup by nradov on Thursday March 20, 2008 @03:14PM (#141849)
We have coding standards. The developers are good about following them, and some coding standards problems can even by caught automatically by static analysis tools.

But coding standards don't prevent logic errors, or requirements misunderstandings. Even the best developers make those kinds of mistakes and we catch a significant fraction of them during code reviews.

The other major benefit of consistently doing code reviews is learning. The attendees often pick up useful new techniques. And you spread knowledge of the code base around the team, which reduces the organization's dependence on any one developer and also brings out opportunities for code reuse.

Re:Design/code reviews in a startup by gruseom on Thursday March 20, 2008 @02:45PM (#141832)
Good standards make code reviews trivial.

If that were so, then you could standardize the production of good code.

Re:Design/code reviews in a startup by dgabriel on Thursday March 20, 2008 @01:42PM (#141779)
I'm totally with you, but bear in mind that standards documents (especially within small start-up teams) tend to be built slowly over time, and reviewed and modified when necessary. Code reviews are great for establishing and tweaking standards when none exist.
Re:Design/code reviews in a startup by joecomotion on Thursday March 20, 2008 @08:48PM (#142082)
Yeah, variable naming standards are nice for consistency and I take it "common functions" is a euphemism for "don't cut and paste," but do you really want to dictate 'how to do iteration' in a way that's more limiting than the implementation language's idioms?
Re:Design/code reviews in a startup by kingkongrevenge on Thursday March 20, 2008 @02:19PM (#141808)
> Variable naming, standard routines, syntax, how to do iteration, branching, common functions

You're seriously wasting time if your code reviews could be done by a batch of regular expressions and a cron job, ala http://search.cpan.org/dist/Perl-Critic/lib/Perl/Critic.pm

I always thought the point was to keep things well factored: find hard to spot duplication or inconsistencies in disparate areas of the code, point out a library/api that could have done the task better, etc.