传递结构体数组成员触发访问违规及vector下标越界问题求助

咱们一步一步拆解你遇到的问题，把每个错误点理清楚，再给出对应的解决办法：

一、原始结构体传值+非法数组访问导致的访问违规 #

你的ReadFromStruct函数里藏着两个致命问题，直接触发了访问违规：

1. 值传递结构体的坑：函数参数是struct testStruct，这会把传入的结构体完整拷贝一份到函数栈里。虽然副本的arr指针会指向原内存，但更严重的是下面的语法错误。
2. 把结构体当数组用了：循环里写testStruct[i].x是完全错误的——testStruct是单个结构体实例，不是指针也不是数组！这种写法会把结构体的内存地址当作数组首地址来偏移访问，直接乱读内存，不触发访问违规才怪。

另外你还有个笔误：三次赋值都给了tempArr[i].x，y和z的值根本没被正确拷贝，这也是个影响逻辑的小问题。

修正后的ReadFromStruct应该这样写（用引用传递避免拷贝，同时正确访问结构体的arr成员）：

void ReadFromStruct(testStruct& ts) { // 用引用，避免拷贝且能访问原数据
    vec3 tempArr[1500] = {};
    int vertexCount = vertices.length();
    for (int i = 0; i < vertexCount; i++) {
        tempArr[i].x = ts.arr[i].x;
        tempArr[i].y = ts.arr[i].y; // 这里改回赋值给y
        tempArr[i].z = ts.arr[i].z; // 这里改回赋值给z
    }
}

如果坚持用原生数组，记得在结构体的析构函数里加delete[] arr;，不然会造成内存泄漏。

二、std::vector使用错误导致的下标越界 #

你改用vector时，核心错误是搞混了reserve()和resize()的作用：

• reserve(n)：只是给vector预留n个元素的内存空间，但实际元素数量（size）还是0，这时候直接用arr[i]访问下标i的元素，相当于访问不存在的元素，必然触发越界错误。
• resize(n)：会直接创建n个元素（默认构造或者用指定值初始化），此时vector的size变成n，可以安全访问arr[0]到arr[n-1]。

另外你第二个尝试里的临时vectortemp也犯了同样的错：temp.reserve()后size还是0，temp[i]直接越界访问。

正确的vector使用方式 #

首先修改结构体定义，把原生指针换成vector：

struct testStruct{
    std::vector<vec3> arr; // 替换原来的vec3* arr
    void resizeVerts(int size) {
        arr.resize(size); // 直接创建size个元素，之后可以直接用下标访问
        // 如果是需要逐个添加元素的场景，也可以用reserve：
        // arr.reserve(size);
        // arr.clear(); // 清空原有元素（如果需要的话）
    }
};

然后填充数组的两种正确写法：

// 方式1：resize后直接赋值（适合已知元素数量的场景）
testStruct->resizeVerts(vertices.length());
int vertexCount = vertices.length();
for (int i = 0; i < vertexCount; i++) {
    testStruct->arr[i].x = vertices[i].x;
    testStruct->arr[i].y = vertices[i].y;
    testStruct->arr[i].z = vertices[i].z;
}

// 方式2：reserve后push_back（适合逐个生成元素的场景）
testStruct->arr.reserve(vertices.length());
testStruct->arr.clear(); // 清空原有元素（如果之前有数据的话）
for (int i = 0; i < vertices.length(); i++) {
    // 如果vec3支持拷贝构造，直接push_back更简洁
    testStruct->arr.push_back(vertices[i]);
    // 或者手动构造：
    // vec3 temp;
    // temp.x = vertices[i].x;
    // temp.y = vertices[i].y;
    // temp.z = vertices[i].z;
    // testStruct->arr.push_back(temp);
}

三、总结一下最佳实践 #

• 如果用原生数组：一定要用指针/引用传递结构体，记得在析构函数里释放内存，避免泄漏。
• 强烈推荐用std::vector：它会自动管理内存，避免手动分配/释放的错误，而且用起来更安全。只要分清reserve（预留空间）和resize（创建元素）的区别，就不会再踩下标越界的坑。
• 写代码时多注意笔误，比如你之前三次赋值给x的问题，这种小错误也会导致逻辑异常。

内容的提问来源于stack exchange，提问作者Abdominator