0

我面临一个问题，我肯定做错了什么。我需要调用一个函数，该函数返回一个指向 int 数组的指针，但是当它返回后，数组中的值是错误的并且某些值丢失了。

int* patternForFirstDigit(int digit) {
    int *pattern;
    pattern = (int [6]){1,1,1,1,1,1};

    switch (digit) {
        case 0:
            pattern = (int [6]){1,1,1,1,1,1};
            break;

        case 1:
            pattern = (int [6]){1,1,2,1,2,2};
            break;

        default:
            pattern = (int [6]){0,0,0,0,0,0};
            break;
    }

    for (int i = 0; i < 6; i++) {
         printf("%i\n", pattern[i]);
    }

    return pattern;
}

在数字 = 1 的情况下，这是打印的内容

1, 1, 2, 1, 2, 2

但回国后

int *pattern = patternForFirstDigit(0);
for (int i = 0; i < 6; i++) {
     printf("%i\n", pattern[i]);
}

这是打印的内容

1, -1405451528, -1405449120, 366001

你知道出了什么问题吗？

多谢你们

PS：我使用的是 Xcode 4.6，我的项目使用的是 ARC，但我很确定这不是我的问题的原因。

4

3 回答 3

5

您不能返回指向在函数中创建的数组的指针。该数组在函数返回后不再存在，因此您的指针指向内存中某个随机的无效位置。

为指针分配内存（malloc()例如使用），然后返回一个指针。这也意味着您需要在完成后释放指针（使用free()）。

伪代码类似于：

int* patternForFirstDigit(int digit) {
  int *pattern = (int*) malloc(sizeof(int)*N);
  pattern[0] = 0;
  pattern[1] = 1;
  ...

  // Alternatively just create a local array and use a for-loop
  // to copy the contents to the pattern array.

  return pattern;
}

int *p = patternForFirstDigit(M);
// use p
free(p);
于 2013-03-28T10:08:15.723 回答
1

这

(int [6]){1,1,2,1,2,2};

是函数中定义的局部数组。因此，当函数完成执行时，可以删除数组的数据。所以这就是为什么你在你的printf

1)malloc用于分配数组

int* patternForFirstDigit(int digit) {
    int *pattern = malloc(6*sizeof(int));

    memcpy(pattern, (int [6]){1,1,1,1,1,1}, 6*sizeof(int));

    switch (digit) {
        case 0:
            memcpy(pattern, (int [6]){1,1,1,1,1,1}, 6*sizeof(int));
            break;

        case 1:
            memcpy(pattern, (int [6]){1,1,2,1,2,2};, 6*sizeof(int));
            break;

        default:
            memcpy(pattern, (int [6]){0,0,0,0,0,0}, 6*sizeof(int));
            break;
    }

    for (int i = 0; i < 6; i++) {
         printf("%i\n", pattern[i]);
    }

    return pattern;
}

以及代码中的某些地方pattern变得无用时，然后将其释放free(pattern);

2）或者static在数组的定义中使用：

int* patternForFirstDigit(int digit) {
    int *pattern; int i;
    static int A[6]={1,1,1,1,1,1};
    static int B[6]={1,1,2,1,2,2};
    static int C[6]={0,0,0,0,0,0};
    pattern = A;

    switch (digit) {
        case 0:
            pattern = A;
            break;

        case 1:
            pattern = B;
            break;

        default:
            pattern = C;
            break;
    }

    for (i = 0; i < 6; i++) {
         printf("%i\n", pattern[i]);
    }

    return pattern;
}
于 2013-03-28T10:08:48.387 回答
0

问题是因为您要返回一个局部变量。局部变量是一个临时变量，在其范围消失后不再可用。您可以尝试以下方法：

int* patternForFirstDigit(int digit, int* pattern) {

    int* pattern1 = (int [6]){1,1,1,1,1,1};
int i;
    switch (digit) {
        case 0:
            pattern1 = (int [6]){1,1,1,1,1,1};
            break;

        case 1:
            pattern1 = (int [6]){1,1,2,1,2,2};
            break;

        default:
            pattern1 = (int [6]){0,0,0,0,0,0};
            break;
    }
memcpy(pattern,pattern1,6*sizeof(int));
    for ( i = 0; i < 6; i++) {
         printf("%i\n", pattern[i]);
    }
    return pattern;
}

然后你可以像这样使用它：

pattern = malloc(6*sizeof(int));
pattern=patternForFirstDigit(1, pattern);
int i;
for ( i = 0; i < 6; i++) {
     printf("%i\n", pattern[i]);
}
free(pattern);
于 2013-03-28T10:11:24.910 回答