use forEach instead