0

Доброго времени суток. Пишу достаточно простую программку под Linux. Вот кусок кода в котором вылазит ошибка сегментации:

void GetNames(vector<string> &names, vector<string> &types) { // Cчитывание и запись нужных sm-переменных из текст. файла в матрицу

if (!FlashHere()) return;

string buf;
unsigned int num = 0;
ifstream in(NAMES_DIR);
while (!in.eof()) {
    getline(in, buf, ';');
    num++;
            cout << num << endl;
}

    num = num -1;

names.resize(num);
types.resize(num);
    in.clear();
in.seekg(0, ios::beg); //возврат потока в начало файла

for (unsigned int i = 0; i < names.size(); i++)
    {

            getline(in, buf, '.');
            types[i] = buf;

    getline(in, buf, ';');
    names[i] = buf;

            cout<< "|" << names[i] << "|" << types[i] << "|" << endl;
}
in.close();

}

void InitFBD(vector<string> &names, vector<string> &types, Shm &mymem) {



    for (unsigned int num = 0; num < names.size(); num++) {

        if (types[num] == "bool" && mymem.getType(names[num].c_str()) == BOOL)
                    {cout<< num << names[num] << " is ok"<< endl;continue;}
        else if (types[num] == "float" && mymem.getType(names[num].c_str()) == FLOAT)
             {cout<< num << names[num] << " is ok"<< endl;continue;}
        else if (types[num] == "short" && mymem.getType(names[num].c_str()) == SHORT)
             {cout<< num << names[num] << " is ok"<< endl;continue;}
        else if (types[num] == "long" && mymem.getType(names[num].c_str()) == LONG)
             {cout<< num << names[num] << " is ok"<< endl;continue;}
                    else{ cout<< num << " "<< "oooops";exit(1);}
    }
            cout << "oooops";

}

Видно я что то не понимаю. За пределы массива вроде нигде не выхожу. Последний цикл успешно завершается, но вместо последнего вывода в консоли видно Segmentation fault. Голову сломал. Help please.

Segfaut Вариант с push_back и содержимое читаемого файла

4
  • А зачем вам num = num -1;? Я так понял, в num вы храните что-то вроде количества записей, начинаете его с 0, при чтении каждой увеличиваете. Таким образом, прочитав 4 записи, при выходе из цикла num будет как раз равен количеству записей.
    – wirtwelt
    18 апр 2018 в 6:57
  • Вообще в целом подход с resize + [] = не безопасен. Может быть лучше просто читать записи и сразу их складывать через push_back() в вектор? Куча возможных проблем пропадет
    – wirtwelt
    18 апр 2018 в 6:58
  • Скрины приложил. У меня появляется на один элемент больше при чтении файла, по непонятной мне причине. поэтому num-1
    – Uporka
    18 апр 2018 в 8:05
  • И сам segfault выползает в конце функции InitFBD, сразу после цикла, т.к как cout << "ooops" уже не выводится. Это я так понимаю ><
    – Uporka
    18 апр 2018 в 8:20

2 ответа 2

1

Проблема решена. Неправильно передавал объект класса в функцию (Shm mymem).

0

Попробуйте не использовать потенциально опасное обращение к несуществующему элементу вектора через at(), он же оператор []:

void GetNames(vector<string> &names, vector<string> &types) {

if (!FlashHere()) return;

string buf;
ifstream in(NAMES_DIR);

while (!in.eof()) {
    getline(in, buf, '.');
    types.push_back(buf);

    getline(in, buf, ';');
    names.push_back(buf);

    cout<< "|" << names.back() << "|" << types.back() << "|" << endl;
}

in.close();

Самая частая ошибка при работе с вектором - думать, что вектор это просто динамический массив. Это С++, тут само ничего не происходит, и даже если расширение data у вектора спрятано под капотом STL, знать о нем и о его механике работы мы все равно обязаны

9
  • Возникает проблема с лишней ячейкой. Не пойму откуда это берется. Приложил скрины.
    – Uporka
    18 апр 2018 в 8:04
  • Так же пробовал заполнять/выводить именно методом .at(). Он же вроде как сообщает, о том что программа выходит за пределы массива. Только в моем случае результат был такой же.
    – Uporka
    18 апр 2018 в 8:13
  • В любом случае, даже если убрать exit(1) из конца цикла, результат seg fault.
    – Uporka
    18 апр 2018 в 9:02
  • Лишний элемент у вас возникает из-за чтения на конца файла, getline() читает что-то между последним ; и EOF и возвращает это дело, а вы думаете, что это еще один элемент. В любом случае, добавьте побольше отладочного вывода на каждый чих, чтобы посмотреть, что просисходит
    – wirtwelt
    18 апр 2018 в 9:28
  • у меня такое впечатление что как то некоректно работает вывод. (iostream). потому что когда я вырезал совершенно другой кусок кода, появилась другая ошибка, segfault пропал.
    – Uporka
    18 апр 2018 в 10:22

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.