I would also advice you to get rid of the new. Just return a const reference to the object pointed to by the iterator (or in this case, always .begin())!