no work
refactor controller and model