Segfault при прасинге файла

@JustOneProblem · Регистрация: 19.08.2016

Author24 — интернет-сервис помощи студентам

С++ знаю плоховато. Решил написать себе генератор to-do list`a, вроде даже начало было неплохим, но потом обнаружил неприятную ошибку. Суть программы в том, что на основе секций в текстовом файле вида:
{
title строка
level натуральное число
depends строка
description строка
urgency число от 1 до 20
importance число от 1 до 20
}
программа формировала бы список дел и записывала его в файл. Заступорился я на этапе парсинга файла с секциями: если секция в файле одна, то всё парсится в переменные нормально, но если больше, то происходит ошибка сегментации.
Код программы (незавершённый):

C++

#include <iostream>
#include <fstream>
#include <cstring>
 
#define DAILY "todo.daily"
 
using namespace std;
ofstream fout;
ifstream fin;
 
struct item {
    string title;
    int level;
    string depends;
    string description;
    int urgency;
    int importance;
};
 
void get (item *task) {
    fin.open (DAILY);
    char *tmp = new char;
    int i = 0;
    bool section = false;
    while (fin >> tmp) {
        if (!(strcmp("{",tmp)))
            section = true;
        else if (!(strcmp("title",tmp)) && section == true) {
            fin.get();
            getline(fin,task[i].title);
        }    
        else if (!(strcmp("level",tmp)) && section == true) {
            fin >> task[i].level;
        }    
        else if (!(strcmp("depends",tmp)) && section == true) {
            fin.get();
            getline(fin,task[i].depends);
        }    
        else if (!(strcmp("description",tmp)) && section == true) {
            fin.get();
            getline(fin,task[i].description);
        }    
        else if (!(strcmp("urgency",tmp)) && section == true) {
            fin >> task[i].urgency;
            if (task[i].urgency > 20)
                task[i].urgency = 20;
            else if (task[i].urgency < 1)
                task[i].urgency = 1;
        }    
        else if (!(strcmp("importance",tmp)) && section == true) {
            fin >> task[i].importance;
            if (task[i].importance > 20)
                task[i].importance = 20;
            else if (task[i].importance < 1)
                task[i].importance = 1;
        }    
        else if (!(strcmp("}",tmp)) && section == true) {
            section = false;
            i++;
        }  
    }
    //delete tmp;
}
 
int main (int argc, char **argv) {
    item *task = new item;
    get (task);
   /*
    for (int i(0); i < 1; i++) {
        cout << task[i].title << endl;
        cout << task[i].level << endl;
        cout << task[i].depends << endl;
        cout << task[i].description << endl;
        cout << task[i].urgency << endl;
        cout << task[i].importance << endl;
    }
    */
    //delete task;
    return 0;
}

Если между 25 и 26 строкой добавить cout << tmp << endl;, то при одной секции в todo.daily вывод команды будет:
{
title
level
depends
description
urgency
importance
}
А если увеличить количество секций до 2 или больше, то вывод будет таким:
{
title
level
depends
description
urgency
importance
}
{
title
Ошибка сегментирования (core dumped)
Т. е. сегфолт в функции get во время обработки заголовка второй секции. Я сам не особо понимаю, в чём может быть проблема, прошу объяснить, что происходит, и как это пофиксить.

@DrOffset · 19.08.2016, 17:48

Сообщение от JustOneProblem

char *tmp = new char;

Сколько тут памяти выделилось по-твоему?

@JustOneProblem · 19.08.2016, 17:55 **[ТС]**

Не знаю...

@DrOffset · 19.08.2016, 17:58

Сообщение от JustOneProblem

Не знаю...

1 байт.
А пытаешься ты читать в него целую строку.
В итоге получаешь ошибку времени исполнения.
Используй std::string. Он сам следит за перераспределением памяти, и обеспечит чтение строки произвольного размера.

@JustOneProblem · 19.08.2016, 18:12 **[ТС]**

Сообщение от DrOffset

1 байт.
А пытаешься ты читать в него целую строку.
В итоге получаешь ошибку времени исполнения.
Используй std::string. Он сам следит за перераспределением памяти, и обеспечит чтение строки произвольного размера.

Как понять "читать строку в него"? Почему первый цикл проходит нормально? Пробовал делать char *tmp = new char [число], ничего не изменяется. Если со string делать, то что использовать вместо strcmp("text",tmp))?

Добавлено через 9 минут
Сегфолт происходит даже если tmp -- это просто массив из N символов.

@DrOffset · 19.08.2016, 18:19

Сообщение от JustOneProblem

Как понять "читать строку в него"?

Да очень просто, у тебя выделен один байт.
Оператор >> потока принимает указатель на этот один байт и считает, что это указатель на начало буфера, и пытается записать в него строку. Но т.к. реально в нашем "буфере" только один байт, он пишет за пределы выделенной нам памяти. Что в конечном итоге приводит к падению.

Сообщение от JustOneProblem

Почему первый цикл проходит нормально?

Везет.

Сообщение от JustOneProblem

Сегфолт происходит даже если tmp -- это просто массив из N символов.

Правильно, потому что у тебя точно такая же проблема в main.
Там ты опять выделил память только для одного объекта item. А внутри функции get пытаешься работать с указателем так, как будто бы у тебя массив. В итоге опять тот же сценарий. Пишем за пределы выделенной памяти - получаем сегфолт.

Сообщение от JustOneProblem

Если со string делать, то что использовать вместо strcmp("text",tmp))?

C++

1	else if ("title" == tmp && section == true) {

Ну и т.д.

@JustOneProblem · 19.08.2016, 18:36 **[ТС]**

Ладно, понял. А как тогда сделать, чтобы элементы массива структур данных создавались по мере надобности (каждый инкремент переменной i), а не сразу в определённом количестве при объявлении item *task = new item[N];?

@DrOffset · 19.08.2016, 18:56

Сообщение от JustOneProblem

А как тогда сделать, чтобы элементы массива структур данных создавались по мере надобности

Посмотри std::vector или std::list.

Добавлено через 3 минуты
JustOneProblem,

Кликните здесь для просмотра всего текста

C++

void get (std::vector<item> & tasks) {
    fin.open (DAILY);
    string tmp;
    bool section = false;
 
    item current;
    while (fin >> tmp) {
        if("{" == tmp) {
            current = item();
            section = true;
        }
        else if ("title" == tmp && section) {
            fin.get();
            getline(fin, current.title);
        }
        else if ("level" == tmp && section) {
            fin >> current.level;
        }
        else if ("depends" == tmp && section) {
            fin.get();
            getline(fin, current.depends);
        }
        else if ("description" == tmp && section) {
            fin.get();
            getline(fin, current.description);
        }
        else if ("urgency" == tmp && section) {
            fin >> current.urgency;
            if (current.urgency > 20)
                current.urgency = 20;
            else if (current.urgency < 1)
                current.urgency = 1;
        }
        else if ("importance" == tmp && section) {
            fin >> current.importance;
            if (current.importance > 20)
                current.importance = 20;
            else if (current.importance < 1)
                current.importance = 1;
        }
        else if ("}" == tmp && section) {
            section = false;
            tasks.push_back(current);
        }
    }
}

@JustOneProblem · 20.08.2016, 23:44 **[ТС]**

Сообщение от DrOffset

Посмотри std::vector или std::list.

Добавлено через 3 минуты
JustOneProblem,

Кликните здесь для просмотра всего текста

C++

void get (std::vector<item> & tasks) {
    fin.open (DAILY);
    string tmp;
    bool section = false;
 
    item current;
    while (fin >> tmp) {
        if("{" == tmp) {
            current = item();
            section = true;
        }
        else if ("title" == tmp && section) {
            fin.get();
            getline(fin, current.title);
        }
        else if ("level" == tmp && section) {
            fin >> current.level;
        }
        else if ("depends" == tmp && section) {
            fin.get();
            getline(fin, current.depends);
        }
        else if ("description" == tmp && section) {
            fin.get();
            getline(fin, current.description);
        }
        else if ("urgency" == tmp && section) {
            fin >> current.urgency;
            if (current.urgency > 20)
                current.urgency = 20;
            else if (current.urgency < 1)
                current.urgency = 1;
        }
        else if ("importance" == tmp && section) {
            fin >> current.importance;
            if (current.importance > 20)
                current.importance = 20;
            else if (current.importance < 1)
                current.importance = 1;
        }
        else if ("}" == tmp && section) {
            section = false;
            tasks.push_back(current);
        }
    }
}

А что означает строчка current = item();, и можно ли обойтись без создания ещё одного элемента структуры (current)?

Добавлено через 13 минут
Ах, да почему & tasks, а не просто tasks?

@DrOffset · 21.08.2016, 00:20

Сообщение от JustOneProblem

можно ли обойтись без создания ещё одного элемента структуры (current)?

Можно. Придется немного менять алгоритм разбора.

Сообщение от JustOneProblem

А что означает строчка current = item();

Это сброс значений полей, нужно в случае, если у нас i-я итерация с неполностью заполненным полями - чтобы не записались значения предыдущей записи.

Сообщение от JustOneProblem

Ах, да почему & tasks, а не просто tasks?

Ну мы вектор внутри функции заполняем же.

@DrOffset 19409 / 10028 / 2443 Регистрация: 30.01.2014 Сообщений: 17,678
	21.08.2016, 00:20	10
	Сообщение от JustOneProblem можно ли обойтись без создания ещё одного элемента структуры (current)? Можно. Придется немного менять алгоритм разбора. Сообщение от JustOneProblem А что означает строчка current = item(); Это сброс значений полей, нужно в случае, если у нас i-я итерация с неполностью заполненным полями - чтобы не записались значения предыдущей записи. Сообщение от JustOneProblem Ах, да почему & tasks, а не просто tasks? Ну мы вектор внутри функции заполняем же. 1

@DrOffset 19409 / 10028 / 2443 Регистрация: 30.01.2014 Сообщений: 17,678
	19.08.2016, 17:48	2
	Сообщение от JustOneProblem char *tmp = new char; Сколько тут памяти выделилось по-твоему? 0

@JustOneProblem 0 / 0 / 0 Регистрация: 19.08.2016 Сообщений: 23
	19.08.2016, 17:55 [ТС]	3
	Не знаю... 0

@DrOffset 19409 / 10028 / 2443 Регистрация: 30.01.2014 Сообщений: 17,678
	19.08.2016, 17:58	4
	Сообщение от JustOneProblem Не знаю... 1 байт. А пытаешься ты читать в него целую строку. В итоге получаешь ошибку времени исполнения. Используй std::string. Он сам следит за перераспределением памяти, и обеспечит чтение строки произвольного размера. 0

@JustOneProblem 0 / 0 / 0 Регистрация: 19.08.2016 Сообщений: 23
	19.08.2016, 18:12 [ТС]	5
	Сообщение от DrOffset 1 байт. А пытаешься ты читать в него целую строку. В итоге получаешь ошибку времени исполнения. Используй std::string. Он сам следит за перераспределением памяти, и обеспечит чтение строки произвольного размера. Как понять "читать строку в него"? Почему первый цикл проходит нормально? Пробовал делать `char tmp = new char [число]`, ничего не изменяется. Если со string делать, то что использовать вместо `strcmp("text",tmp))`? Добавлено через 9 минут* Сегфолт происходит даже если tmp -- это просто массив из N символов. 0

@JustOneProblem 0 / 0 / 0 Регистрация: 19.08.2016 Сообщений: 23
	19.08.2016, 18:36 [ТС]	7
	Ладно, понял. А как тогда сделать, чтобы элементы массива структур данных создавались по мере надобности (каждый инкремент переменной i), а не сразу в определённом количестве при объявлении `item *task = new item[N];`? 0