將std :: list轉換爲char * [size]

Question

由於某些原因，我無法解釋，字符數組中的每個項目...等於最後添加的項目...例如progArgs [0]通過progArgs [size]包含最後一項的值。

我無法弄清楚我在做什麼對我來說是錯誤的。有什麼建議麼？

int count = 0; 
char *progArgs[commandList.size()] 

    for(list<string>::iterator t=commandList.begin(); t!=commandList.end(); t++) 
    { 
     char item[strlen((*t).c_str())]; //create character string 
     strcpy(item, (*t).c_str()); //convert from const char to char 
     progArgs[count] = item; 
     count++; 
    } 

編輯：

感謝所有的快速反應大家......我看你說什麼

Answer 1

您正在爲每個元素progArgs分配相同的指針（堆棧數組item的第一個元素的地址），然後重複覆蓋該內存。你可以這樣做：

progArgs[count] = strdup(t->c_str()); 

並擺脫了身體的前兩行。

strdup分配內存，所以你將不得不在每個元素後面用free釋放。另外，你沒有爲NUL終結符分配一個字符。您需要strlen + 1.但是，這不是strdup的問題，因爲它爲您分配。

Answer 2

progArgs是指向字符數組。

您將每個指針設置爲指向item。 item是循環中的局部變量，因此一旦循環退出，item不再存在，並且指針不再有效[*]。但是，在您的C++實現中，它們仍然指向堆棧上曾經是數組item的內存位。該內存包含上次使用的內容，它是列表中最後一個字符串的字符序列。

如果你想字符串列表複製到一個數組，它會更好，如果可以使用字符串數組：

std::string progArgs[commandList.size()] // if your compiler has C99 VLAs as an extension 
int count = 0; 

for(std::list<std::string>::iterator t=commandList.begin(); t != commandList.end(); ++t) { 
    progArgs[count] = *t; 
    ++count; 
} 

甚至更​​好，用一個向量而不是數組：

std::vector<std::string> progArgs(commandList.begin(), commandList.end()); 

[*]更準確地說，的item範圍是循環的一個單重複，它的標稱「創造」和「破壞」圍繞每個時間。但是這並沒有做任何工作 - 在你的C++實現中，每次都重用相同的內存區域，並且不需要在堆棧上創建或銷燬char數組。

Answer 3

您每次都將progArgs[count]設置爲相同的項目！

int count = 0; 
char *progArgs[commandList.size()] 

for(list<string>::iterator t=commandList.begin(); t!=commandList.end(); t++) 
{ 
    char * item = new char[strlen((*t).c_str()) + 1]; //create new character string 
    strcpy(item, (*t).c_str()); //convert from const char to char 
    progArgs[count] = item; 
    count++; 
} 

然後記得要爲progArgs中的每個元素調用delete[]。

就我個人而言，我會創建一個string的數組，並根據需要隨時轉換爲char *。

Answer 4

char數組item是局部變量，僅在for循環內可用。

改爲動態分配它。

Answer 5

你說得對：

for(list<string>::iterator t=commandList.begin(); t!=commandList.end(); t++) 
{ 
    char item[strlen((*t).c_str())]; //create character string 

...這並創建一個數組，但它會創建它在堆棧上，所以當你到達其範圍的結束，它就會被銷燬。

strcpy(item, (*t).c_str()); //convert from const char to char 
    progArgs[count] = item; 
    count++; 
} 

......並且這個右大括號表示其範圍的結束。這意味着你正在創建一個字符串，將它放入你的數組中，然後在添加下一個之前銷燬它。由於你在堆棧中創建它們，所創建的每個新創建的目標都與上一個創建的完全相同，所以毫不奇怪，在最後，你會有一堆相同的字符串

如果你告訴我們你想在這裏完成什麼，也許會更好，所以我們可以幫你做到這一點。

Answer 6

item具有本地循環範圍。因此，propArgs數組包含一堆基於堆棧的指針，可能都是一樣的。你可以在調試器中檢查它是如何工作的，只需循環兩次，應該清楚發生了什麼。

當您退出循環時，由公共指針尋址的緩衝區包含最近複製的c_str()。

你可以通過做

char* item = new char[strlen((*t).c_str()) + 1]; 

解決這個問題，但是，你不得不在退出循環刪除[]所有propArgs數組項。

此代碼顯示了對內存管理的基本缺乏理解，因此在重構代碼之前進一步閱讀可能會有用。如果這裏使用的代碼只比這個例子稍微複雜一些，那麼它可能只會崩潰，因爲在循環外部任何訪問propArgs都將依賴於無效（不在範圍內）item。