c++ - 访问冲突读取数组元素

Question

我已经编写了自己的代码来解析 .obj 模型文件——本质上只是 ASCII 文本。根据我的测试，文件被正确解析并存储在类中。我可以在加载函数中(从数据成员)读回值。

当我尝试在主渲染循环中读回值时出现问题。在以“int v”开头的行中存在访问冲突错误:

 for(int i = 0; i<data.numFaces; i++){
  for(int j = 0; j<3; j++){ //Assuming triangles for now.

   int v = data.faceList[i].vertex[j]; // Access violation here.
   double vX = data.vertexList[v].x;
   double vY = data.vertexList[v].y;
   double vZ = data.vertexList[v].z;
   glVertex3d(vX, vY, vZ);
  }
 }

我不确定为什么会发生这种情况，我已经检查了我能想到的所有内容。我在 C++ 方面不是很有经验。尽管我以前用 C++ 编写过一个中型项目，但我的大部分编程经验都是使用 Java、Python 和 PHP。

我确定问题出在与内存分配或用于动态数组的指针相关的基本问题上。

下面是obj加载类中的相关代码部分:

ObjData ObjLoader::LoadObj(std::string filename){

    //... Initalization ...

 // 1st pass: find number of elements so array sizes can be defined.
 while(!file.eof()){
  //...
 }

 //...close file...

 _data.faceList = new ObjFace[_data.numFaces];
 _data.vertexList = new ObjVert[_data.numVertices];
 _data.uvList = new ObjUV[_data.numUVcoords];
 _data.normalList = new ObjNormal[_data.numNormals];

    //TODO: Make size dynamic according to each face. Just use the first 3 points for now.
 for (int i = 0; i < _data.numFaces; i++){
  _data.faceList[i].vertex = new int[3];
  _data.faceList[i].normal = new int[3];
  _data.faceList[i].uv = new int[3];
 }

 //... file stuff ...

 // 2nd pass: read values into arrays.
 while(!file.eof()){
  //...

  if(type=="v"){
   _data.vertexList[currentVertex].x = atof(param1.c_str());
   _data.vertexList[currentVertex].y = atof(param2.c_str());
   _data.vertexList[currentVertex].z = atof(param3.c_str());
   currentVertex++;
  }else if(type=="vt"){
   _data.uvList[currentUV].u = atof(param1.c_str());
   _data.uvList[currentUV].v = atof(param2.c_str());
   currentUV++;
  }else if(type=="vn"){
   _data.normalList[currentNormal].x = atof(param1.c_str());
   _data.normalList[currentNormal].y = atof(param2.c_str());
   _data.normalList[currentNormal].z = atof(param3.c_str());
   currentNormal++;
  }else if(type=="f"){
  //...Within loop over each vertex in a single face ...

        if(endPos != string::npos){
        // Value before 1st "/" (Vertex index).
        // ...find value in string...
        _data.faceList[currentFace].vertex[i] = atoi(token.c_str()) -1; // File format begins indices from 1.

        // Value between slashes (UV index).
        // ...find value in string...
        _data.faceList[currentFace].uv[i] = atoi(token.c_str()) -1;

        // Value after 2nd "/" (Normal index).
        // ...find value in string...
        _data.faceList[currentFace].normal[i] = atoi(token.c_str()) -1;
   }
//...End of loop over every vertex in a single face...
currentFace++;
}

}

 return _data;

    }

结构体 ObjFace、ObjVert、ObjUV 和 ObjNormal 定义为:

    struct ObjVert{
       float x, y, z;
    };

    struct ObjUV{
      float u, v;
    };

    struct ObjNormal{
       float x, y, z;
    };

    // Contains indexes.
       struct ObjFace{
       int* vertex;
       int* uv;
       int* normal;
    };

感谢您的帮助。此外，我们将不胜感激任何有关避免将来出现此类错误的好资源。

Answer 1

我输入了一些愚蠢的回复，我意识到这是不对的……但我不得不继续思考，然后我想出了另一个主意。

这个对象被分配到哪里？

你的代码不清楚 data 和 _data 是否是同一个对象，但我注意到你的方法似乎返回了 _data 作为一个对象。我被引导相信，也许您正在某处使用像 ObjData data = LoadObj("myfilename"); 这样的赋值？

如果是这种情况，我相信您的问题可能是因为您的 ObjData 类缺少复制构造函数或重载赋值运算符。 (我不是 C++ 大师，所以我不记得它属于哪一个。希望其他人可以确认我们是否走在正确的轨道上)。

如果在复制和赋值过程中你的指针没有被正确复制(从 LoadObj 返回调用复制构造函数 iirc，然后明显赋值给 data)，那么即使您打算在该位置已经有一个 int 数组，您实际上可能正在访问未初始化的内存，从而导致您的访问冲突。

我不是复制构造函数或重载赋值运算符方面的专家，但解决此问题的一种快速方法是返回指向 ObjData 的指针，而不是返回对象本身。