從矢量擦除後重復的指針

Question

從指針向量中我重載了operator - =（）來移除元素（指針是唯一的，所以不需要刪除所有的事件，循環可以在指針被擦除後終止）：

Rooms& Rooms::operator-=(Course *c) { 
    for (Iter i = rooms.begin(); i != rooms.end(); ++i) { 
     if (**i == *c) { 
      *i = NULL; 
      i = rooms.erase(i); 
      break; 
     } 
    } 
    return *this; 
} 

問題是，在應用到向量後，我得到了向量的最後一個元素的重複指針。之前：

------------------------------------------------------------------- 
| POL | CAE | RUS | ENG | BUS | JPY |  |  |  |  |  | 
| G 1 | G 1 | G 2 | G 2 | G 2 | G 2 |  |  |  |  |  | 
| 9 | 9 | 10 | 10 | 10 | 10 |  |  |  |  |  | 

去除ENG &公交車經過：

------------------------------------------------------------------- 
| POL | CAE | RUS | JPY | JPY | JPY |  |  |  |  |  | 
| G 1 | G 1 | G 2 | G 2 | G 2 | G 2 |  |  |  |  |  | 
| 9 | 9 | 10 | 10 | 10 | 10 |  |  |  |  |  | 

什麼應該改爲真正得到結果爲：

------------------------------------------------------------------- 
| POL | CAE | RUS | JPY |  |  |  |  |  |  |  | 
| G 1 | G 1 | G 2 | G 2 |  |  |  |  |  |  |  | 
| 9 | 9 | 10 | 10 |  |  |  |  |  |  |  | 

任何幫助將不勝感激。

編輯：

我的打印功能如下：

std::ostream& operator<<(std::ostream& out, const Rooms& rs) { 
    std::vector<std::string> output(3); 
    std::ostringstream temp; 
    for (int i = 0; i < rs.rooms.capacity(); ++i) { 
     if (rs.rooms[i]) { 
      temp << "| " << rs.rooms[i]->getCode() << " "; 
      output[0] += temp.str(); 
      temp.str(""); 
      temp << "| G" << std::setw(CODE_LENGTH - 1) 
       << rs.rooms[i]->getGroup() << " "; 
      output[1] += temp.str(); 
      temp.str(""); 
      temp << "| " << std::setw(CODE_LENGTH) 
       << rs.rooms[i]->getSize() << " "; 
      output[2] += temp.str(); 
      temp.str(""); 
     } else { 
      output[0] += "|  "; 
      output[1] += "|  "; 
      output[2] += "|  "; 
     } 
    } 
    output[0] += "|"; 
    output[1] += "|"; 
    output[2] += "|"; 
    out << printHorizont(rs.size) << output[0] << std::endl 
     << output[1] << std::endl << output[2] << std::endl; 
    return out; 
} 

看起來很複雜，但無法找到它的打印方式，我想的更好的方法。

Answer 1

在operator<<()，

for (int i = 0; i < rs.rooms.capacity(); ++i) { 

對於rs.rooms.size() <= i < rs.rooms.capacity()，基本上rs.rooms[i]是非法的。

使用迭代器遍歷向量，你永遠不會犯這樣的錯誤。如果你真的需要打印那些「空白空間」，你可以寫成：

for (int i = 0; i < rs.rooms.capacity(); ++i) { 
    if (i < rs.rooms.size() && rs.rooms[i]) { 
     // access rs.rooms[i] 
    } else { 
     // print empty placeholder 
    } 
}