-1

我不明白这段代码有什么问题。在编译期间没有错误，但在执行 Enqueu 的给定选项时，它会突然停止。问题发生在 Queue->rear->next=NULL 附近，在我看来这是正确的。我不知道我哪里错了。

struct ListNode
{
       int data;
       struct ListNode *next;
};

struct Queue
{
      struct ListNode *front;
      struct ListNode *rear;
};

struct Queue *createQueue()
{
      struct Queue *Q;
      Q=malloc(sizeof(struct Queue));
      if(!Q)
           return NULL;
      Q->front=Q->rear=NULL;
      return Q;
}

int IsEmptyQueue(struct Queue *Q)
{
      return (Q->front==NULL);
}

int EnQueue(struct Queue *Q,int data)
{
    struct ListNode *newNode;
    newNode=malloc(sizeof(struct ListNode));
    if(!newNode)
               return NULL;
    newNode->data=data;
    newNode->next=NULL;
    Q->rear->next=newNode;
    Q->rear=newNode;
    if(Q->front==NULL)
                Q->front=newNode;
}

int main()
{
    int choice=0,size,n;
    struct Queue *q;
    while(1)
    {
         printf("\nEnter the following");
         printf("\n1. Create a queue "); 
         printf("\n2.Enqueue");
         printf("\n7.Exit ");
         scanf("%d",&choice);

         switch(choice)
         {
                    case 1:printf("\nCreate Queue");
                           q=createQueue();
                           break;
                    case 2:printf("\nInsert");
                           printf("\nEnter element to be inserted");
                           scanf("%d",&n);
                           EnQueue(q,n);
                           break;

                    case 7:exit(0);
                    break;

      }
   }
}
4

2 回答 2

2

当队列为空时，其front成员rear为NULL. EnQueue然后取消引用NULL行中的指针

Q->rear->next = newNode;

当它第一次被调用时。该行不是必需的，因此可以简单地删除。

您还可以查看其他一些小错误

  • createQueue泄漏temp。你显然不需要声明/分配这个
  • EnQueue缺少对 malloc 失败的错误处理newNode。打印出“newNode Created”在这里有点误导！
  • 在%p打印出指向队列尾部的指针时用作格式说明符。
于 2013-01-10T15:12:43.230 回答
0

在createQueue()你设置Q->front和Q->rear到NULL但EnQueue()你正在使用Q->rear->next.

于 2013-01-10T15:12:55.347 回答