为什么当我尝试访问结构数组时它返回随机值？ （C）

Question

我有一个 function 返回指向结构数组的指针。 但是，当我尝试访问返回结构的任何值时，它会打印随机符号。

#include <stdio.h>

struct MY {
   int i;
   char string[30];
};

struct MY* myFunc() {
  struct MY arrayOfStructs[3];
  struct MY tempStruct;
  struct MY* arrayOfStructsPtr = arrayOfStructs;

  tempStruct.i = 1;
  tempStruct.string[0] = 'H';
  tempStruct.string[1] = 'i';
  arrayOfStructs[0] = tempStruct;

  tempStruct.i = 2;
  tempStruct.string[0] = 'L';
  tempStruct.string[1] = 'o';
  arrayOfStructs[1] = tempStruct;

  tempStruct.i = 3;
  tempStruct.string[0] = 'M';
  tempStruct.string[1] = 'Y';
  arrayOfStructs[2] = tempStruct;

  return arrayOfStructsPtr;
}

int main()
{
   struct MY* arrayOfStructs = myFunc();
   for(int i = 0; i < 3; i++) printf("%d\n", arrayOfStructs[i].i);
   return 0;
}

Answer 1

您返回对本地数组的引用，该数组在 function 返回时停止存在。 这是未定义的行为。

你需要：

struct MY* myFunc(void) {
  static struct MY arrayOfStructs[3];

或者

struct MY* myFunc(void) {
  struct MY *arrayOfStructs = malloc(3 * sizeof(*arrayOfStructs));

或传递调用者分配的缓冲区。

struct MY *myFunc(struct MY *arrayOfStructs) {
   /* .... */

如果你动态分配 memory 你应该在使用后free它

Answer 2

当 function 返回时，您返回一个指向 scope 之外的局部变量的指针。 一些选择：

1. 调用者main()分配变量并将其传递给myFunc()进行初始化。

#include <stdio.h>
#include <string.h>

#define N 3

struct MY {
    int i;
    char string[30];
};

void myFunc(struct MY arrayOfStructs[N]) {
    char *strings[N] = { "Hi", "Lo", "MY" };
    for(size_t i = 0; i < N; i++) {
        arrayOfStructs[i].i = i + 1;
        strcpy(arrayOfStructs[i].string, strings[i]);
    }
}

int main() {
    struct MY arrayOfStructs[N];
    myFunc(arrayOfStructs);
    for(int i = 0; i < N; i++)
        printf("%d\n", arrayOfStructs[i].i);
}

如此处所用，您实际上不需要存储i ，因为它只是结构 + 1 的索引。

2. myFunc()使用malloc() ) 动态分配变量并返回指针。 呼叫者负责释放分配的 memory。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define N 3

struct MY {
    int i;
    char string[30];
};

struct MY *myFunc() {
    struct MY *arrayOfStructs = malloc(N * sizeof *arrayOfStructs);
    if(!arrayOfStructs) return NULL; // malloc failed
    char *strings[N] = { "Hi", "Lo", "MY" };
    for(size_t i = 0; i < sizeof strings / sizeof *strings; i++) {
        arrayOfStructs[i].i = i + 1;
        strcpy(arrayOfStructs[i].string, strings[i]);
    }
    return arrayOfStructs;
}

int main() {
    struct MY *arrayOfStructs = myFunc();
    if(!arrayOfStructs) return 1;
    for(int i = 0; i < N; i++)
        printf("%d\n", arrayOfStructs[i].i);
    free(arrayOfStructs);
}

3. myFunc() ：生成变量 static（不推荐）。