Code Review Check List

Code Review Check List ,

I can categorize the code Review Check list in Java and DB side

Java Side :

1. Comments and Format should be in all the places
2. Single Responsibility Function level and class level
3. Operation and Class should small 
4. Avoid the Duplication 
5. Acceptability and modifier should be in all the places
6. Release Resource should be mandatory 
7. Try to use static block for common stuff like create the current date and loading the property
8. Try to minimize the Synchronized statement/block 
9. Avoid to use the printstacktrace and System .out. in the program 
10. Avoid to use constant inside the loop 
11. Avoid to create the unneccessary object
12. Use Checked Exception in recoverable condition and Runtime exception in Program/logic errors
13. Dont ignore the exception
14. Dont return null instead return empty object 
15. Use Enum instead of Constant
16. Unit Test must be exists in all the classes /method level
17. Try to use composition over inheritance 
18. Try to implement SOLID Principle 
19. Try to use DP as much as possible 
20. Avoid the Batch processing logic in the program 
21. use logger for capturing the log details 
22. Property should be externalize instead of inside the war/ear.
23. Validate the input before perform any manipulation 
24. All the request/response(transfering remotly) should be serilizable 
25. Utalization of any resource through pooling instead of create manually
26. Remove unused code from Program
27. Try to remove the empty method 

DB Side 

1. Query should be properly tune
2. Implement the Cache
3. Lazy load instead of eagar 
4. Search Fields should be indexed
5. Try to implement the sharding in DB level 
6. Try to avoid the DB trigger 
7. Create the View in DB to retrieve the data from multiple Table (Read Only)

 
Tools :

Sonar Qube -Static Analysis
JUnit -Unit Test
Check Style - Format /Comment