如何在win32中使用GetSaveFileName保存文件?

ham*_*med 8 c++ winapi savefiledialog

我写这个代码来获取fileName来保存我的文件:

#include "stdafx.h"
#include <windows.h>


int _tmain(int argc, _TCHAR* argv[])
{            
    OPENFILENAME ofn;

    char szFileName[MAX_PATH] = "";

    ZeroMemory(&ofn, sizeof(ofn));

    ofn.lStructSize = sizeof(ofn); 
    ofn.hwndOwner = NULL;
    ofn.lpstrFilter = (LPCWSTR)L"Text Files (*.txt)\0*.txt\0All Files (*.*)\0*.*\0";
    ofn.lpstrFile = (LPWSTR)szFileName;
    ofn.nMaxFile = MAX_PATH;
    ofn.Flags = OFN_EXPLORER | OFN_FILEMUSTEXIST | OFN_HIDEREADONLY;
    ofn.lpstrDefExt = (LPCWSTR)L"txt";

    GetSaveFileName(&ofn);
    printf("the path is : %s\n", ofn.lpstrFile);
    getchar();
    return 0;
}
Run Code Online (Sandbox Code Playgroud)

但输出是:

 the path is : H 
Run Code Online (Sandbox Code Playgroud)

为什么?难道我做错了什么 ?
我在Windows 7上使用Visual Studio 2008.

Adr*_*thy 9

根本问题在于以下几行:

char szFileName[MAX_PATH] = "";
...
ofn.lpstrFile = (LPWSTR)szFileName;
ofn.nMaxFile = MAX_PATH;
Run Code Online (Sandbox Code Playgroud)

这会创建一个MAX_PATH字符的缓冲区,但它会告诉GetSaveFileName函数它是MAX_PATH 字符的缓冲区.当有人选择长路径名时,这很可能会崩溃(或者无声地践踏内存).

赠品是演员.不要欺骗编译器或库.他们不喜欢这样,他们最终会报复.用这个替换这些行:

WCHAR szFileName[MAX_PATH] = L"";
...
ofn.lpstrFile = szFileName;  // no cast needed
ofn.nMaxFile = MAX_PATH;
Run Code Online (Sandbox Code Playgroud)

现在,所选文件名将作为宽字符串返回.Tony The Lion的答案是正确的,因为你需要使用wprintf而不是printf打印宽字符串:

wprintf(L"the path is : %s\n", ofn.lpstrFile);  // though I'd use szFileName at this point
Run Code Online (Sandbox Code Playgroud)

如果您需要8位字符而不是宽字符的字符串,则可以使用WideCharToMultiByte.但我总是坚持使用宽字符API.

除非你确切知道它的作用以及为什么在你的特定情况下它是必要的,否则永远不要施放.


Ton*_*ion 8

这一行:

printf("the path is : %s\n", ofn.lpstrFile);
Run Code Online (Sandbox Code Playgroud)

应该使用printf的宽字符版本.

wprintf(L"the path is : %s\n", ofn.lpstrFile);
Run Code Online (Sandbox Code Playgroud)

  • -1此代码仍然很危险,因为缓冲区太小. (2认同)