Question

背景：

我使用calloc（）创建了一个数组，一切都运行良好。然后我使用realloc（）来使数组更大。它似乎只是创建一个没有任何内容的新指针，并在我尝试访问数组中的元素时调用运行时错误。

我的代码：

#include <stdio.h>

int main() {

  int *arr;

  for (int i = 0; i < 5; i++){
      if (i == 0) {
          if ((arr = (int *) calloc(1, sizeof(int))) == NULL) {
              printf("NO MORE SPACE TERMINATING PROGRAM");

              return NULL;
          }

      } else {
          int *tmp;

          if ((tmp = (int *)realloc(arr, 4*sizeof(int)) == NULL)){
              printf("NO MORE SPACE TERMINATING PROGRAM");
          }

          arr = tmp;
      }

      arr[i] = i;
  }
}

当i = 1时，它会在行arr[i]=i;上抛出运行时错误;

当我在arr = temp;之上添加一个断点时，它显示temp是完全空白的。

为什么会这样？

--------更新----------------

感谢所有帮助。这是工作更新的代码：

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

int main(){
    int *arr = NULL;

    for (int i = 0; i < 5; i++){
        int *tmp;

        if ((tmp = realloc(arr, (i+1) * sizeof(int))) == NULL){
            printf("NO MORE SPACE TERMINATING PROGRAM");
        }

        arr = tmp;

        arr[i] = i;
    }

    for (int i = 0; i < 5; i++) {
        printf("%i",arr[i]);
    }
}

输出为：01234

感谢大家的帮助。

Answer 1

始终留意警告。这样做会显示：

warning: assignment makes pointer from integer without a cast [enabled by default]
           if ((tmp = (int *)realloc(arr, 4*sizeof(int)) == NULL)){

这告诉您编译器正在为int分配tmp。怎么可能？如果你仔细观察，你会发现括号不正确。实际上，它将realloc(arr, 4*sizeof(int)) == NULL的结果赋予tmp。正确的代码如下。注意括号在作业周围的位置。

if ((tmp = realloc(arr, 4*sizeof(int))) == NULL)

对于#include <stdlib.h>和calloc的定义，您遗漏了realloc。

更新：正如M.M和其他人所建议的那样，即使修复了上述问题，realloc仍然不正确，因为它只分配了4个元素，而for循环将访问5个元素。因此，需要将4更改为5或i+1。即使它不会导致任何功能问题，也不必在0之后的每个循环上realloc。

Realloc不会复制旧数据

1 个答案: