Выводом данных из динамической структуры между функциями

@ziqp · Регистрация: 03.09.2015

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

Некорректно выводится информация из динамической структуры. Может затирается информация после основной функции?

C++

struct procStruct     //Структура
{
    DWORD pid = 0;       
    char *filename; 
    char *filepath;   
};
 
void somefunc(procStruct procList, int procCount)
{
char filename[MAX_PATH];
//открываю хэндл hProcess
GetModuleFileNameEx(hProcess, NULL, filename, MAX_PATH);//пролучаю путь до файла
procList[procCount].filepath = filename;
}
 
int main()
{
procStruct *procList = new procStruct[cProcesses]; //создаю динамическую структуру
somefunc(procList, 0);
cout << procList[0].filepath;
}

В вывод получаются каракули нечитаемые.
Но если выводить через cout внутри somefunc (тот же procList[0].filepath), то все выводит корректно. Что не так?

@schdub · 08.03.2018, 18:34

ziqp, в строке 13, в filepath вы копируете указатель на текстовую строку, которая существует во фрейме стека somefunc(). После того как мы вышли из этой функции никто вам не гарантирует, что память, где раньше была переменная filename, и на которую указывает procList[N].filepath, будет оставаться в неприкосновенности. Проблему можно решить несколькими путями:

1) использовать std::string в структуре (имхо лучшее решение: 21 век на дворе);
2) изменить procStruct::filepath так что он будет не указателем, а тесктвовой строкой размером MAX_PATH, а в somefunc() уже обойтись без стековой переменной filename;
3) в строке 13 использовать strdup(), но затем не забыть удалить динамическую память, выделенную данной функцией.

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

void somefunc(procStruct procList, int procCount)

в первом аргументе опечатка? наверное имелось ввиду так:

C

1	void somefunc(procStruct * procList, int procCount)

@ziqp · 08.03.2018, 18:57 **[ТС]**

Спасибо большое! string исправил положение. Только вот почему копировался указатель? Из-за *? Я так понял, нужно было делать фиксированный массив (если забыть про string)?

@schdub · 08.03.2018, 21:58

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

string исправил положение

Покажите как исправили.

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

Только вот почему копировался указатель

Потому что это наследие C.

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

Я так понял, нужно было делать фиксированный массив

Да.

@ziqp · 08.03.2018, 22:21 **[ТС]**

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

Покажите как исправили.

Вместо char *filename поставил string filename.
Char легко присваивается к string, а обратно через .c_str().
По крайней мере, теперь работает вне функции

@ziqp 129 / 65 / 16 Регистрация: 03.09.2015 Сообщений: 832
	08.03.2018, 18:57 [ТС]	3
	Спасибо большое! string исправил положение. Только вот почему копировался указатель? Из-за *? Я так понял, нужно было делать фиксированный массив (если забыть про string)? 0

@schdub 3071 / 1409 / 425 Регистрация: 19.01.2009 Сообщений: 3,880
	08.03.2018, 21:58	4
	Сообщение от ziqp string исправил положение Покажите как исправили. Сообщение от ziqp Только вот почему копировался указатель Потому что это наследие C. Сообщение от ziqp Я так понял, нужно было делать фиксированный массив Да. 0

@ziqp 129 / 65 / 16 Регистрация: 03.09.2015 Сообщений: 832
	08.03.2018, 22:21 [ТС]	5
	Сообщение от schdub Покажите как исправили. Вместо char *filename поставил string filename. Char легко присваивается к string, а обратно через .c_str(). По крайней мере, теперь работает вне функции 0