[英]Win32 program crashes on Windows 8 when calling GetSaveFileName() or SHBrowseForFolder()
[英]how to save file with GetSaveFileName in win32?
我写这个代码来获取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。
根本问题在于:
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。
除非你确切知道它的作用以及为什么在你的特定情况下它是必要的,否则永远不要施放。
这一行:
printf("the path is : %s\n", ofn.lpstrFile);
应该使用printf的宽字符版本。
wprintf(L"the path is : %s\n", ofn.lpstrFile);
你错了,这是一个简单的C指针/堆栈问题。
// WRONG:
char szFileName[MAX_PATH] = "";
这会混淆数组和指针,您在堆栈上声明了一个数组,但随后将其内存地址更改为指向数据部分中的空字符串。 换句话说,缓冲区溢出。
// RIGHT:
char szFileName[MAX_PATH];
ZeroMemory(szFileName, MAX_PATH);
这在堆栈上声明了一个字符数组,并将所有元素初始化为null终止符。
希望有所帮助!
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.