refactoring userController show