I would think that map is inappropriate here since you are using it in a void context. foreach makes much more sense, as moritz and ikegami have done.

Update: Tanktalus beat me to it.