Điều gì xảy ra nếu bạn gọi erase () trên một phần tử bản đồ trong khi lặp từ đầu đến cuối?

Trong đoạn mã sau tôi lặp qua bản đồ và kiểm tra xem có cần xóa phần tử không. Có an toàn để xóa phần tử và tiếp tục lặp lại hay tôi cần thu thập các khóa trong một thùng chứa khác và thực hiện một vòng lặp thứ hai để gọi erase ()?

map<string, SerialdMsg::SerialFunction_t>::iterator pm_it;
for (pm_it = port_map.begin(); pm_it != port_map.end(); pm_it++)
{
    if (pm_it->second == delete_this_id) {
        port_map.erase(pm_it->first);
    }
}

CẬP NHẬT: Tất nhiên, sau đó tôi đọc câu hỏi này mà tôi không nghĩ sẽ liên quan nhưng trả lời câu hỏi của tôi.

C ++ 11

Điều này đã được sửa trong C ++ 11 (hoặc xóa đã được cải thiện / thực hiện nhất quán trên tất cả các loại container).
Phương thức xóa bây giờ trả về iterator tiếp theo.

auto pm_it = port_map.begin();
while(pm_it != port_map.end())
{
    if (pm_it->second == delete_this_id)
    {
        pm_it = port_map.erase(pm_it);
    }
    else
    {
        ++pm_it;
    }
}

C ++ 03

Xóa các phần tử trong bản đồ không làm mất hiệu lực bất kỳ trình lặp nào.
(ngoài các trình lặp trên phần tử đã bị xóa)

Trên thực tế, việc chèn hoặc xóa không làm mất hiệu lực của bất kỳ trình lặp nào:

Cũng xem câu trả lời này:
Kỹ thuật Mark Ransom

Nhưng bạn cần phải cập nhật mã của mình:
Trong mã của bạn, bạn tăng pm_it sau khi gọi xóa. Tại thời điểm này, nó đã quá muộn và đã bị vô hiệu.

map<string, SerialdMsg::SerialFunction_t>::iterator pm_it = port_map.begin();
while(pm_it != port_map.end())
{
    if (pm_it->second == delete_this_id)
    {
        port_map.erase(pm_it++);  // Use iterator.
                                  // Note the post increment.
                                  // Increments the iterator but returns the
                                  // original value for use by erase 
    }
    else
    {
        ++pm_it;           // Can use pre-increment in this case
                           // To make sure you have the efficient version
    }
}

Đây là cách tôi làm điều đó ...

typedef map<string, string>   StringsMap;
typedef StringsMap::iterator  StrinsMapIterator;

StringsMap m_TheMap; // Your map, fill it up with data    

bool IsTheOneToDelete(string str)
{
     return true; // Add your deletion criteria logic here
}

void SelectiveDelete()
{
     StringsMapIter itBegin = m_TheMap.begin();
     StringsMapIter itEnd   = m_TheMap.end();
     StringsMapIter itTemp;

     while (itBegin != itEnd)
     {
          if (IsTheOneToDelete(itBegin->second)) // Criteria checking here
          {
               itTemp = itBegin;          // Keep a reference to the iter
               ++itBegin;                 // Advance in the map
               m_TheMap.erase(itTemp);    // Erase it !!!
          }
          else
               ++itBegin;                 // Just move on ...
     }
}

Đây là cách tôi sẽ làm điều đó, khoảng:

bool is_remove( pair<string, SerialdMsg::SerialFunction_t> val )
{
    return val.second == delete_this_id;
}

map<string, SerialdMsg::SerialFunction_t>::iterator new_end = 
    remove_if (port_map.begin( ), port_map.end( ), is_remove );

port_map.erase (new_end, port_map.end( ) );

Có điều gì đó kỳ lạ về

val.second == delete_this_id

nhưng tôi chỉ sao chép nó từ mã ví dụ của bạn.