Wouldn't foreach make more sense than map?