10

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

4

3 回答 3

10

这一行：

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

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

wprintf(L"the path is : %s\n", ofn.lpstrFile);
于 2012-05-04T13:08:26.863 回答
10

根本问题在于以下几行：

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。

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

于 2012-05-04T17:05:06.777 回答
-1

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

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

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

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

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

希望有帮助！

于 2015-03-03T21:33:54.923 回答