switch from for..in to each