13. Checklist
1. Make sure that there shouldn't be any project warnings.
2. It will be much better if Code Analysis is performed on a project (with all Microsoft
Rules enabled) and then remove the warnings.
3. All unused usings need to be removed. Code cleanup for unnecessary code is
always a good practice.
4. 'null' check needs to be performed wherever applicable to avoid the Null
Reference Exception at runtime.
5. Naming conventions to be followed always. Generally for variables/parameters,
follow Camel casing and for method names and class names, follow Pascal casing.
6. Make sure that you are aware of SOLID principles.
7. Code Reusability
8. Code Consistency
9. Code Readability
10. Disposing of Unmanaged Resources
14. Checklist
11. Proper implementation of Exception Handling
12. Making sure that methods have less number of lines of code.
13. Peer code reviews.
14. The code is unit testable
15. Logging used appropriately (proper logging level and details)
16. Avoid nested for/foreach loops
17. Use anonymous types if code is going to be used only once.
18. Try using LINQ queries and Lambda expressions to improve Readability.
19. Use access specifiers
20. Use interfaces wherever needed to maintain decoupling.
21. Mark a class as sealed or static or abstract as per its usage and your need.
15. Reasons Why Code Reviews
1. Double-Check if Requirements Are Fulfilled
2. Fewer Bugs in the Code
3. Improved Code Readability
4. Knowledge-Sharing
5. Style of Code Becomes Similar
6. Removes the Single-Point of Failure
7. Better Estimates and Better Planning
8. Improves Code Quality by Peer Pressure
9. The Cost of Defects Is Lower the Earlier They Are Detected
10.Mentoring Junior Developers
16. Some Popular Code Review Tools
1. Best Code Review Tools in the Market
2. Collaborator
3. Review Assistant
4. Codebrag
5. Gerrit
6. Codestriker
7. Rhodecode
8. Phabricator
9. Crucible
10.Veracode
11.Review Board