从 C++ 中的函数返回指针

Question

当我从函数返回指针时，它的值可以单独访问。但是，当使用循环输出该指针变量的值时，会显示错误的值。我在哪里犯错误，无法弄清楚。

#include <iostream>
#include <conio.h>

int *cal(int *, int*);

using namespace std;

int main()
{
    int a[]={5,6,7,8,9};
    int b[]={0,3,5,2,1};
    int *c;
    c=cal(a,b);

    //Wrong outpur here
    /*for(int i=0;i<5;i++)
    {
        cout<<*(c+i);
    }*/

    //Correct output here
    cout<<*(c+0);
    cout<<*(c+1);
    cout<<*(c+2);
    cout<<*(c+3);
    cout<<*(c+4);

return 0;
}   

int *cal(int *d, int *e)
{
    int k[5];
    for(int j=0;j<5;j++)
    {
        *(k+j)=*(d+j)-*(e+j);
    }
    return k;
}

Answer 1

您正在返回一个指向局部变量的指针。

k 在堆栈上创建。当cal() 退出时，堆栈被展开并且内存被释放。之后引用该内存会导致未定义的行为（如此处精美的解释：https://stackoverflow.com/a/6445794/78845）。

您的 C++ 编译器应该对此发出警告，并且您应该注意这些警告。

对于它的价值，这是我在 C++ 中实现它的方法：

#include <algorithm>
#include <functional>
#include <iostream>
#include <iterator>

int main()
{
    int a[] = {5, 6, 7, 8, 9};
    int b[] = {0, 3, 5, 2, 1};
    int c[5];
    std::transform (a, a + 5, b, c, std::minus<int>());
    std::copy(c, c + 5, std::ostream_iterator<int>(std::cout, ", "));
}

See it run!

Answer 2

int k[5] 数组在堆栈上创建。因此，当它从cal 返回超出范围时，它会被销毁。您可以使用第三个参数作为输出数组：

void cal(int *d, int *e, int* k)
{
    for(int j=0;j<5;j++)
    {
        *(k+j)=*(d+j)-*(e+j);
    }
}

像这样拨打cal：

int a[]={5,6,7,8,9};
int b[]={0,3,5,2,1};
int c[5];
cal (a, b, c); // after returning from cal, c will be populated with desired values

Answer 3

正如其他人所指出的，您正在返回一个指向本地的指针 变量，这是未定义的行为。然而，真正的问题是 您需要返回一个数组，而 C 样式的数组已损坏。 用std::vector<int> 替换你的数组，忘记指针 （因为您正在处理值），并且代码将起作用。