C++ vector : unused capacity

Question

我是 C++ 的新手，我目前正在处理这个类(class)表项目，以下是我的 add_course 函数。我在 map 中存储学生的 ID、学期和类(class)列表。它有效(我可以向学生添加类(class)并在打印学生的时间表时看到它)但是每次运行代码时，总是会弹出一个异常:

Exception thrown: read access violation.
std::_Vector_alloc<std::_Vec_base_types<course *,std::allocator<course *> > >::_Myend(...) returned 0xC.

我很困惑，为什么会这样。这是否意味着我的载体的容量超出了它的需要？

void add_course(map<int, map<int, vector<course *> * > > &DB, int semester, int id, course c) {
    const string a = c.name;
    course* b = new course(a, c.section, c.credits);
    if (DB[id].empty()) {
        vector<course*> *p2 = new vector<course*>;
        (*p2) = { b };
        DB[id][semester] = p2; 
        return;
    }
    else
        DB[id][semester]->push_back(b);
    return;
}

Answer 1

您的代码假设如果 DB[id] 有any 个学期，它有学期 semester:

if (DB[id].empty()) {
   // ...
}
else
    DB[id][semester]->push_back(b);

如果那不是真的，如果它有 一些 个学期但不是这个学期，那么您就没有捕获它，并且您正在 push_back 默认 -构造空指针。这意味着您的程序有未定义的行为，今天是崩溃的原因。

相反，您可以尝试:

if (!DB[id].count(semester)) {
   // ...
}
else
    DB[id][semester]->push_back(b);

…实际上检查特定内部 map 元素是否存在。

不过，如果没有所有这些动态分配，你会过得更好:

void add_course(
   map<int, map<int, vector<course>>>& DB,
   const int semester,
   const int id,
   const course c
)
{
    DB[id][semester].emplace_back(c.name, c.section, c.credits);
}

那不是更好吗？!