Code Review Checklist

1) Clarity

Is the code clear and easy to understand?
Did the programmer unnecessarily obfuscate any part of it?
Can the code be refactored to make it clearer?

 

2) Maintainability

Will other programmers be able to maintain this code?
Is it well commented and documented properly?

 

3) Accuracy

Does the code accomplish what it is meant to do?
If an algorithm is being implemented, is it implemented correctly?

4) Readability and Robustness

Is the code fault-tolerant? Is the code error-tolerant?
Will it handle abnormal conditions or malformed input?
Does it fail gracefully if it encounters an unexpended condition?

5) Security

Is the code vulnerable to unauthorized access, malicious use, or modification?

6) Scalability

Could the code be a bottleneck that prevents the system from growing to accommodate increase load, data, users, or input?

7) Reusability

Could this code be reused in other applications?
Can it be made more general?

 

8) Efficiency

Does the code make efficient use if memory, CPU cycles, bandwidth, or other system resources?
Can it be optimized?

 

MANY SAMPLE CheckLists out there ---(system software tools/inhouse) --> some detailed oriented some broader in scope