在通过对函数的引用传递结构时，需要使用 & 运算符吗？

Question

我的查询相对简单。存在一个名为war_name 的结构，其定义如下：

struct war_name
{
    char name[40];
    char location[20];
    int deaths;
    int injuries;
    char victor_name[20];
    char loser_name[20];
};

我创建的上述结构的输入函数如下：

void enter_details(struct war_name *d)
{
    printf("\n________________________________________\n");
    printf("Enter the name of the battle: ");
    fgets(d->name,sizeof(d->name),stdin);
    printf("Enter the location: ");
    fgets(d->location,sizeof(d->location),stdin);
    printf("Enter the number of deaths: ");
    scanf("%d",&d->deaths);
    printf("Enter the number of injuries: ");
    scanf("%d",&d->injuries);
    printf("Enter the victor's name: ");
    fgets(d->victor_name,sizeof(d->victor_name),stdin);
    printf("Enter the defeated party's name: ");
    fgets(d->loser_name,sizeof(d->loser_name),stdin);
    printf("\n________________________________________\n");
}

现在，main() 部分具有以下组成：

int main(void)
{
    int n;
    printf("How many war details do you wish to input?\n");
    scanf("%d",&n);
    struct war_name *ptr=(struct war_name *)malloc(n*sizeof(struct war_name));
    if(ptr==NULL)
        printf("\nInitialization unsuccessful\n");
    for(int i=0;i<n;i++)
    {
        enter_details(ptr[i]);
    }
    exit(EXIT_SUCCESS);
}

此时有两个查询到达。

首先，当我将ptr 值直接传递给enter_details() 函数时，为什么我的编译器会吐出错误cannot convert 'war_name' to 'war_name*' for argument '1' to 'void enter_details(war_name*)'？为了验证仅将ptr[i] 传递给函数的假设会发送第 i^th 结构的位置（这是函数所要求的），我直接打印了 ptr 的值，然后 lo 和看哪，该结构的地址被打印出来了。为什么尽管有这种发展，编译器会吐出错误并要求我使用 & 运算符传递结构的地址，因为 ptr[i] 本身持有 i^th 的值结构变量？

其次，关于ptr指针（指向结构war_name）的概念和malloc()的用法，是我用上述函数授予的可用空间初始化ptr的方式可以接受吗？为了参考起见，请注意，我所关注的书首先创建了一个指向 war_name 结构的数组指针，然后将所述结构的地址传递给函数？我的方法合理吗？考虑到我的方法节省了宝贵的内存，是否有必要构思一个指向上述结构的指针数组，然后将每个结构变量的地址传递给enter_details() 函数？

Answer 1

正如 cmets 中所说，您的代码存在问题：

---

第一：

enter_details(ptr[i]);

错了。 enter_details() 接受指向 struct war_name 的指针，而 ptr[i] 的类型为 struct war_name。所以你应该做的是传递ptr[i]的地址：

enter_details(&ptr[i]);

---

其次，fgets() 读取最后一个\n，所以你应该用\0 替换它：

fgets(d->name, sizeof(d->name), stdin);
d->name[strcspn(d->name, "\n")] = '\0';

---

第三，你还应该避免使用scanf() 来读取整数。使用fgets()，然后使用sscanf()解析输入：

char buffer[255];
fgets(buffer, sizeof(buffer), stdin);
buffer[strcspn(buffer, "\n")] = '\0';

if (sscanf(buffer, "%d", &d->deaths) != 1)
    printf("Could not read int\n");

---

第四，

struct war_name *ptr=(struct war_name *)malloc(n*sizeof(struct war_name));

太长太冗长了。 You don't need to cast the return value of malloc() except if you use a C++ compiler:

struct war_name *ptr = malloc(n * sizeof(*ptr));