working but form code is not good