尝试从指针向量中取消引用指针时出现分段错误

Question

我有一个指向对象的指针向量，我正在使用 std::vector::iterator` 对其进行迭代。由于返回的元素本身就是一个指针，因此我取消引用迭代器两次，一次返回指针，一次将指针解析为实际对象。

我正在尝试调用返回 std::string 的成员函数 (getClass)，并且我尝试了 (**it).getClass() 和 (*it)->getClass() 但两者都给我一个分段错误。我总觉得我错过了一些明显的东西。

部分功能代码：

void dataSet::createFolds()
{
   // Shuffle the data vector
   std::random_shuffle( m_records.begin(), m_records.end());

   std::cout << "STARTING MAIN LOOP.  THERE ARE " << m_records.size() << " RECORDS\n";
   // iterate through the data vector and assign each to a fold
   std::vector<dataRecord *>::iterator it = m_records.begin();
   while (it != m_records.end())
   {
      std::string currentClass = (*it)->getClass();  // SEG FAULT HERE
      .
      .
      .
   }
   .
   .
   .
}

向量为m_records ... 代码

dataRecord 定义如下... code

回答有关填充向量的问题：

数据是从文本文件中读取的，我真的不想发布整个内容除非我必须（212 行），但填充向量的相关代码如下。 dataRecord 对象的构造函数采用字段对象向量。我使用临时指针，使用 new 创建对象，然后 Push_back 指针。

while ...
{
   std::vector<field> fields;

   // build the fields vector
   for (unsigned int i = 0; i < numAttribs; ++i)
      fields.push_back(field(data.at(i), attribTypes[i]));

   // create the new dataRecord
   dataRecord * newRecord = new dataRecord(fields);

   // add the record to the set
   m_records.push_back(newRecord);

   ++recordNum;
   std::cout << "read record " << recordNum << std::endl;
}

Answer 1

在我看来，向量元素的初始化很糟糕。也许您必须在测试提取向量之前独立测试填充向量的代码。对不起我的英语;)

Answer 2

容器中的指针要么为空，要么它们是指向已释放内存的悬空指针。

仔细检查填充 m_records 的代码。

Answer 3

您

std::string dataRecord::getClass() {return m_data.at(m_data.size() - 1).getTextData();}

必须验证 m_data.size() 因为可能为 0，因此您将得到超出或范围的异常。

Answer 4

// create the new dataRecord
   dataRecord * newRecord = new dataRecord(fields);

我猜这个错误是在 dataRecord 的构造函数中。你确定它正确地完成了它的工作吗？

Answer 5

这不一定适用于OP的问题，但对于那些通过谷歌搜索到达这里的人...

如果您在解除引用向量迭代器上收到段错误并且正在多线程应用程序中工作，请记住向量不一定是线程安全的！

如果没有互斥锁，以下示例是不安全的。

线程 1

myVector.push_back(new Object());

线程 2

std::vector<Object*>::iterator = myVector.begin();
for (it; it != myVector.end(); it++) {
    Object* obj = *it;
}

相反，应该执行类似的操作：

myMutex.lock();
myVector.push_back(new Object());
myMutex.unlock();

线程 2

myMutex.lock();
std::vector<Object*>::iterator = myVector.begin();
for (it; it != myVector.end(); it++) {
    Object* obj = *it;
}
myMutex.unlock();