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

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

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

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

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

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

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
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 **[ТС]**

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

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .

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

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

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

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

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

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

Решение