Click here to Skip to main content
15,117,095 members
Articles / Web Development / CSS
Article
Posted 16 Nov 2016

Stats

115.4K views
939 downloads
148 bookmarked

Code Review Checklist

Rate me:
Please Sign up or sign in to vote.
4.91/5 (96 votes)
24 Feb 2017CPOL4 min read
An Ideal Code Review Checklist that applies for most programming languages

Introduction

The purpose of this article is to propose an ideal and simple checklist that can be used for code review for most languages. Even though there are a lot of code review techniques available everywhere along with how to write good code and how to handle bias while reviewing, etc., they always miss the vital points while looking for the extras. The main idea of this article is to give straightforward and crystal clear review points for code reviewers to review efficiently with least time possible.

Purpose of Code Review

The ultimate purpose of code review is to investigate the code to find weak spots, faults, strengths and ways to optimize the code for better performance. It is mainly to deliver a bug-free (at least near perfect) application that meets the purpose (requirements) while meeting the industry standards.

The code review checklists are illustrated in two parts:

  • Code Review Checklist - Fundamental
  • Code Review Checklist - Comprehensive

Code Review Checklist - Fundamental

Fundamental code review

  1. Objective based [Purposeful]

    The code achieves its purpose. In simple terms, it does what it is supposed to.

  2. Unbreakable [Validated]

    Validations are used wherever necessary. The code never breaks under any circumstances. Especially under invalid inputs that come from the user end. Regardless of it being a negative, over-sized, invalid format, etc., every input passed should be processed, sanitized before taking it further. Every object is checked for its actual data existence before accessing its properties.

  3. Responses are handled [Error handling and Data formatting]

    Not just the error messages, every response that is returned by the server must be properly handled. It should have necessary headers, response messages, error codes and any other necessary details attached with it in required format. All possible scenarios are tested to avoid deadlocks, timeouts, etc.

  4. Follows architecture [No design deviation]

    Verify that the approved architecture/design is followed throughout the application (If there is none, consider putting it in place). If there are any design changes required, ensure that these are documented, baselined and approved before implementing them in the existing code.

  5. Unit tested [Reliable]

    Every core method has a unit test which passes.

  6. Reusable [No repetition]

    All methods serve a limited and clear purpose (follows DRY principle). Functions are reused wherever applicable and written in such a way that they can be re-used in the future implementations. There is no duplication of code. Logics make use of general functions without ambiguity.

  7. Performance oriented [Speedy response & Scalability]

    The landing of the application is swift. There are no long delays between the requests and responses. Raw string concatenations are avoided and proper methods such as StringBuilder are used. The code is scalable and able to handle a large amount of data and upcoming features.

  8. Secure [Safe code]

    The code is secure in terms of authentications (with encryption), injections, roles, unauthorized access, directory browsing, SQL injection, cross-side scripting, etc. It follows the OWASP 10 security principles.

Code Review Checklist - Comprehensive

Advanced code review

  1. Manageable [Crisp and Formatted]

    The code is readable, commented and easy to manage. It is friendly formatted and easy to read/understand. Methods are not too big to manage and they don't exceed readable size.

  2. Meets coding conventions and standards [Standardized approach]

    The code follows the coding conventions, standards and is consistent with the existing application code. There are no commented code and hard coded values.

  3. Detach connections after usage [Memory handling]

    Resources that are not automatically released after usage are freed. Connections, ports are closed properly.

  4. Has configurable logging[Traceability]

    Log every transaction or the ones that require logging. They are stored in a repository (as a file) as well as in the database (as text). Logging in different stages for different purposes can be enabled/disabled in the configuration file (like web.config)

  5. Code coverage is more than 95% [Efficiency]

    Code coverage is as important as the unit test cases passing. 95% of the code is covered (which means 95% code is actually tested via unit test cases).

  6. On-demand resource delivery[Fast]

    Resources are fetched and delivered only on demand. Necessary options are available for dealing with huge data such as paginations, etc.

  7. No warnings / console logs [Data safety]

    No compiler warnings arise while running the application. Logs that are used while developing are cleared and none of the application information (especially the sensitive ones) are written in the browser console.

  8. Legal usage of third-party tools/libraries[Licencing]

    External libraries are used only if proven necessary for the application. If there are third-party tools or libraries used, then the licenses and legal usages are verified and complaint.

References

Conclusion

The above checklist is vital to have a high-quality code that meets the requirements and performs at its best while being secure, scalable and swift. There will always be more points to add to this list. It is recommended to go for every good to have points while reviewing after making sure the primary checklists are checked.

License

This article, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Share

About the Author

Ebenezar John Paul
Software Developer Gislen Software
India India
Ebenezar John Paul is a Software programmer & Blogger. He loves to code and rethinks the conventional way of the world.

He believes that pushing limits is the only way to find out what we are really capable of, which is how the human race can redefine impossible and perfection.

And he prefers ideas over rules, results over excuses, people over things, satisfaction over completion and passion over any other.

Comments and Discussions

 
AnswerRe: Very good. Pin
Ebenezar John Paul24-Nov-16 4:50
professionalEbenezar John Paul24-Nov-16 4:50 
GeneralMy vote of 5 Pin
Marc Clifton22-Nov-16 3:02
mvaMarc Clifton22-Nov-16 3:02 
AnswerRe: My vote of 5 Pin
Ebenezar John Paul22-Nov-16 7:26
professionalEbenezar John Paul22-Nov-16 7:26 
GeneralRe: My vote of 5 Pin
Marc Clifton23-Nov-16 4:20
mvaMarc Clifton23-Nov-16 4:20 
GeneralRe: My vote of 5 Pin
Ebenezar John Paul24-Nov-16 4:47
professionalEbenezar John Paul24-Nov-16 4:47 
GeneralMy vote of 5 Pin
jicking20-Nov-16 22:38
Memberjicking20-Nov-16 22:38 
AnswerRe: My vote of 5 Pin
Ebenezar John Paul21-Nov-16 16:32
professionalEbenezar John Paul21-Nov-16 16:32 
GeneralConcise Pin
sosplombierorleans20-Nov-16 10:50
Membersosplombierorleans20-Nov-16 10:50 
Thanks for that
AnswerRe: Concise Pin
Ebenezar John Paul20-Nov-16 21:59
professionalEbenezar John Paul20-Nov-16 21:59 
GeneralMy vote of 5 Pin
jrobb22918-Nov-16 8:10
Memberjrobb22918-Nov-16 8:10 
GeneralRe: My vote of 5 Pin
Ebenezar John Paul20-Nov-16 22:03
professionalEbenezar John Paul20-Nov-16 22:03 
GeneralMy vote of 5 Pin
Gaston Verelst18-Nov-16 3:52
MemberGaston Verelst18-Nov-16 3:52 
GeneralRe: My vote of 5 Pin
Ebenezar John Paul20-Nov-16 22:04
professionalEbenezar John Paul20-Nov-16 22:04 
GeneralMy vote of 5 Pin
Anurag Gandhi17-Nov-16 1:20
professionalAnurag Gandhi17-Nov-16 1:20 
GeneralRe: My vote of 5 Pin
Ebenezar John Paul20-Nov-16 22:03
professionalEbenezar John Paul20-Nov-16 22:03 
GeneralMy vote of 5 Pin
Maríanna Ólafsdóttir16-Nov-16 21:11
MemberMaríanna Ólafsdóttir16-Nov-16 21:11 
GeneralRe: My vote of 5 Pin
Ebenezar John Paul20-Nov-16 22:06
professionalEbenezar John Paul20-Nov-16 22:06 
QuestionMy vote Pin
Vara3216-Nov-16 21:03
MemberVara3216-Nov-16 21:03 
AnswerRe: My vote Pin
Ebenezar John Paul20-Nov-16 22:01
professionalEbenezar John Paul20-Nov-16 22:01 
PraiseMy vote of 5 Pin
Code Ninja R16-Nov-16 18:29
MemberCode Ninja R16-Nov-16 18:29 
AnswerRe: My vote of 5 Pin
Ebenezar John Paul20-Nov-16 22:05
professionalEbenezar John Paul20-Nov-16 22:05 

General General    News News    Suggestion Suggestion    Question Question    Bug Bug    Answer Answer    Joke Joke    Praise Praise    Rant Rant    Admin Admin   

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.