Is Unneeded code?

[zh820952173]

_cache_items_map.erase(it);
Is the line of code redundent?

[zh820952173]

_cache_items_map[key] = _cache_items_list.begin(); Reference to the mapped value of the new element if no element with key key existed. Otherwise a reference to the mapped value of the existing element whose key is equivalent to key.