0

这是代码：A[0]（在 main 函数中）应该等于 0，而不是 1。我找不到我的错误。我想问题出在 and1 函数的某个地方，但同样，我似乎无法找到它。无论如何，我很确定第一句话很好地解决了这个问题，但是网站迫使我写更多的信息。

#include <iostream>
#include <string>
// V and ^ or
using namespace std;
int A[] = {0, 1, 1};
int B[] = {1, 0, 1};

 int* and1(int A[], int B[])
{
    int ret[3];
    for(int i = 0; i < 3; i++)
    {
        if(A[i] == 1 && B[i] == 1 )
        {
            ret[i] = 1;
        }
        else
        {
            ret[i] = 0;
        }
    }
    return ret;
}

int* or1(const int A[], const int B[])
{
    int ret[] = {0 ,0 ,0};
    for(int i = 0; i < 3; i++)
    {
        if(A[i] == 1 || B[i] == 1)
        {
            ret[i] = 1;
        }
        else
        {
            ret[i] = 0;
        }
    }
    return ret;
}

int main()
{
    int* a = and1(A, B);
    int* b = or1(A, B);
    if(*(a+1) == *(b+1))
    {
        cout << a[0] << endl;
    }
    return 0;
}
4

3 回答 3

3

您正在返回指向函数本地数组的指针，并且当函数范围{ }结束时这些本地数组不存在。你得到的是一个指向不存在的东西的指针和一个未定义的行为。

于 2013-03-09T07:47:10.310 回答
2

int ret[3];in 函数and1是一个局部变量and1。当and1完成执行时，它会超出范围。所以返回它的地址是没有意义的。相反，您可以将ret数组传递给and1（与 or1 类似），原型为：

void and1(const int A[], const int B[], int ret[]);
于 2013-03-09T07:48:46.853 回答
2

您正在从 function 返回一个临时数组的指针and1。结果是不确定的。

int* and1(int A[], int B[])
{
   int ret[3];
   //...
   return ret;
}

int* a = and1(A, B); // <-- Undefined behavior

之后return ret，数组被ret破坏，它并不意味着更多的使用。

于 2013-03-09T07:50:54.990 回答