- Does this change accomplish what it's supposed to do?
- Could this solution be simplified?
- Could the maintainability or readability be improved?
- Does similar functionality already exist in the code? If so, why is it not used?
- Does the solution use hard-coded variables? If so, could they be changed?
- Are there any best practices that could substantially improve this code?
- Can you think of any use case in which the code doesn't behave as intended?