did hte for loop wrong