use const
pass id() instead of iterator