將字符串中的浮點數放到void數組中

Question

當我的函數從字符串中讀取浮點數並將它們放入void數組中時，出現分段錯誤。

// Allocate memory 
void** data; 
data = (void**)malloc(num_vals * sizeof(float)); 

// Convert text to floats 
(*(float**)data)[0] = atof(strtok(text, " ")); 
for(int index=1; index<num_vals; index++) { 
    (*(float**)data)[index] = atof(strtok(NULL, " ")); 
    std::cout << (*(float**)data)[index] << std::endl; 
} 

因爲該串中的大小和數據的類型是在運行時確定的空隙陣列是必要的：在段錯誤後約200次迭代的在下面的代碼環路發生。我試過增加malloc大小，但它不會改變任何東西。有什麼想法嗎？

Answer 1

爲什麼你轉換爲void ** ???您的代碼包含在索引夫婦的錯誤，所以讓我表現出一定的合理變化

float* data; 
data = (float*)malloc(num_vals * sizeof(float)); 

// Convert text to floats 
data[0] = atof(strtok(text, " ")); 
for(int index=1; index<num_vals; index++) { 
data[index] = atof(strtok(NULL, " ")); 
std::cout << data[index] << std::endl; 
} 

Answer 2

我想，既然你要定義空指針的指針和分配/它鑄造void指針的指針，它分配4個字節的存儲器對於每個元素，因爲在C/C++中，指針類型的指針，指針始終是4個字節，它們不夠大。

Answer 3

你把你的類型混淆在你編造「C++」的幌子下，你莫名其妙地嘗試創建這個怪物。無論如何。什麼你mallocing不過是float*，所以你需要投data回float*：

((float*)data)[0] = myfloat; 

Answer 4

是有幾個問題。一個是 - data應該是void *，你有多餘的*。其他可能是對齊，我不確定你能夠在內存中的任何位置放置一個浮點。

Answer 5

認真??

std::vector<float> data; 
std::istringstream str(text); 
float fv; 
while (str >> fv) 
{ 
    data.push_back(fv); 
} 

現在，這是C++

Answer 6

，因爲它的痛苦，我也這樣做了，這裏是你的代碼的一個版本，可能你想要做什麼。

// Allocate memory 
void* data; 
data = malloc(num_vals * sizeof(float)); 

// Convert text to floats 
((float*)data)[0] = atof(strtok(text, " ")); 
for(int index=1; index<num_vals; index++) { 
    ((float*)data)[index] = atof(strtok(NULL, " ")); 
    std::cout << ((float*)data)[index] << '\n'; 
} 

但是請注意，如果你爲我工作，並試圖在該代碼來檢查，我們就必須對你選擇職業的認真討論。

我寧願看到這樣的事情：

std::vector<float> v; 
std::copy(std::istream_iterator<float>(std::istringstream(text)), 
      std::istream_iterator<float>(), 
      std::back_inserter(v)); 

附：羅布的規則＃47：當你​​的意思是'\n'時，千萬不要說std::endl。