- Skinny controller , fat model convention , please follow
- Ambiguous variables
- wrong identation , 2 spaces