An Ounce of Goat or a Pound of Hero: Software Reviews and Inspections

[article]

When do your testers join the software development process? If you're like most teams, testing doesn't start until late in the development cycle. Once testing is engaged, you work around the clock finding the important bugs, fixing those you can fix, and trying to get the code ready to ship. In all, your testers and developers will put in several extremely late nights and probably a few weekends to get the code out the door.

Last-Minute Heroes
Testing that starts late in the software development cycle gives many bugs that were originally small and simple a chance to become deeply entangled with other bugs in the system. These entrenched bugs are often difficult to find and nearly impossible to fix well.

On the bright side, everyone who survives the dreaded days of debugging will likely be proclaimed a hero. But I wonder if heroism in these situations is as necessary as we are often led to believe.

Let's step back for a moment and look at similar heroics in a totally different context. Smokejumpers are firefighters who parachute into forest fires armed with shovels, picks, and little else. This matches up pretty well with my idea of heroism. Their work is important, exhausting, and dangerous. Fatalities occur on a regular basis.

But an ounce of prevention may really be worth a pound of cure--or hundreds of acres of forest. Some communities have begun using goats to deal with wildfires. Instead of waiting until a fire starts, these communities hire goats to munch away on the underbrush. This starves a potential fire of the fuel that would allow the flames to spread. This is not glamorous work, but the goats effectively clear many acres in a short time.

Mike McMillan/Spotfire Images

Goats vs. Smokejumpers

  1. Goats work cheap. 650 goats probably cost as much as one firefighter.
  2. Goats are not nearly as exciting as smokejumpers. Watching goats munch on underbrush is only marginally more interesting than watching the underbrush grow.
  3. Using goats requires planning. Goats are of little use once the fire has started. (Unless you plan to feed them to the firefighters.) To be effective, they must be used before the emergency hits.
  4. Goats are not a complete solution. Fires will still happen and you will still need firefighters-hopefully not as many or for as long.
  5. Measuring the success of goats is tricky. You can't talk about fires extinguished; you have to talk about fires prevented. Fortunately, if you have some idea of how many fires would usually flare up, you can get an estimate of how helpful the goats have been.

Now, let's return to your team's software development efforts. On your last project, did you get pulled into firefighting mode because your testing found serious bugs only late in the development cycle? Did you repeatedly send in your developers to contain and debug the problems? I'll bet your developers worked mainly in reactive mode, moving from one hot spot to the next since they had no time to create long-lasting solutions. In other words, your people were heroes heading straight into burnout.

But what if your team had taken a more goat-like approach, stopping small, isolated bugs before they could become large, tangled bugs? What if your developers and testers used cheap, simple methods during the development phase to lessen the presence of bugs later in the process?

 
Reused with permission: San Diego Union-Tribune

The truth is we have had such tools and methods around for a long time. We just haven't always used them as well or as often as we could have. For instance, to name just a few:

  1. Code reviews and inspections. Code review is probably not on anyone's list of exciting ways to spend a few hours, but its reported 70-80% find rate certainly deserves attention.
  2. Automated Inspections. For the lazier among us, there are tools that will automatically inspect code for suspicious constructs.
  3. Unit testing. With the emergence of extreme programming, unit testing is now in fashion.

Early Detection vs. Late Night Heroism

  1. Early detection tools cost much less than finding showstopper bugs at the last minute.
  2. Early detection methods find bugs in a simple and boring way, which is good. Software development currently has too many nail-biting moments and death marches.
  3. Early detection tools require planning. Once you are in reactive mode, it's too late.
  4. These tools do not find all bugs. You still need testers and developers to fix bugs-hopefully not as many since the number of bugs will be smaller.
  5. Measuring the impact of these tools can be tricky since their real value is in preventing worse bugs down the line.

Keeping It All Under Control
Being a hero is great, but only if your heroism doesn't involve a situation you could have avoided in the first place. Good engineering doesn't consist of random acts of heroism. Good engineering should celebrate a subtler form of accomplishment.

Further Reading

  1. For quick background on the effectiveness of code reviews, read "Inspections - Evolution and History: 1972-2001" by Michael Fagan
  2. JUnit and www.NUnit.org provide good starting points for finding out about the latest craze in unit testing.
  3. For astounding photos of smokejumpers in action, visit Mike McMillan's website, www.spotfireimages.com.
  4. For un-astounding photos of goats in action, visit www.signonsandiego.com/uniontrib/20040416/news_1n16goats.html
  5. And, finally, "Nobody Ever Gets Credit for Fixing Problems that Never Happened," at web.mit.edu/nelsonr/www/Repenning=Sterman_CMR_su01_.pdf is a good read about making sure that prevention techniques get recognized, rewarded, and sustained.

User Comments

44 comments
Michael Cummings's picture

Great article. It was well-received by both the test and dev teams here. One comment I would make is that we have to try to make sure that the heros are not the ones starting the fires - which seems to be the case all to often. MC

October 15, 2004 - 11:23am
Michael Cummings's picture

Great article. It was well-received by both the test and dev teams here. One comment I would make is that we have to try to make sure that the heros are not the ones starting the fires - which seems to be the case all to often. MC

October 15, 2004 - 11:23am
Michael Cummings's picture

Great article. It was well-received by both the test and dev teams here. One comment I would make is that we have to try to make sure that the heros are not the ones starting the fires - which seems to be the case all to often. MC

October 15, 2004 - 11:23am
Michael Cummings's picture

Great article. It was well-received by both the test and dev teams here. One comment I would make is that we have to try to make sure that the heros are not the ones starting the fires - which seems to be the case all to often. MC

October 15, 2004 - 11:23am
David Ramger's picture

I heard on ESPN recently a comment from one of the sportscasters who was discussing baseball's weekly Top 10 Plays of the Week. He pointed out that a large number of the exciting diving catches that were being highlighted on the show were a result of a very good athlete making a great play only because he was out of position. There seemed to be a lot of younger players who made the highlight reel simply because they were good athletes who *had* to make a diving reach to catch a ball that the older, more experienced players made easily because they were in the right spot in the first place, but made the same catch look routine. <br/><br/>The same thing applies to programmers. The younger, less experienced coders tend to expend a lot of energy fixing problems that older, more experienced coders would never have made in the first place. And yet it seems to be the flashy heroes that are rewarded for fixing the problem rather than the one who prevented it in the first place.

October 11, 2004 - 2:44pm
David Ramger's picture

I heard on ESPN recently a comment from one of the sportscasters who was discussing baseball's weekly Top 10 Plays of the Week. He pointed out that a large number of the exciting diving catches that were being highlighted on the show were a result of a very good athlete making a great play only because he was out of position. There seemed to be a lot of younger players who made the highlight reel simply because they were good athletes who *had* to make a diving reach to catch a ball that the older, more experienced players made easily because they were in the right spot in the first place, but made the same catch look routine. <br/><br/>The same thing applies to programmers. The younger, less experienced coders tend to expend a lot of energy fixing problems that older, more experienced coders would never have made in the first place. And yet it seems to be the flashy heroes that are rewarded for fixing the problem rather than the one who prevented it in the first place.

October 11, 2004 - 2:44pm
David Ramger's picture

I heard on ESPN recently a comment from one of the sportscasters who was discussing baseball's weekly Top 10 Plays of the Week. He pointed out that a large number of the exciting diving catches that were being highlighted on the show were a result of a very good athlete making a great play only because he was out of position. There seemed to be a lot of younger players who made the highlight reel simply because they were good athletes who *had* to make a diving reach to catch a ball that the older, more experienced players made easily because they were in the right spot in the first place, but made the same catch look routine. <br/><br/>The same thing applies to programmers. The younger, less experienced coders tend to expend a lot of energy fixing problems that older, more experienced coders would never have made in the first place. And yet it seems to be the flashy heroes that are rewarded for fixing the problem rather than the one who prevented it in the first place.

October 11, 2004 - 2:44pm
David Ramger's picture

I heard on ESPN recently a comment from one of the sportscasters who was discussing baseball's weekly Top 10 Plays of the Week. He pointed out that a large number of the exciting diving catches that were being highlighted on the show were a result of a very good athlete making a great play only because he was out of position. There seemed to be a lot of younger players who made the highlight reel simply because they were good athletes who *had* to make a diving reach to catch a ball that the older, more experienced players made easily because they were in the right spot in the first place, but made the same catch look routine. <br/><br/>The same thing applies to programmers. The younger, less experienced coders tend to expend a lot of energy fixing problems that older, more experienced coders would never have made in the first place. And yet it seems to be the flashy heroes that are rewarded for fixing the problem rather than the one who prevented it in the first place.

October 11, 2004 - 2:44pm
Jim Bernstein's picture

Harry<br/><br/>One of the most truly entertaining articles on Sticky Minds! The problem you describe plagues most industries however most companies are so busy firefighting they can't go to the market to buy the goats. I try to convince people each day that finding a problem in development costs much less than finding it in pre (or worse) post production.<br/><br/>Jim<br/>Solstice Software

October 11, 2004 - 3:35pm
Jim Bernstein's picture

Harry<br/><br/>One of the most truly entertaining articles on Sticky Minds! The problem you describe plagues most industries however most companies are so busy firefighting they can't go to the market to buy the goats. I try to convince people each day that finding a problem in development costs much less than finding it in pre (or worse) post production.<br/><br/>Jim<br/>Solstice Software

October 11, 2004 - 3:35pm

Pages

About the author

StickyMinds is a TechWell community.

Through conferences, training, consulting, and online resources, TechWell helps you develop and deliver great software every day.