c - 使用链表的现有实现来实现队列: Bad File Number error

Question

我收到了一些用于实现链表的现有 C 代码(一个头文件和一些源代码)，并收到了一项使用它来实现队列的任务。

这是我获得的头文件的一部分，其中包含相关功能描述:

/* List is a pointer to a list_t struct */
typedef struct list_t* List;

struct list_t {
    void *data;
    List next;
};

/* Pushes data as the new head of list. May be used to create a new list:
 * new_list = push(NULL, data) */
extern List push(List list, void *data);

/* Pop the head off the list */
extern void *pop(List *list);

/* Return the length of the list */
extern int len(List list);

/* Returns a reversed copy of list */
List reverse(List list);

/* Prepend data to list and update list */
extern List prepend(List *list, void *data);

/* Append l1 to the end of l2 */
void append(List l1, List *l2);

/* Inserts data into the tail of list */
void insert(void *data, List *list);

/* Inserts data into the tail of list or position equal to the next element     */
void insert_by(bool (*eq)(void *data, void *node), void *data, List *list);

/* Inserts data into the tail of list. Returns true if sucessful,
 * false if it finds an element already equal to data */
bool insert_if(bool (*eq)(void *data, void *node), void *data, List *list);

/* Returns the node equal to aim in list, returns NULL if not found */
extern List find(bool (*eq)(void *aim, void *node), void *aim, List list);

/* Removes and returns the element equal to aim in list,
 * returns NULL if not found */
extern void *del(bool (*eq)(void *aim, void *node), void *aim, List *list);

/* Returns a new list that passes the predicate p */
List filter(bool (*p)(void *data), List list);

/* Print list to f by applying print to each node that is not NULL */
extern void print_list(void (*print)(FILE *f, void *data), FILE *f, List node);

/* Free the memory allocated to each list node */
extern void free_list(List node);

我知道为了实现队列，我至少需要两个函数:enqueue() 和 dequeue()。我继续使用上述头文件中的 List 类型，使用这些函数和队列的 typedef 创建了自己的头文件:

//Queue.h
#include "list.h"

typedef List Queue;

//Add item to queue...
void enqueue(Queue q, void *data);

//removes and returns an item from the queue...
void dequeue(Queue *q);

然后我继续在 queue.c 中实现源代码。我现在只实现了enqueue，因为我想在继续之前确保它有效:

#include "queue.h"

void enqueue(Queue q, void *data){
    if (q == NULL){
        q = push(q, data);
    }
    else {
        insert(data, &q);
    }
}

很简单，我知道。我计划使用以下文件 main.c 来测试队列:

#include <stdio.h>
#include "queue.h"

int main(int argc, char **argv){
    Queue q = NULL;
    int i;
    for (i = 0; i < 10; i++){ enqueue(q, &i); } //one line for brevity
    return 0;
}

此时，我并没有期望在运行 main.c 时看到任何输出，我所期望的只是程序运行没有错误，然后停止。一切都编译得很好，但是当我运行 main.c 时，我得到的是:

sh: ./main.exe: bad file number

这是什么意思？有人可以查明是什么原因导致了这个问题吗？

编辑:源代码编译如下:

gcc -c list.c
gcc -c queue.c
gcc -c main.c -o main.exe

Answer 1

您的enqueue函数破坏了push方法的约定。

push方法返回新的头。

您需要:

Queue enqueue(Queue q, void *data){
    return push(q, data);
}

在您的通话中:

for (i = 0; i < 10; i++){ 
    q = enqueue(q, &i); 
}

但请注意，这将在每次迭代中推送相同的指针。如果更改i，您将更改队列中每个节点的值。这可能不是您想要的!

另请注意，对本地(自动)变量的地址进行排队可能是一件坏事，并且当变量超出范围时可能会导致堆栈问题。 (在您的情况下，i 是在 main 中声明的，因此在程序结束之前它不会超出范围。)