刪除字符串

Question

這裏堆損壞時是我的代碼：

std::string readString() 
{ 
    int strLen = Read<int>(); 
    char* rawString = new char[strLen]; 
    Read(rawString, strLen); 
    rawString[strLen] = '\0'; 
    std::string retVal(rawString); 
    delete [] rawString; 
    return retVal; 
} 

第一行讀取字符串的長度。
第二行創建一個字符串長度爲
的新字符數組（字符串）第三行讀取字符串（從文件中讀取它）
第4行將NULL添加到最後。
第5行創建一個std ::字符串出c字符串。
第6行刪除c字符串（HEAP CORRUPTION HAPPENS HERE）
第7行返回字符串，但由於錯誤，它永遠不會到達這一點。

在第6行，我得到一個堆損壞錯誤： CRT檢測到應用程序在堆緩衝區結束後寫入內存。

我的問題可能很明顯，但爲什麼我得到堆腐敗？當我創建一個std :: string時，它應該複製該字符串，並且我應該安全地刪除該c字符串。

目前，我懷疑std :: string在刪除它後試圖訪問c字符串。

任何想法？

Answer 1

變化：

char* rawString = new char[strLen]; 

到：

char* rawString = new char[strLen + 1]; 

Answer 2

int strLen = Read<int>()可能只返回一個非空值終止字符串的長度，而當你嘗試寫\0字節的字符串，您遇到緩衝區溢出問題。

你應該檢查什麼strLen是的，很有可能你要麼必須分配是這樣的：

char *rawString = new char[strlen+1]; 

或使用std::string(const char *, size_t n)這樣的重載的構造函數：

std::string retVal(rawString, strlen); 

Answer 3

你所訪問超過字符串的保留字節。您保留了strLen個字符，但在strLen處輸入了\0。從0開始計數爲C數組，字符strLen位於strLen + 1，因此您要爲該字符串的保留空間外放置一個值。您應在main的第二行中保留strLen + 1，以便您的代碼正常工作。

Answer 4

由於陣列在C++ 0索引，在創建大小strLen的陣列，然後將一個0在strLen位置，你是在你分配的數組結束後寫入一個零。

Answer 5

rawString[strLen] = '\0'; 

將NUL寫入已分配空間的末尾。

如果strLen爲10，則爲10個字符分配空間，讀取10個字符，然後將此NUL寫入位置11。Ooops

Answer 6

到目前爲止的許多建議，但沒有一個解決異常安全問題：你如何擺脫潛在的內存泄漏？

有兩種方法可以避免與new（因此面臨內存泄漏）分配。首先是非常簡單，並且利用被稱爲VLA爲可變長度數組一個編譯器擴展的：

std::string readString() 
{ 
    int strLen = Read<int>(); 
    char rawString[strLen+1]; // VLA: the length is determined at runtime 
          // but the array is nonetheless on the stack 
    Read(rawString, strLen); 
    rawString[strLen] = '\0'; 

    std::string retVal(rawString); 
    return retVal; 
} 

另一種是符合標準：string具有可以訪問（感謝GMAN內部緩衝器，data是不正確的訪問方法）

std::string readString() 
{ 
    int strLen = Read<int>(); 

    std::string retVal(strLen, '\0'); // no need to allocate extra space 

    Read(&retVal[0], strLen);  // &retVal[0] gives access to the buffer 

    return retVal; 
} 

我相信最後的版本更好。不再涉及任何複製:)