3

我有一个非常简单的函数，它应该读取一个 txt 文件并通过一个字符串类型的向量返回所有行。我已经使用调试器进行了很多次，我唯一注意到的是“fileName”的值在 ifstream 声明的行上发生了变化。我究竟做错了什么？？谢谢。

vector<string> readFile(char* fileName)
{
    vector<string> fileLines;
    fileLines.clear();
    string line;
    ifstream myfile (fileName);
    if (myfile.is_open())
    {
        while ( myfile.good() )
        {
            getline (myfile,line);
            fileLines.push_back(line);
        }
        myfile.close();
    }
    return fileLines;
}

//////////////////////////////

我有这个函数应该返回当前目录中所有 txt 文件的文件名。

   vector<char*> getFileList()
    {
        vector<char*> fileNames;
        fileNames.clear();
        WIN32_FIND_DATA FindFileData;
        HANDLE hFind = INVALID_HANDLE_VALUE;
        char currentPath[_MAX_PATH];
        getCurrentPath(currentPath);
        strncat(currentPath, "\\*", 3);
        if (hFind = FindFirstFile(currentPath, &FindFileData))
        {

            string fileExtension = getExt(FindFileData.cFileName);
            if (fileExtension == "txt" || fileExtension == "TXT")
            {
                fileNames.push_back(FindFileData.cFileName);
            }
            while(FindNextFile(hFind, &FindFileData) != 0)
            {
                string fileExtension = getExt(FindFileData.cFileName);
                if (fileExtension == "txt" || fileExtension == "TXT")
                    fileNames.push_back(FindFileData.cFileName);
            }
        }
        return fileNames;
    }

这就是函数的调用方式：

vector<char*> inputFileList = getFileList();
if (inputFileList.size() > 0)
{
    for (int a=0; a<inputFileList.size(); a++)
    {
        fileLines = readFile(inputFileList[a]);
    }
}
4

1 回答 1

5

我敢打赌，你会从一个函数中获取你的文件名，该函数返回一个指向局部变量的指针，例如：

char * getFilename() {
 char Filename[100] = "/foo";
 return Filename;
}

以上无效 -Filename内容在堆栈上，getFilename()返回时将失效。vector<string> fileLines;将位于同一内存中，因此fileName当您跨过vector构造函数时，指向的数据会发生变化。

编辑：参见例如C++ 编译器警告 - 返回局部变量以获取更多信息

于 2011-04-12T11:30:25.997 回答