dolist is better than mapcar for side-effecty code