Best Kept Secrets of Peer Code Review: Modern Approach. Practical Advice. (Modern Approach. Practical Advice.) [Paperback]

Jason Cohen (Author)

Book Description

Publication Date: January 1, 2006

Ten practical Essays from industry experts giving specific techniques for effective peer code review.

Product Details

• Paperback: 164 pages
• Publisher: Smart Bear Inc. (January 1, 2006)
• Language: English
• ISBN-10: 1599160676
• ISBN-13: 978-1599160672
• Product Dimensions: 8.1 x 5.3 x 0.5 inches
• Shipping Weight: 8 ounces
• Average Customer Review: 3.8 out of 5 stars  See all reviews (5 customer reviews)
• Amazon Best Sellers Rank: #1,059,534 in Books (See Top 100 in Books)

Customer Reviews

Most Helpful Customer Reviews

7 of 7 people found the following review helpful:

4.0 out of 5 stars Keeping a good idea good, April 11, 2007

By 

wiredweird "wiredweird" (Earth, or somewhere nearby) - See all my reviews
(HALL OF FAME REVIEWER)    (TOP 500 REVIEWER)   

This review is from: Best Kept Secrets of Peer Code Review: Modern Approach. Practical Advice. (Modern Approach. Practical Advice.) (Paperback)

First, let me say that I have decidedly mixed experience of code inspections. At their best, I've seen them bring many diverse skills to bear on subtle problems, creating a product better than any one of the very capable developers could have built on their own. At worst, inspections range downwards from useless, not just worthless in themselves but divisive and demoralizing to the development group. I want to believe the process can be used safely and effectively, but I have some hard-won doubts. The fact that the publisher sells a technical product for managing the review process added other doubts.

I'm happy to say that my doubts are addressed - not banished utterly, but faced head on. Despite a bit of jargoneering (is "trial" really a verb?), the authors present updated techniques that appear to reduce some of the problems working against effective inspections. And no, the techniques don't demand that you buy their product first. The techniques dovetail well with modern ideas, including the Law of Demeter (p.46, on module coupling), SEI's PSP, and the rich IDEs common on developers' desktops today. Finally, the last chapter - and only the last chapter - addresses the company's product.

Despite the book's commercial origin, it has lots of great information beyond the writers' product. In fact, it reminds me of "Planecraft," written over 70 years ago. That book was written as a sales tool for the greater corporate glory of a company selling woodworking planes, but has enough worthwhile technical content that it was reprinted a few years back. Maybe "Best Kept Secrets" won't last 70 years, but I was struck by the analogy.

The book's discussion discussion is wide-ranging, readable, and helpful. A few editing glitches put potholes in the reading path, but only a few. On the whole, I came away somewhat more willing to throw myself back into the fray of inspections, as long as they're run carefully, in accordance with many of the ideas shown here.

//wiredweird, reviewing a copy given out as conference swag

5 of 5 people found the following review helpful:

4.0 out of 5 stars Good discussion-starter about code review, May 12, 2008

By 

T. Harris "http://talkaboutquality.wordpress.com" - See all my reviews
(REAL NAME)   

This review is from: Best Kept Secrets of Peer Code Review: Modern Approach. Practical Advice. (Modern Approach. Practical Advice.) (Paperback)

This is a book with an agenda: selling Smart Bear Software's multi-user peer code review application. So you have to keep in mind as you read: how much of what's written makes sense on its own, and how much is carefully worded to lead you to the desired conclusion--that Smart Bear are the experts and their product is the greatest thing since sliced bread? That said, the book, or rather, collection of essays, is a good discussion-starter about code review. The chapter on Five Types of Code Review is a bit too assertive for me. I thought the chapter on Personal Software Process and code review was excellent. And the selling point of the book is the data they've gathered. Everything they could find on the Internet about actual industrial code review data (not much) plus their own real-life study at one of their customers--CISCO. So if you're ready to do code review, then just do it, and then read this book to compare notes. If you're not ready yet, this book will certainly make you wonder why not.

1 of 1 people found the following review helpful:

3.0 out of 5 stars Contains some useful information, April 18, 2010

By 

davez (LA, CA, United States) - See all my reviews

This review is from: Best Kept Secrets of Peer Code Review: Modern Approach. Practical Advice. (Modern Approach. Practical Advice.) (Paperback)

For people who consider doing code review, this book provides some useful information. It brings up many issues involved with code review, technical and sociological. However, without doing your own research and experimentation, it is hard to say if some of authors' conclusions are valid.