question & refactor controller