指针失败 (C++)

Question

可能的重复：
c++ 警告：局部变量的地址

char* strdup(const char* s)
{
    char dup[strlen(s)];
    for (int i = 0; i<strlen(s); i++)
    {
        dup[i]=s[i];
    }
    return dup;
}

该函数应该保存新数组已向后读取并加上另一个插槽。当我编译它时，出现错误“警告：局部变量'dup'的地址返回”，当我运行程序时，它返回内存地址。

Answer 1

您的定义指定了一个 char 数组指针作为其返回类型，但您在函数内初始化了一个 char 数组并尝试返回它。试试这个：

char* strdup(const char* s)
{
    char *dup = new char[strlen(s)];
    for (int i = 0; i<strlen(s); i++)
    {
        dup[i]=s[i];
    }
    return dup;

}

Answer 2

问题是你从来没有在堆上分配 dup ，所以当你退出栈帧时， dup 会自动和栈帧一起被删除。这意味着不可能有对 dup 的有效引用，因为一旦退出该函数，它就不再存在。

Answer 3

这应该有效：

char* strdup(const char* s)
{
    char* dup = new char[strlen(s)];
    for (int i = 0; i<strlen(s); i++)
    {
        dup[i]=s[i];
    }
    return dup;
}

编辑：完成后，不要忘记使用“删除”来释放内存；）

Answer 4

你不能返回 dup[] ，因为它是一个局部变量，在函数外部无效（好吧，它指向的内存将不再有效）。您必须调用类似 malloc() 的方法，它在堆上分配内存（所有应用程序都可见的空间）

Answer 5

char* strdup(const char* s)
{
    char dup[strlen(s)]; // defines *local* variable on *stack*
    for (int i = 0; i<strlen(s); i++)
    {
        dup[i]=s[i];
    }
    return dup; // returning dup[0] = dup address
}

您正在返回在堆栈上创建的局部变量的地址。当您从函数返回时，堆栈将倒带并且您的dup变量消失。

Answer 6

该行在

char dup[strlen(s)];

C++ 中不起作用。数组需要在编译时指定一个常量大小； strlen(s) 是一个变量。

就您的实际警告而言，向调用者返回指向局部变量的指针是一种不好的做法；由于局部变量（在本例中为数组 dup）是在堆栈上分配的，因此当函数返回时，它会被释放，因此返回的指针可能无效。编译器旨在捕获此类错误并标记警告，指出这可能是潜在的问题来源。

Answer 7

dup 变量是一个 char 数组，分配在堆栈上，而不是堆上（通过>new 或malloc）。一旦堆栈帧离开（即：函数离开），这就是未定义的内存，很快就会被其他东西覆盖。

您需要将dup转换为char *并使用new或malloc来分配必要的内存。

Answer 8

char dup[strlen(s)] 定义一个本地堆栈变量；当函数结束时，this 就会超出范围，因此此后任何访问它的尝试都将导致未定义的行为。

您的选择是：

1. 使用堆变量（使用 new 获得）。您必须记住在某个时候删除它。
2. 让函数写入调用者提供的现有缓冲区（例如void strdup(char *out, const char *in)）。
3. 使用 C++ 结构，例如 std::string，它可以完成所有操作为你辛苦工作。

由于您已将问题标记为“C++”，因此我强烈推荐选项#3。