C++不知道內容改變的原因

Question

代碼來自我的朋友。請看看它，並給我們一些建議。

in VoltageForDisplayAndAnalyze.h file 

private: 

string * grpVolFileName[ MAX_NUM_OF_GROUP ]; //for voltage files 
fstream * aStream_grpVolt[ MAX_NUM_OF_GROUP ]; 

in VoltageForDisplayAndAnalyze.CPP file 

void VoltageForDisplayAndAnalyze::MakeFileNameForGroupCell(int xgrpIndex) 
{ 
    int numOfFiles = (EEnd - EBegin + 1)/255; 
    grpVolFileName[ xgrpIndex ] = new string[ numOfFiles ];  
    assert(grpVolFileName[ xgrpIndex ] != NULL); 
    aStream_grpVolt[xgrpIndex] = new fstream[ numOfFiles ]; 
    assert(aStream_grpVolt[ xgrpIndex ] != NULL); 
    ..... 
    for(int fileIndex=0; fileIndex<numOfFiles; fileIndex++) 
    {    
    char * tempbuf = new char[256]; 
    memcpy(tempbuf, mPath.c_str(), mPath.length()); 
    tempbuf[mPath.length()] = '\0'; 
    char * numChar = new char[4]; 
    itoa(xgrpIndex, numChar, 3); 
    numChar[3] = '\0'; 

    strcat(tempbuf, numChar); 
    grpVolFileName[ xgrpIndex ][ fileIndex ].assign(tempbuf); 

    }  
    .... 
} 

我們在上面的方法中打印出來，給grpVolFileName賦值，結果看起來不錯。這些值是某些文件的位置。然後調用以下方法來使用上面的字符串指針的grpVolFileName數組。這對第一個4是好的，然後打印出來顯示目錄不能正確打印出來，有些正方形打印出來，併發生錯誤。但它並不總是在第四次，有時它剛剛開始。

void VoltageForDisplayAndAnalyze::SaveDataForGrpCell(int grpIndex, int fileIndex, int xBegin, int xEnd  { 

    int i = 0; 
    char answer; 

//save to hard disk 
aStream_grpVolt[grpIndex][fileIndex].open( grpVolFileName[grpIndex][fileIndex].c_str() , fstream::out | std::ofstream::app); 
cout << "open file: grpIndex= " << grpIndex <<" fileIndex= " << fileIndex << " : " << grpVolFileName[grpIndex][fileIndex] << " to save." << endl; 
if(aStream_grpVolt[grpIndex][fileIndex].good()) 
{ 
    //..... 
} 
else 
{ 

    cout << "Cannot open file: " << grpVolFileName[grpIndex][fileIndex] << endl; 
    } 

    //....... 
} 

打印出從上方象那些方法：

... 
open file: grpIndex= 2 fileIndex= 0 : C:\Workspace\UncusJava\DataFiles\spc1\2 to save. 
open file: grpIndex= 3 fileIndex= 0 : C:\Workspace\UncusJava\DataFiles[][][][]4@ to save. 

這意味着grpVolFileName陣列已經在過程被改變。如果條款if(aStream_grpVolt[grpIndex][fileIndex].good()) 導致此錯誤，則返回錯誤併發生錯誤。

我搜索了整個解決方案（Visual Studio 2008），上面兩個方法都是隻使用grpVolFileName指針數組的地方。這是這部分的單線程。

有何評論？

Answer 1

我讀了兩次你的問題，但我不知道你想達到什麼。從我所瞭解的你本質上遍歷目錄層次結構。它（幾乎）並不關乎你正在處理的平臺：有可能比C++更好的工具。有機會，您可以更快速，更輕鬆地敲擊劇本，其代碼更短，並且正確可用於執行相同的任務。

Answer 2

我很抱歉地說你的朋友的代碼真的是一團糟。我懷疑你正在for循環中破壞你的字符串。你可以嘗試把它改寫爲：

在VoltageForDisplayAndAnalyze.cpp：

void VoltageForDisplayAndAnalyze::MakeFileNameForGroupCell(int xgrpIndex) 
{ 
    int numOfFiles = (EEnd - EBegin + 1)/255; 
    grpVolFileName[xgrpIndex] = new string[numOfFiles]; 
    aStream_grpVolt[xgrpIndex] = new fstream[numOfFiles]; 
    // The asserts are not needed: new will throw an exception 
    // if the allocation fails. 
    ... 
    for (int fileIndex = 0; fileIndex < numOfFiles; fileIndex++) 
    {    
     // You should use std::string instead of C strings whenever 
     // possible (and it means almost always: std::string can 
     // interoperate with C strings). 
     string tempbuf = mpath; 

     // This is the C++ way of converting numbers to strings. 
     ostringstream oss; 
     oss << xgrpIndex; 
     string numChar = oss.str(); 

     tempbuf += numChar; 
     grpVolFileName[xgrpIndex][fileIndex] = tempbuf; 

    } 
    ... 
} 

// Remember to delete the arrays you have allocated with new: 
// delete[] grpVolFileName[xgrpIndex]; 
// delete[] aStream_grpVolt[xgrpIndex]; 

雖然仍不乾淨簡潔（在for循環是相當多餘的代碼，我們可以使用的std ::載體，而不是C數組），現在這個代碼保證將正確的數據分配給grpVolFileName[xgrpIndex][fileIndex]。

嘗試運行此代碼並查看您的錯誤是否已解決。

Answer 3

雖然代碼是醜陋的，顯然有內存泄漏，我沒有看到任何東西會損壞你的字符串。它們有可能在代碼中的其他地方被覆蓋。

您可以通過在Visual Studio中設置數據斷點來測試這種情況，以便在字符串被修改時斷開。

像Valgrind（不適用於Windows）和Purify（商業）的軟件也可以幫助檢測像這樣的內存損壞。