Question

我编写此代码以获取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;
}

但输出是：

 the path is : H

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

Answer 1

根本问题在于：

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

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

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

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。

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

Answer 2

这一行：

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

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

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

Answer 3

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

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

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

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

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

希望有所帮助！

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

3 个答案: