- Is the functionality correct?
- Are the classes named suitably?
- Are the functions named suitably?
- How's the datastructure being used? Is it the correct DS or it needs improvement?
- Can the classes be further borken into small classes?
- Do we need an interface?
- If it contain functions that can be reused later then are there Utils created for them?
- Can it use already available Util functions?
- Does the large set of input for a function deserve a new bean to be created?