C/C++でのポインタ（char配列）の比較

Question

ポインタによるchar配列の比較に問題があります。私は文字列ライブラリとiostreamの両方を使わずに作業しています。そのようにしたいと思います。

char *GetCurrentPath() 
{ 
char buffer[MAX_PATH]; 
if (!GetModuleFileNameA(NULL, buffer, MAX_PATH)) { 
    printf("GetModuleFileNameA failed, error: %d\n", GetLastError()); 
} 
return (buffer); 


} 
char *GetInstallPath() 
{ 
char buffer[MAX_PATH]; 
if (!SHGetSpecialFolderPathA(NULL, buffer, CSIDL_APPDATA, FALSE)) { 
    printf("SHGetSpecialFolderPathA failed, error: %d\n", GetLastError()); 
} 
strcat(buffer, "\\service.exe"); 

    return (buffer); 
} 



char *InstallPath = GetInstallPath(); 
char *CurrentPath = GetCurrentPath(); 

if (InstallPath == CurrentPath)...... 

ifステートメントは即時クラッシュを引き起こします。同じことがstrcompになります。

提案？

Answer 1

あなたが現在行っていることは、未定義の動作です。 2つの関数で使用しているバッファは、それらの関数にローカルに定義され、関数が終了するとスコープの外に出て、ランダムなスタックアドレスへのポインタを与えます。

は交換してください：：char *buffer = new char[MAX_PATH]

や機能にメインから割り当てられたバッファを渡します：付きchar buffer[MAX_PATH];

を

char *InstallPath = new char[MAX_PATH]; 
GetInstallPath(InstallPath); 

をあなたは関数でバッファを割り当てるためにどちらかの必要

getパス関数を変更してください：

char *GetInstallPath(char *buffer) 

どちらの場合でも、メモリを解放するためにプログラムを終了する前に、ポインタを削除する必要があります。

さらに、2つの変数を比較しようとすると、文字列の内容ではなくポインタのアドレスが比較されます。あなたはstrcmp()かその機能のファミリーの何かを使用する必要があります。

Answer 2

あなたの関数は、内部で宣言された配列（char *）を返します。あなたのポインタは関数の外では未定義の値を持っています。あなたはそれをすることはできません。

あなたが動的に割り当てることができる：機能の実行が終了した後

char *buffer = malloc(MAX_PATH*sizeof(char)); 

Answer 3

両方のバッファがスタックに割り当てられているので、彼らは解放されます。したがって、ポインタ関数の戻り値は無効になります。

これを修正するには、char * buffer = new char [MAX_PATH];を使用して、ヒープにbuffer []を割り当てる必要があります。

ただし、手動でメモリを解放する必要があります。