Segfault from std :: getline with delimiter

Question

数時間にわたって、私はこのsegfaultを修正しようとしています。このコードは、500回目の反復後に常にSIGSEGVを送信します。 Here's the TEST.csv I've been using。 whileループが2番目のAMU値セットにヒットすると、getlineはすぐにプログラムをクラッシュさせます。私はこれを梳き、私の人生を救うために問題を見つけることができません。

WORTH NOTING：このエラーはすべてのマシンで再現できません。間違いなく、どこかに乱暴になっているいくつかのメモリが、私はそれを把握することはできません！

#include <iostream> 
#include <fstream> 
#include <string> 
#include <cstdlib> 
#include <climits> 
using namespace std; 
int main(int argc, char **argv) 
{ 
    ifstream data; 
    data.open("TEST.csv", ifstream::in); 
    float amudata[505]; 
    //initialize amudata 
    for (int x = 0; x < 505; x++) amudata[x] = INT_MIN; 
    std::string line; 
    //toss out the first 137 lines of config data 
    for (int x = 0; x < 137; x++) 
    { 
     std::getline(data, line); 
    } 
    //debug iteration counter 
    int x = 0; 
    //toss out the first part of the timestamp 
    //this is where SEGV happens 
    while (std::getline(data, line, ' ')) 
    { 
     x++; 
     //toss out second part of timestamp 
     getline(data, line, ','); 
     //read and store the index, which is amu times 10 
     std::getline(data, line, ','); 
     int index = std::atof(line.c_str()) * 10; 
     //read and store the amu intensity 
     std::getline(data, line, ','); 
     float intensity = std::atof(line.c_str()); 
     //some debug output 
     cout << index << " " << intensity << " "; 
     cout << line << " " << x << endl; 
     //keep track of only the maximum intensities 
     if (amudata[index] < intensity) 
     { 
      amudata[index] = intensity; 
     } 
    } 
    for (int x = 0; x < 505; x++) 
    { 
     printf("%f\n", amudata[x]); 
    } 
    data.close(); 
    return 0; 
} 

Answer 1

amudataの配列が小さすぎます。

あなたのプログラムがクラッシュしますが、この行を処理した後：あなたが行うと

2016/11/23 16:49:06.146, 50.500, -3.6263e-010, 

を：

int index = std::atof(line.c_str()) * 10; 

lineは"50.500"あるので、これはindex = 505を設定します。次に、あなたが実行します。

amudata[index] = intensity; 

しかし、あなたは未定義の動作の原因となる配列の範囲、外に書いているようamudataの許容インデックスは、0から504にあります。

float amudata[506]; 
//initialize amudata 
for (int x = 0; x < 506; x++) amudata[x] = INT_MIN; 

をそして、それは、プログラムのその周りのようなマジックナンバーを広める定数やマクロを使用しないことが最善です：

あなたが必要です。