- Long finder methods instead of using named scopes.
- Not using restful routes.
- Too much code in their views.
- After saves that should be validations or other confusing AR lifecycle mistakes/abuses.