如何在win32中使用GetSave文件名保存文件

Question

我写这段代码来获取文件名来保存我的文件：

#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)*.txtAll Files (*.*)*.*";
    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 : %sn", ofn.lpstrFile);
    getchar();
    return 0;
}

但输出是：

 the path is : H 

为什么？我做错了什么吗？
我在 Windows 7 上使用 Visual Studio 2008。

Answer 1

这一行：

printf("the path is : %sn", ofn.lpstrFile);

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

wprintf(L"the path is : %sn", ofn.lpstrFile);

Answer 2

根本问题出在这些行中：

char szFileName[MAX_PATH] = "";
...
ofn.lpstrFile = (LPWSTR)szFileName;
ofn.nMaxFile = MAX_PATH;

这将创建包含 MAX_PATH 个字符的缓冲区，但它告诉 GetSaveFileName 函数它是MAX_PATH宽字符的缓冲区。 当有人选择长路径名时，这可能会崩溃（或静默地践踏内存）。

赠品是演员阵容。 不要对编译器或库撒谎。 他们不喜欢这样，他们最终总会报仇。 将这些行替换为：

WCHAR szFileName[MAX_PATH] = L"";
...
ofn.lpstrFile = szFileName;  // no cast needed
ofn.nMaxFile = MAX_PATH;

现在，所选文件名将作为宽字符字符串返回。 Tony The Lion的回答是正确的，因为您需要使用wprintf而不是printf来打印宽字符的字符串：

wprintf(L"the path is : %sn", ofn.lpstrFile);  // though I'd use szFileName at this point

如果需要 8 位字符而不是宽字符的字符串，则可以使用 WideCharToMultiByte。 但我一般会坚持使用宽字符 API。

除非您确切地知道它的作用以及为什么在您的特定情况下需要它，否则永远不要投射。

Answer 3

你们都错了，这是一个简单的 C 指针/堆栈问题。

// WRONG:
char szFileName[MAX_PATH] = "";

这会混淆数组和指针，您在堆栈上声明一个数组，但随后将其内存地址更改为指向数据部分中的空字符串。 换句话说，缓冲区溢出。

// RIGHT:
char szFileName[MAX_PATH];
ZeroMemory(szFileName, MAX_PATH);

这将在堆栈上声明一个字符数组，并将所有元素初始化为 null 终止符。

希望对您有所帮助！