what mistake i have done in this program