Take the 2-minute tour ×
Programmers Stack Exchange is a question and answer site for professional programmers interested in conceptual questions about software development. It's 100% free, no registration required.

This question already has an answer here:

We are a small team working on a web application using MVVM design pattern using technologies like .NET, Knockout and HTML. I am trying to come up with a code review checklist for this, so that my team can make use of it while coding as well as while reviewing. I tried to find in google, but no help there.

Can you guys suggest me what I have to look for code review checklist?

share|improve this question
add comment

marked as duplicate by gnat, MichaelT, GlenH7, Bart van Ingen Schenau, Kilian Foth May 23 '13 at 11:08

This question has been asked before and already has an answer. If those answers do not fully address your question, please ask a new question.

1 Answer

Instead of using a checklist I think what you're looking for is a coding standard, something that tells your team what you expect of their code and provide guidelines.

The coding standard could be something as simple as a pattern for variable naming and a convention for how the code should be laid out (for example parenthesis on a line of its own), or it could be something far more in-depth, it entirely depends on your requirements and your teams requirements. If the whole team sticks to this standard it will make the code a lot easier for anyone to pick up, and it will mean there's a guideline to review the code against.

Since the code review is something that the team is going to be doing it's worth getting the team involved in creating the coding standard. If the team is involved in they'll feel a lot happier about sticking to the standard that if they are handed down "rules from above" because it's something that the team has come up with.

Once you have this standard it means that the team can review the code against the standard and highlight any issues in the code.

share|improve this answer
add comment

Not the answer you're looking for? Browse other questions tagged or ask your own question.