why aren't you using a for loop here?