short refactory in controller