Traditional approaches put a lot of emphasis on Code Inspection or Reviews for finding defects. In human terms, that is like an editing process. The problem with code inspection is that, even under the best circumstances, it finds very few defects, and the ones that it does find are shallow. The challenge is that our brains are just not designed to read through hundreds of lines of code in a single sitting. When we do this activity, we tend to notice small opportunities for improvements (e.g. a method or variable are poorly named) and we might spot a way of simplifying a few lines of code. But what we fail to do is ask bigger questions about the design, flow, etc. Most things that a human code review finds are better spotted by static analysis tools.
- The case against code reviews – Jo Crossick
- Code review vs. Pair programming
- Evolving Code Reviews Into Pair Programming
- Pair Programming > Code Reviews – Theodore Nguyen-Cao
Mark Levison has been helping Scrum teams and organizations with Agile, Scrum and Kanban style approaches since 2001. From certified scrum master training to custom Agile courses, he has helped well over 8,000 individuals, earning him respect and top rated reviews as one of the pioneers within the industry, as well as a raft of certifications from the ScrumAlliance. Mark has been a speaker at various Agile Conferences for more than 20 years, and is a published Scrum author with eBooks as well as articles on InfoQ.com, ScrumAlliance.org an AgileAlliance.org.
*Thank you for visiting the World's Largest Opinionated Agile Reference Library. This content is created and the links are curated through the lens of Agile Pain Relief Consulting's view of what is effective in the practice of Scrum and Agile. We don't accept submissions and emails to that effect are marked as spam. Book listings may use affiliate links that could result in a small commission received by us if you purchase, but they do not affect the price at all. From experience, this won't amount to anything more than a cup of coffee in a year.« Back to Glossary Index