This post originated from an RSS feed registered with .NET Buzz
by Richard Jonas.
Original Post: Code Review Checklist
Feed Title: Richard Jonas
Feed URL: http://feeds.feedburner.com/blogspot/ouNA
Feed Description: Richard Jonas's blog about .NET, web development and agile methodologies.
Here's a checklist for code reviews (in any language).
1) Would the logic be understandable by a developer of average ability proficient with other programming languages but not this one?
If it uses language features or obscure design patterns that are "clever" without good reason (e.g. something needs to have the maximum performance), then it won't be maintainable and someone will replace it...
Previous Topic
