Плохо считываю файл или что то в этом роде

@Hustman · Регистрация: 23.05.2019

Здравствуйте. Проблема в том что у меня есть код который считывает из файла данные банкоматов и делает с ними разные вещи типа закидывает в двунаправленный список и другое. Проблема в том что когда я написал код у меня, на моем ноуте и других ноутах с которых я пробовал ошибки не возникает. Но вот человек который проверял говорит что у меня проблема с памятью при считывании файла. Я знаю что трудно копаться в чужом коде такого неумельца как я, но вот прошу посмотреть или скомпилировать у себя код (C++11), что не так в функции которой я считываю или хотя бы сказать в каком направлении двигаться. (Код с нужным файлом прикрепил)

Если что кидаю еще отдельно код функции с которой якобы у меня проблема

p.s Заранее извиняюсь за Польский язык

C++

void readfile(string sciezka,dlistVar *L, int &liczba_bankomatow){
    struct dlistEl *p = (struct dlistEl*)malloc(sizeof(struct dlistEl));
   ifstream plik;
    plik.open(sciezka);
    string linia;
    plik >> liczba_bankomatow;
    p = new dlistEl[liczba_bankomatow];
    plik >> linia;
    string nb,cz,m,spr,b10,b20,b50,b100,b200;
           plik>>linia;
 for(int i=0; i<liczba_bankomatow; i++)
    {
        plik>>linia;
        istringstream ss(linia);
        getline(ss, nb, ';');
        getline(ss, cz, ';');
        getline(ss, m, ';');
        getline(ss, spr, ';');
        getline(ss, b10, ';');
        getline(ss, b20, ';');
        getline(ss, b50, ';');
        getline(ss, b100, ';');
        getline(ss, b200, ';');
    pushforlist(L, nb,cz, m,spr,b10,b20, b50, b100,b200);
 
 
 
    }
    plik.close();
 
}

@zayats80888 · 16.06.2019, 22:51

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

что не так в функции которой я считываю

Зачем 2 и 7 строки?

@Hustman · 16.06.2019, 22:54 **[ТС]**

Забыл убрать когда перерабатывал

Они не нужны

Добавлено через 1 минуту
Забыл убрать когда перерабатывал Они не нужны

@zayats80888 · 16.06.2019, 22:55

так же во всех функция для работы со списком вы используете
struct dlistEl *p = (struct dlistEl*)malloc(sizeof(struct dlistEl)); но malloc не конструирует объект(просто память выделяет) а у вас структура нуждается в конструкторе

@Hustman · 16.06.2019, 23:00 **[ТС]**

Окей, понял

Добавлено через 4 минуты
Так в этом ли есть главная проблема?

@zayats80888 · 16.06.2019, 23:01

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

Так в этом ли есть главная проблема?

Не знаю, я не тестил ваш код, просто пробежался взглядом.

@Hustman · 16.06.2019, 23:03 **[ТС]**

Понял, спасибо

@zayats80888 · 16.06.2019, 23:14

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

а у вас структура нуждается в конструкторе

Hustman, я коряво выразился, имелось ввиду, нужен вызов конструктора(из-за полей типа string), короче замените malloc/free на new/delete

Добавлено через 2 минуты
Еще заметил, что вы память то не освобождаете нигде

@Hustman · 16.06.2019, 23:24 **[ТС]**

Блин действительно, я забыл совсем про это, наверное из-за этого. Спасибо

@zayats80888 4046 / 2295 / 958 Регистрация: 07.02.2019 Сообщений: 5,987
	16.06.2019, 22:51	2
	Сообщение от Hustman что не так в функции которой я считываю Зачем 2 и 7 строки? 1

@Hustman 0 / 0 / 0 Регистрация: 23.05.2019 Сообщений: 7
	16.06.2019, 22:54 [ТС]	3
	Забыл убрать когда перерабатывал Они не нужны Добавлено через 1 минуту Забыл убрать когда перерабатывал Они не нужны 0

@zayats80888 4046 / 2295 / 958 Регистрация: 07.02.2019 Сообщений: 5,987
	16.06.2019, 22:55	4
	так же во всех функция для работы со списком вы используете `struct dlistEl p = (struct dlistEl)malloc(sizeof(struct dlistEl));` но malloc не конструирует объект(просто память выделяет) а у вас структура нуждается в конструкторе 1

@Hustman 0 / 0 / 0 Регистрация: 23.05.2019 Сообщений: 7
	16.06.2019, 23:00 [ТС]	5
	Окей, понял Добавлено через 4 минуты Так в этом ли есть главная проблема? 0

@zayats80888 4046 / 2295 / 958 Регистрация: 07.02.2019 Сообщений: 5,987
	16.06.2019, 23:01	6
	Сообщение от Hustman Так в этом ли есть главная проблема? Не знаю, я не тестил ваш код, просто пробежался взглядом. 0

@Hustman 0 / 0 / 0 Регистрация: 23.05.2019 Сообщений: 7
	16.06.2019, 23:03 [ТС]	7
	Понял, спасибо 0

@zayats80888 4046 / 2295 / 958 Регистрация: 07.02.2019 Сообщений: 5,987
	16.06.2019, 23:14	8
	Сообщение было отмечено Hustman как решение Решение Сообщение от zayats80888 а у вас структура нуждается в конструкторе Hustman, я коряво выразился, имелось ввиду, нужен вызов конструктора(из-за полей типа string), короче замените malloc/free на new/delete Добавлено через 2 минуты Еще заметил, что вы память то не освобождаете нигде 0

@Hustman 0 / 0 / 0 Регистрация: 23.05.2019 Сообщений: 7
	16.06.2019, 23:24 [ТС]	9
	Блин действительно, я забыл совсем про это, наверное из-за этого. Спасибо 0

Опции темы

Плохо считываю файл или что то в этом роде

Решение