在 C++ 中使用 cstrings 构建路径的正确方法

Question

我需要建立一个文件的路径。我有以下类方法：

void Directory::scanDirectory(char *directory) {
    DIR *dirp;
    struct dirent *entry;
    char path[1];

    if(dirp = opendir(directory)) {
        while(entry = readdir(dirp)) {
            if (entry->d_name[0] != '.') {
                strcpy(path, directory);
                strcat(path, "/");
                strcat(path, entry->d_name);
                if (entry->d_type == 8) {
                    // Files
                } else if (entry->d_type == 4) {
                    //scanDirectory(path);
                }
                printf("Name: %s, Type: %d\n", entry->d_name, entry->d_type);
            }
        }
        closedir(dirp);
    }
}

我需要通过连接目录和 entry->d_name 来构建文件路径。当我尝试运行这段代码时，它出现段错误。据我所知，在我构建路径的地方出现了段错误。有更好的方法吗？

Answer 1

您仅为路径分配一个字节 (char path[1])。您需要分配足够的空间来实际容纳您正在创建的整个路径。给定 C++ 标记，明显的可能性是使用 std::string，并且在将所有部分组合成完整路径后，使用其 c_str() 成员函数来访问 C 风格字符串的内容。

Answer 2

为什么不使用 Boost.Filesystem ？

Answer 3

缓冲区path需要有足够的空间来容纳整个路径。现在它只有一个字符的空间。尝试把它做得更大。 strcat 本身不分配空间。您必须手动管理该内存。

至于更好的方法，您可能需要考虑使用字符串。您无需担心内存问题，并且可以使用 + 运算符进行连接。

Answer 4

将 char path[1]; 更改为：

char path[512]; //or whatever value you like.

在您的代码中，path 仅为 1 个字符和 \0 分配了空间。显然你需要一个更大的，据我所知，在unix中，目录名最多可以有255个字符，所以我认为512就足够了。

Answer 5

使用strcpy时要小心。它不进行边界检查，因此即使 path 只是 char[1]，它也会尝试将所有 directory 复制到其中。这可能是你的段错误。

在如何构建字符串方面，您有很多选择。这是一篇关于 C++ 字符串连接效率的长篇文章：

Efficient string concatenation in C++

如果您使用 C++，是否有任何原因不能仅将内置 string 库与 + 运算符一起使用？例如：

string path;
//...
path += directory;
path += "/";
path += entry->d_name;
//etc.

使用 string 类可能效率稍低，但它会有额外的好处，可以帮助您避免缓冲区溢出问题和内存异常，例如您遇到的分段错误（我不是说string 将避免所有这些，但它会让你的生活更轻松）。

之前还有另一篇关于如何在 C++ 中构建目录字符串的文章：

c++ 如何从路径创建目录