如何将结构体数组传递给函数

Question

我想将结构数组传递给函数，但不断收到错误。我不确定一般如何做，因为我从未被教过。我已经做了一些谷歌搜索，但我找不到任何东西。你们能帮我解释一下为什么这不起作用吗？这是整个程序的代码（很小，只是一个测试程序）：

 #include <stdio.h>
 #include <stdlib.h>
 #define hour 60


int getNum(void);
int distance(int start, int end , int flight_time[5], int flight_layover[5]);

int main(void)
{
    int user_start=-1;
    int user_end=-1;
    int travel_time=0;

struct flight 
{
    int flight_time;
    int flight_layover;
};

struct flight flights[5]=
{
        {4 * hour + 15, 1 * hour + 20},
        {3 * hour + 58, 0 * hour + 46},
        {3 * hour + 55, 11 * hour + 29},
        {2 * hour + 14, 0 * hour + 53},
        {3 * hour + 27, 0 * hour + 0}
};

printf ("Hello sir. Please enter you starting city:\n");
user_start=getNum();
user_start--;

printf ("Good. Now enter the city you would like to end it:\n");
user_end=getNum();
user_end--;

travel_time = distance(user_start,user_end, flights[5].flight_layover,          flights[5].flight_time);

printf ("The total travel time from %d to %d is %d.",user_start, user_end, travel_time);

return 0; 
}



int distance(int start, int end , int flight_time[5], int flight_layover[5])
{
    int total_mins=0;
    int i=0;
    for (i=end+1;i--;i=start)
    {
        total_mins=total_mins + flight_time[i] + flight_layover[i];
    }

    return total_mins;
}


int getNum(void)
{
    char record[121] = {0}; 
    int number = 0;
    fgets(record,  121, stdin);
    if(sscanf_s(record, "%d", &number) != 1 )
    {
        number  = -1;
    }
    return  number;
}

Answer 1

您的 distance 函数完全错误。由于您有一个包含 flight_time 和 flight_layover 的 struct，因此您希望将这些 struct 的数组传递给函数，而不是每个 int 值的数组。 IE。

int distance(int start, int end, struct flight flights[])
{
    int total_mins=0;
    int i=0;
    for (i=end+1;i--;i=start)
    {
        total_mins=total_mins + flights[i].flight_time + flights[i].flight_layover;
    }

    return total_mins;
}

更改了函数签名和 for 循环内的行。

然后，在调用 distance 时，您可以将调用更改为：

distance(user_start,user_end, flights)

这将传递一个指向数组开头的指针 (flights) 和 user_start 和 user_end 将指定用于计算距离的数组的边界。

另请注意，您可能会访问 flights 数组范围之外的索引。我特别不明白为什么您有 i=end+1 ，也许您想要i = end - 1？

您还反转了 for 循环的条件和递减部分，我认为它应该是这样的：

for (i = start; i <= end; i++)

编辑： 显然您的函数原型需要更新，并且您的 struct 应该在任何地方使用之前声明（包括在原型之前）。 这是经过我提到的修改的完整代码。

另外，请小心使用 getNum 函数，因为它可能会在出错时返回 -1，并且如果您在数组之前访问元素，则会导致未定义的行为。您还应该检查用户输入，以确保起始值和结束值介于 1 和 5 之间，并递减，以便它们是介于 0 和 4 之间的有效数组索引。

此外，您还可以使用 += 运算符添加到变量的当前值，即

total_mins=total_mins + flights[i].flight_time + flights[i].flight_layover;

可以更改为：

total_mins += flights[i].flight_time + flights[i].flight_layover;

EDIT2: 另一件事。按照惯例，常量#define应该是大写的，即。

#define HOUR 60 与 #define hour 60 相反。

Answer 2

我相信你想要这样的东西：

int distance(int start, int end , struct flight flights[5])
{
    int total_mins=0;
    int i=0;
    for (i=end+1;i--;i=start)
    {
        total_mins=total_mins + flights[i].flight_time + flights[i].flight_layover;
    }

    return total_mins;
}

然后被称为

travel_time = distance(user_start,user_end, flights);