× The internal search function is temporarily non-functional. The current search engine is no longer viable and we are researching alternatives.
As a stop gap measure, we are using Google's custom search engine service.
If you know of an easy to use, open source, search engine ... please contact support@midrange.com.



We do do code review (sometimes.) We're not as formalized as many
companies but when we're working on the harder parts of a system it's
nice to have a second set of eyes. We've even gone as far as doing
quasi-pair-programming for some of the deeper parts. The code where
we've done this is code that is complex, multi-threaded, critical
performance, heavy load type code. For your average subfile I don't see
the point.

As for how the programmers take it, they're actually the ones that
usually ask for it. There should be a shop standard that they're
following, so there's only so much "style" involved. And a code review
isn't to say that what you did is wrong (usually), but rather to add a
set(s) of eyeballs to say "did you think about..." and "what happens
if...."

As the open-source community is fond of saying, "with enough eyeballs,
all bugs are shallow."

-Walden


As an Amazon Associate we earn from qualifying purchases.

This thread ...

Replies:

Follow On AppleNews
Return to Archive home page | Return to MIDRANGE.COM home page

This mailing list archive is Copyright 1997-2024 by midrange.com and David Gibbs as a compilation work. Use of the archive is restricted to research of a business or technical nature. Any other uses are prohibited. Full details are available on our policy page. If you have questions about this, please contact [javascript protected email address].

Operating expenses for this site are earned using the Amazon Associate program and Google Adsense.