Thursday, 5 January 2012

Code Reviewing

Who does it?
  • The tech expert within the team or
  • All team members?
When is it done?
  • Every time a code is checked in by a team member or
  • Only once or twice a week at a certain time
  • Check whether the code design could be improved or is good enough?
    • Readability
    • Maintainability
    • Low coupling
    • Reusability
    • ...
  • Check whether the code performance could be improved?
  • Check for any Code Smells
    • Code duplication
    • Long methods
    • Large classes
    • ...
How is it done?
  • Manually or
  • By a tool depending on your source control e.g. TFS or Subversion

No comments: