指针混淆 - c++

Question

我的任务是对 C++ 应用程序进行简单的更改。 不幸的是，我来自 Java 背景，并且遇到了一些指针问题。

有问题的代码从给定目录读取文件列表（使用环境变量设置），并对每个文件执行一些操作。

char * rebuildDir = getenv("REBUILD_DIR");
char * currentFile;
DIR *asciiDir;
struct dirent *ent;

asciiDir = opendir(rebuildDir);
if (asciiDir != NULL)
{
    while ((ent = readdir(asciiDir)) != NULL)
    {
        std::cout << "rebuild sensor, rebuild dir is " << getenv("REBUILD_DIR") << std::endl;
        currentFile = rebuildDir;
        strcat(currentFile, ent->d_name);
        ifstream raw(currentFile);
        while(raw)
        {
            ...snip...
        }
        raw.close();
    }

    closedir(asciiDir);
}

可以看到，目的是将环境变量存储一次，然后将其复制到currentFile，然后将当前文件名连接到currentFile，准备传入ifstream。

问题是

currentFile = rebuildDir;

不会重置为环境变量，因此 strcat 继续使用旧文件名并附加到它，所以：

/home/file1
/home/file2
/home/file3

将执行为

/home/file1
/home/file1/home/file2
/home/file1/home/file2/home/file3

通过循环。 我猜我的指针犯了一个基本错误，但我无法追踪它。

感谢您的帮助，并为这个琐碎的问题道歉。

PS - 如果有明显更好的方法来完成我的任务，请随时指出:)

Answer 1

当前文件指向与rebuilddir 相同的memory，因此您可以就地修改字符串。 您需要复制字符串。 您可以执行以下操作：

char currentFile[MAX_PATH];
snprintf(currentFile, MAX_PATH, "%s%s", rebuildDir, ent->d.name);

Answer 2

“...不会重置为环境变量”。 为什么它会“重置”到任何东西？ 您将指针currentFile指向rebuildDir指向的实际环境变量值。 然后您通过strcat修改指针，即您实质上是在修改（即破坏）您的原始环境变量值。

你不应该那样做。 如果要从环境变量的值构建新名称，则必须将该值复制到一边，然后修改副本，而不是试图破坏原始值。

例如，您可以使用std::string在原始值上创建副本

const char *rebuildDir = getenv("REBUILD_DIR");
...
std::string currentFile = rebuildDir;
currentFile += ent->d_name;
ifstream raw(currentFile.c_str());

Answer 3

int main(int argc, char *argv[])
{
    char* pDir = getenv("REBUILD_DIR");

    if (! pDir)
    {
        cerr << "did not find ENV var\n";
        exit(1);
    }

    string rebuildDir(pDir);

    DIR* asciiDir;

    if ((asciiDir = opendir(rebuildDir.c_str())) != NULL)
    {
        std::cout << "rebuild sensor, rebuild dir is " << rebuildDir << std::endl;

        struct dirent *ent;

        while ((ent = readdir(asciiDir)) != NULL)
        {
            string currentFile(rebuildDir);

            currentFile += '/' + string(ent->d_name);

            //probably want to skip "." and ".." entries...

            /*
            ifstream raw(currentFile);

            while(raw)
            {
                ...snip...
            }

            raw.close();
            */
        }

        closedir(asciiDir);
    }
    else
    {
        cerr << "coult not open dir\n";
        exit(1);
    }

    return 0;
}