map will return value, use forEach instead of map