Proper way to have a controller