Возврат строки из функции

@Алексей_Либанов · Регистрация: 21.02.2015

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

Здравствуйте. Пишу программу для работы с процессами, мне нужно, чтобы в функцию передавался PID, а возвращалось имя, но выводятся краказябры, а точнее "ММ...МММ несколько других символов, не имеющих смысла ММММММММММММММ". Функция рабочая, дело в возврате строки. Вот код:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
char *getProcessNamebyPID(DWORD PID)
{
    HANDLE snapshot = CreateToolhelp32Snapshot(TH32CS_SNAPPROCESS, 0);
    PROCESSENTRY32 pInfo;
    pInfo.dwSize = sizeof(PROCESSENTRY32);
    if (Process32First(snapshot, &pInfo))
    {
        while (Process32Next(snapshot, &pInfo))
        {
            if (PID == pInfo.th32ProcessID)
            {
                CloseHandle(snapshot);
                char *c;
                c = pInfo.szExeFile;
                //std::cout << c << '\n'; тут выводит корректно
                return c; //возвращает краказябры
            }
        }
    }
    CloseHandle(snapshot);
    return NULL;
}

Добавлено через 1 минуту
Как доработать код. Заранее спасибо.

@castaway · 27.10.2016, 21:58

Сообщение от Алексей_Либанов

возвращает краказябры

Возвращает указатель на строку, адрес которой находится в локальной переменной, которая после выхода из этой самой функции "затирается".

@IchimaruGin · 27.10.2016, 21:59

может потому что память для PROCESSENTRY32 pInfo; не выделяется динамически и она удаляется при выходе из метода

Добавлено через 1 минуту
не успел

@Алексей_Либанов · 27.10.2016, 22:05 **[ТС]**

castaway, Вы предлагаете использовать static?

Добавлено через 49 секунд
Это не помогает.

Добавлено через 1 минуту
А нет... помогло. Спасибо за подсказку.

@IchimaruGin · 27.10.2016, 22:06

PROCESSENTRY32 *pInfo = new PROCESSENTRY32; как то так

Добавлено через 21 секунду
ну либо static)

@castaway · 27.10.2016, 22:06

Сообщение от Алексей_Либанов

castaway, Вы предлагаете использовать static?
Добавлено через 49 секунд
Это не помогает.

Я пока ещё ничего не предлагал чтобы что-то заработало, но могу предложить, например std::string.
Алексей_Либанов, static - плохой подход. Но если вам всё-равно, то и мне всё-равно.

@Алексей_Либанов · 27.10.2016, 22:09 **[ТС]**

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

static - плохой подход

Почему?

@castaway · 27.10.2016, 22:15

...

@Алексей_Либанов · 27.10.2016, 22:18 **[ТС]**

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

...

В таком случае как пользоваться std::string для данной цели?

@castaway · 27.10.2016, 22:20

Сообщение от Алексей_Либанов

Почему?

Объяснение зависит от того как вы использовали static. Покажите что вы написали.

@Алексей_Либанов · 27.10.2016, 22:21 **[ТС]**

C++
1
static PROCESSENTRY32 pInfo;

@castaway · 27.10.2016, 22:21

Сообщение от Алексей_Либанов

В таком случае как пользоваться std::string для данной цели?

Троеточие - это я затёр сообщение, а не игнорирование вопроса.

@Алексей_Либанов · 27.10.2016, 22:24 **[ТС]**

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

Троеточие - это я затёр сообщение, а не игнорирование вопроса.

Хорошо.

@IchimaruGin · 27.10.2016, 22:24

так просто динамически выделить память можна

@castaway · 27.10.2016, 22:26

Сообщение от Алексей_Либанов

Почему?

Попробуйте такой псевдокод и поймёте.

C++
1
2
3
4
5
6
7
8
9
    ...
 
    char * a = getProcessNamebyPID( A );
    char * b = getProcessNamebyPID( B );
 
    std::cout << a << std::endl;
    std::cout << b << std::endl;
 
    ...

Тут важна последовательность вызовов.
A и B - разные PID.

@Алексей_Либанов · 27.10.2016, 22:30 **[ТС]**

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

Попробуйте такой псевдокод и поймёте.

Ну... я понял

Как решить данную проблему?

@castaway · 27.10.2016, 22:32

Сообщение от Алексей_Либанов

Ну... я понял. Как решить данную проблему?

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
std::string getProcessNamebyPID(DWORD PID)
{
    std::string result;
    HANDLE snapshot = CreateToolhelp32Snapshot(TH32CS_SNAPPROCESS, 0);
    PROCESSENTRY32 pInfo;
    pInfo.dwSize = sizeof(PROCESSENTRY32);
    if (Process32First(snapshot, &pInfo))
    {
        while (Process32Next(snapshot, &pInfo))
        {
            if (PID == pInfo.th32ProcessID)
            {
                result = pInfo.szExeFile;
                break;
            }
        }
    }
    CloseHandle(snapshot);
    return result;
}

@Алексей_Либанов · 27.10.2016, 22:39 **[ТС]**

castaway, спасибо! Все работает!

Новые блоги и статьи Все статьи Все блоги /
BOINC: 22 года — и всё ещё работает Programma_Boinc 12.03.2026 BOINC: 22 года — и всё ещё работает Дэвид Андерсон написал ретроспективу. Кратко: в 2001 году он ушёл из United Devices, где был CTO, и за несколько месяцев написал ядро BOINC — клиент, сервер,. . .	SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .
Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .	Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .

@IchimaruGin 82 / 82 / 44 Регистрация: 14.07.2013 Сообщений: 410
	27.10.2016, 21:59
	может потому что память для PROCESSENTRY32 pInfo; не выделяется динамически и она удаляется при выходе из метода Добавлено через 1 минуту не успел 0

@Алексей_Либанов 1 / 1 / 0 Регистрация: 21.02.2015 Сообщений: 126
	27.10.2016, 22:05 [ТС]
	castaway, Вы предлагаете использовать static? Добавлено через 49 секунд Это не помогает. Добавлено через 1 минуту А нет... помогло. Спасибо за подсказку. 0

@IchimaruGin 82 / 82 / 44 Регистрация: 14.07.2013 Сообщений: 410
	27.10.2016, 22:06
	PROCESSENTRY32 pInfo = new PROCESSENTRY32; как то так Добавлено через 21 секунду* ну либо static) 0

@castaway 4986 / 3093 / 456 Регистрация: 10.11.2010 Сообщений: 11,170 Записей в блоге: 10
	27.10.2016, 22:15
	... 0

@IchimaruGin 82 / 82 / 44 Регистрация: 14.07.2013 Сообщений: 410
	27.10.2016, 22:24
	так просто динамически выделить память можна 0

@Алексей_Либанов 1 / 1 / 0 Регистрация: 21.02.2015 Сообщений: 126
	27.10.2016, 22:39 [ТС]
	castaway, спасибо! Все работает! 0

Возврат строки из функции

Решение