can we please use an iterator here