从函数返回的结构的C++指针导致访问冲突

Question

我是C++的新手.我使用Visual Studio.我的调试版本按预期工作,但发布版本因访问冲突而崩溃.结构:

struct GeneticData
{
    int NumberOfIdDataPairs;
    char** IdBuffer;
    char** DataBuffer;
};

IdBuffer是一个由20个字符组成的char数组,Databuffer是一个由102个字符组成的字符数组.返回指针的函数:

GeneticData* LoadData(FILE* dataFilePtr) //dataFilePtr is valid in the function
{
    char* result = NULL;
    int fileLine = 0;
    GeneticData geneticData = *new GeneticData();

    geneticData.NumberOfIdDataPairs = *new int;
    geneticData.IdBuffer = new char*[gSizeOfBuffer];
    geneticData.DataBuffer = new char*[gSizeOfBuffer];

    for (int i = 0; i < gSizeOfBuffer; i++)
    {
        geneticData.IdBuffer[i] = new char[gSizeOfId];
        geneticData.DataBuffer[i] = new char[gSizeOfData];
    }

    do
    {
        result = fgets(geneticData.IdBuffer[fileLine], gSizeOfId, dataFilePtr);
        fgets(geneticData.DataBuffer[fileLine], gSizeOfData, dataFilePtr);
        fileLine++;
    } while (result != NULL && fileLine < gSizeOfBuffer);

    if (result != NULL)
        geneticData.NumberOfIdDataPairs = fileLine;
    else
        geneticData.NumberOfIdDataPairs = fileLine - 1;

    return &geneticData;
}

gSizeofBuffer是id/data对的数量,目前它的值是1.000.000.在函数内部一切都很好,我可以访问id/dataBuffer的值.问题出现在调用我的LoadData函数的函数中.

GeneticData geneticData;
geneticData = *LoadData(dataFilePtr);
//acces violation, geneticData contained invalid values

随着cout << geneticData.DataBuffer[1]调试器停在这里(验尸调试):

随着cout << geneticData.DataBuffer[1][0]调试器停在这条线.在分析DataBuffer内容的实际算法中,调试器在cout <<的情况下停止geneticData.DataBuffer[1][0].我将GeneticData结构的内存分配为动态内存,我不明白为什么在LoadData函数返回后指针无效.

Answer 1

GeneticData geneticData = *new GeneticData();

这个怪物动态分配一个对象，使用它来复制初始化一个局部变量，然后丢弃指向它的唯一指针，导致内存泄漏。

return &geneticData;

这将返回一个指向局部变量的指针，该指针在调用者可以对其进行任何操作之前被销毁；导致你的错误。

或许你想返回动态对象（用完记得删除它）：

GeneticData * geneticData = new GeneticData();
//...
return geneticData;

或者你想按值返回一个对象：

GeneticData LoadData(FILE* dataFilePtr)
{
    GeneticData geneticData;
    //...
    return geneticData;
}

但你应该帮自己一个忙，不要乱用指针和new。使用std::vector的缓冲区。

Answer 2

你正在返回一个指向临时结构的指针:

GeneticData geneticData = *new GeneticData(); // memoryleak: copies from dynamic memory into local
...
return &geneticData; // returns address of local memory

如果要初始化本地结构,请写入:

GeneticData geneticData; // calls the default constructor GeneticData() to initialise geneticData

请注意,geneticData只有在范围结束时才有效,因此该变量的地址也是如此!

如果需要动态内存写:

GeneticData* pGeneticData = new GeneticData(); // allocates dynamic memory (new), calls the default constructor and assigns the memory address to the pointer pGeneticData