如何正确的返回字符串指针？

hahacker

1. char * GetFilePath(DWORD dwProcessId)
   
2. {
   
3.         //DWORD dwProcessId = 3344;
   
4.         HANDLE hProcess = NULL;
   
5.         BOOL bSuccess = FALSE;
   
6.         hProcess = OpenProcess( PROCESS_QUERY_INFORMATION | PROCESS_VM_READ , FALSE, dwProcessId );
   
7.         char szPath[MAX_PATH + 1] = {0};
   
8.         HMODULE hMod = NULL;
   
9.         DWORD cbNeeded = 0;
   
10.         if( EnumProcessModules( hProcess, &hMod, sizeof( hMod ), &cbNeeded ) )
    
11.         {
    
12.                 GetModuleFileNameEx( hProcess, hMod, szPath, MAX_PATH );
    
13.         }
    
14.         return szPath;
    
15. }

复制代码

arning C4172: 返回局部变量或临时变量的地址

我知道这样肯定是不行的，是不能直接返回局部变量的，但是如果我在不添加参数的情况下，要如何修改，才是正确的。
C++越学越糊涂。

Debug

char* szPath = new char[MAX_PATH + 1];
memset(szPath, 0, MAX_PATH + 1);

小鱼儿

直接return std::wstring(szPath)，返回局部对象就可以了，c++ 建议不要随便用裸指针

68ui

char szPath[MAX_PATH + 1]   // 这是栈内存，由编译器释放，也就是离开了函数编译器就已经释放了，返回的是“野指针”；

解决方案一，返回类对象，由类对象管理内存：

1. std::string GetFilePath(DWORD dwProcessId)
   
2. {
   
3.         //DWORD dwProcessId = 3344;
   
4.         HANDLE hProcess = NULL;
   
5.         BOOL bSuccess = FALSE;
   
6.         hProcess = OpenProcess( PROCESS_QUERY_INFORMATION | PROCESS_VM_READ , FALSE, dwProcessId );
   
7.         char szPath[MAX_PATH + 1] = {0};
   
8.         HMODULE hMod = NULL;
   
9.         DWORD cbNeeded = 0;
   
10.         if( EnumProcessModules( hProcess, &hMod, sizeof( hMod ), &cbNeeded ) )
    
11.         {
    
12.                 GetModuleFileNameEx( hProcess, hMod, szPath, MAX_PATH );
    
13.         }
    
14.         return std::string(szPath);
    
15. }

复制代码

解决方案二，返回堆内存，由程序员释放内存：

1. char * GetFilePath(DWORD dwProcessId)
   
2. {
   
3.         //DWORD dwProcessId = 3344;
   
4.         HANDLE hProcess = NULL;
   
5.         BOOL bSuccess = FALSE;
   
6.         hProcess = OpenProcess( PROCESS_QUERY_INFORMATION | PROCESS_VM_READ , FALSE, dwProcessId );
   
7.         //char szPath[MAX_PATH + 1] = {0};
   
8.         char *szPath = new char[MAX_PATH + 1];
   
9.         HMODULE hMod = NULL;
   
10.         DWORD cbNeeded = 0;
    
11.         if( EnumProcessModules( hProcess, &hMod, sizeof( hMod ), &cbNeeded ) )
    
12.         {
    
13.                 GetModuleFileNameEx( hProcess, hMod, szPath, MAX_PATH );
    
14.         }
    
15.         return szPath;
    
16. }
    
17. 
    
18. /*
    
19. * 使用完后必须调用delete []szPath;
    
20. *
    
21. * char* path = GetFilePath(xx);
    
22. * delete []path;
    
23. */

复制代码

qq06314488

直接返回string对象就好了，也可以考虑智能指针，或者如果是线程安全的，可以考虑全局或者静态，当然还是参数返回最简单方便

hahacker

68ui 发表于 2019-8-3 16:14
char szPath[MAX_PATH + 1]   // 这是栈内存，由编译器释放，也就是离开了函数编译器就已经释放了，返回的 ...

弱弱的问题下

这个到底应该写成delete []path;还是应该写成delete[] path，如果两个都可能，那么推荐哪一种？

就是说，这个[]到底应该和谁组合。

smmloveyou

如果想返回指针，那你指针对象不要是函数体内定义的局部变量。因为函数结束，函数的所有局部变量都会被释放，你返回的指针完全没意义。要么直接返回字符串。要么把需要返回的字符串指针，定义在函数体外

68ui

hahacker 发表于 2019-8-15 10:37
弱弱的问题下

这个到底应该写成delete []path;还是应该写成delete[] path，如果两个都可能，那么推荐 ...

作用是一样的，没有区别，代码风格不一样而已。