Doing a "find" first and then an access through the "[]" operator is problematic for two reasons:
1) It's inefficient since you're doing two lookups
2) The "[]" operator is mutable and won't work with a const method.

You should change this to keeping the iterator returned by "find" and return its value instead.