使用 malloc() 或 realloc() 存储并在 C 中打印数据 [关闭]

Question

有人可以告诉我代码有什么问题吗？抱歉，如果我是新手，但我很长一段时间都在努力解决这个问题。我正在尝试从用户那里获取输入，然后打印这些值。下面的代码只是一个例子。

我尝试运行代码，但出现运行时错误。有人可以帮忙吗？

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

typedef struct poly
{
    int kill;
    float bill;
    char katana[50];
} koly;
 typedef koly* terma;

int main()
{
        int count = 0;
        terma ren;
        ren = (terma)malloc(sizeof(koly));
        ren = (terma)realloc(6*sizeof(koly));
        printf("We can store now:\n\n");
        while(++count<= 2)
        {
            scanf("%d",ren->kill);
            scanf("%f",ren->bill);
            scanf("%s",ren->katana);
        }
        while(++count<= 2)
        {
            printf("\n%d\n",ren->kill);
            printf("\n%f\n",ren->bill);
            printf("\n%s\n",ren->katana);
        }
}

Answer 1

关于你的错误：

scanf("%d",ren->kill);
scanf("%f",ren->bill);

这些应该是指针，所以更正的版本是：

scanf("%d",&ren->kill);
scanf("%f",&ren->bill);

另外，realloc 需要 2 个参数，这与 malloc 不同。第一个应该是指向要重新分配的地址的指针，第二个应该是新的大小。所以：

ren = realloc(ren, 6 * sizeof(koly));

关于您的代码，我还有两件事要说。首先：没有必要在 C 中强制转换 malloc 的返回值（并且不赞成）。此外，切勿将scanf 与"%s" 格式说明符一起使用。这是非常不安全的，很容易导致崩溃甚至更糟。最好的方法是使用fgets，所以scanf("%s",ren->katana);会变成这样：

fgets(ren->katana, sizeof(ren->katana), stdin);

编辑： 以下是我对为什么应该避免转换 malloc/realloc 的返回值的解释：

这不是真的，因为它是错;这是因为在 C 中这样做是不好的做法。由于 void* 类型会自动转换为任何指针类型，因此类型转换是多余的，代码可读性较差（IMO），并且使更改更加困难。例如，以你的情况为例：

ren = (terma)malloc(sizeof(koly));

我个人会这样重写这一行：

ren = malloc(sizeof(*ren));

这允许轻松更改ren 的类型。如果将ren 更改为另一种数据类型（例如int* 或struct sockaddr_in* 或其他任何类型），它将自动工作。您不需要更改演员表中的类型或sizeof() 中的类型。两者都将自动适用于新类型。对于数组，它的工作原理是一样的：

int* myArr = malloc(42 * sizeof(*myArr));

如果我希望 myArr 保存一个包含 42 个 floats 的数组，那么更改很简单：

float* myArr = malloc(42 * sizeof(*myArr));

注意：在 C++ 中，类型转换是必需的，因为它不会自动将 void* 转换为其他指针类型，但是当你有 new 时，你真的不应该在 C++ 中使用 malloc 等（加上这是一个 C 问题，而不是 C++）。

Answer 2

void *realloc(void *ptr, size_t size);

realloc 有两个参数，原型如上，你在代码中遗漏的是给出你想要重新分配内存的指针。

你必须使用，ren = (terma)realloc(rem, 6*sizeof(koly));

另一个错误是你在scanf函数中漏掉了&运算符，应该是scanf("%d",&ren->kill);

而且它不是运行时错误，您是否正确编译了代码...？

Answer 3

Realloc() 有两个参数。它是如何编译的？

ren = (terma)realloc(ren,6*sizeof(koly));

您还需要更正 scanf：

scanf("%d",&ren->kill);
scanf("%f",&ren->bill);

但请注意：您正在覆盖您唯一的一个对象。