C++ - ошибка сегментации при вставке в карту или вектор

Question

Так что я вставляю кучу значений в вектор (я пробовал карту с теми же результатами), и я продолжаю получать ошибку сегментации на моем втором векторе, который является VectorMin , Я пробовал раскомментировать это, и если я работаю только с VectorMax, все отлично работает.

По какой-то причине, если я попытаюсь манипулировать обоими векторами, я получу ошибку сегментации. Если я раскомментирую один, программа загрузится правильно.

Я использую файл 3.5mb с 1000 строками и 362 значениями для каждой строки.

std::vector<float> vectorMax, vectorMin; 

void Parser::isMaxMinVector(float value, int index){ 

    //if the index of the vector is not yet used 
    if (index >= vectorMax.size()){ 
     vectorMax.push_back(value); 
    } 

    if(index >= vectorMin.size()){ 
     vectorMin.push_back(value); 
    } 

    //if new vector is larger, then overwrite it 
    //if(vectorMax[index] > value) vectorMax[index] = value; 
    //if(vectorMin[index] < value) vectorMin[index] = value; 
} 


void Parser::parseLine(char* line){ 
    std::vector<float> vectors; 
    char* point; 

    char* tk1 = strtok(line, ","); 
    char* tk2 = strtok(NULL, ","); 

    int i=0; 

    if(tk1 == NULL || tk2 == NULL) return; 

    float x = strtof(tk1, NULL); 
    float y = strtof(tk2, NULL); 

    XYPair pair = XYPair(x, y); 
    isMaxXY(x,y); 

    while(point=strtok(NULL, ",")){ 
     //convert the token to float 
     float f_point = strtof(point, NULL); 
     //push the float to the vector 
     vectors.push_back(f_point); 
     isMaxMinVector(f_point, i); 
     i++; 
    } 
} 

Answer 1

У вас есть размер векторов, чтобы у них было достаточно места для хранения? Оператор IIRC [] не изменяет размер, его неопределенное поведение, если в этом элементе нет значения. Если вы хотите что-то редкое, вам придется использовать что-то вроде карты.

Answer 2

Вы несколько раз меняли код своего сообщения. Тем не менее этот фрагмент кода

if (index >= vectorMax.size()){ 
    vectorMax[index] = value; 
} 

недействителен, так как вы используете несуществующий элемент с индексом> = size(). Я думаю, что вы имели в виду

if (!vectorMax.empty() && index < vectorMax.size()){ 
    vectorMax[index] = value; 
} 

Или примените функцию члена изменить размер. Например,

if (index >= vectorMax.size()){ 
    vectorMax.resize(index + 1); 
    vectorMax[index] = value; 
} 

Answer 3

Я думаю, что Влад из Москвы ответил, как показано ниже. Правильно ли это работает?

if (index >= vectorMax.size()){ 
     vectorMax[index] = value; 
    } 

Answer 4

Если вы не трогайте векторы в другом месте, это будет работать:

//if the index of the vector is not yet used 
if (index >= vectorMax.size()){ 
    vectorMax.push_back(value); 
    // update index here: 
    index = vectorMax.size(); 
} 

if(index >= vectorMin.size()){ 
    vectorMin.push_back(value); 
    // and here: 
    index = vectorMax.size(); 
} 
//if new vector is larger, then overwrite it 
if(vectorMax[index] > value) vectorMax[index] = value; 
if(vectorMin[index] < value) vectorMin[index] = value; 

Проблема об этом, однако, заключается в том, что в случае изменения индекса index значение не будет обновляться. Вы можете изменить это, используя int &index в вставленной подписи. Для этого подход Владимира намного лучше. С другой стороны, неясно, что содержат другие элементы в векторах (если вы могли бы создавать артефакты, например).