如何在win32中使用GetSaveFileName保存文件？答案

【问题标题】：how to save file with GetSaveFileName in win32?如何在win32中使用GetSaveFileName保存文件？
【发布时间】：2012-05-14 00:37:38
【问题描述】：

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

#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;
}

但输出是：

 the path is : H

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

【问题讨论】：

+1 获取完整示例和精心设计的问题。

标签： c++ winapi savefiledialog

【解决方案1】：

这一行：

printf("the path is : %s\n", ofn.lpstrFile);

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

wprintf(L"the path is : %s\n", ofn.lpstrFile);

【讨论】：

由于代码的其余部分，包括lpstrFile 值，都是基于_TCHAR，你应该使用_tprintf 代替：_tprintf(_T("the path is : %s\n"), ofn.lpstrFile);
-1 这段代码仍然很危险，因为缓冲区太小了。

【解决方案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 : %s\n", ofn.lpstrFile);  // though I'd use szFileName at this point

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

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

【讨论】：

【解决方案3】：

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

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

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

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

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

希望有帮助！

【讨论】：

数组初始值设定项右侧的文字字符串具有特殊含义。声明char szFileName[MAX_PATH] = ""； not 创建一个指针。它初始化szFileName[0] to '\0'，即空终止符。