i do not know if i am doing it right