优先队列未按升序插入元素

Question

这是使用链表的优先队列的代码 我对这段代码有两个疑问

1) 这段代码是我的插入函数的一部分

 if((*addofhead)->priority <= newnode->priority ){
    struct node* temp=*addofhead;
    while(temp->next!=NULL&&temp->priority <newnode->priority ){
        temp=temp->next;
    }
    newnode->next=temp->next;
    temp->next=newnode;
    return;
   }

为什么我们不能在 while 循环而不是 temp->next!=NULL 中执行 temp!=NULL ，因为 temp!=NULL 也会在另一次迭代中退出循环，但它正在崩溃 崩溃的原因是什么

2)我想创建一个优先级队列，使得具有最低优先级的元素 应该首先删除并且具有相同优先级的元素然后将首先删除首先添加的元素 来自main函数的输入部分

    insert(&head,3,5);   
    insert(&head,2,2);   

    insert(&head,1,1);   
    insert(&head,7,1);
    insert(&head,11,1);
    insert(&head,8,5);   
    insert(&head,9,5);

我得到了这个1 2 11 7 3 8 9 的输出，但它的输出应该是1 7 11 2 3 8 9

#include<stdio.h>
#include<stdlib.h>
struct node{
    int data;
    int priority;
    struct node* next;
};
struct node* getnewnode(int data,int priority){
    struct node* newnode=malloc(sizeof(struct node));
    newnode->data=data;
    newnode->next=NULL;
    newnode->priority=priority;
    return newnode;
}
void insert(struct node** addofhead,int data,int priority){

    struct node* newnode=getnewnode(data,priority);
    if(*addofhead==NULL){
        *addofhead=newnode;
        printf("head in insert is %d",*addofhead);
        return;
    }

    if((*addofhead)->priority > newnode->priority){
        newnode->next=*addofhead;
        *addofhead=newnode;
        return;
    }

    if((*addofhead)->priority <= newnode->priority ){
        struct node* temp=*addofhead;
        while(temp->next!=NULL&&temp->priority <newnode->priority ){
            temp=temp->next;
        }
        newnode->next=temp->next;
        temp->next=newnode;
        return;
    }
}
int removee(struct node** head){
    if(*head==NULL)
        return -1;
    int temp=(*head)->data;
    *head=(*head)->next;
    return temp;
}
int main(){
    struct node* head=NULL;
    insert(&head,3,5);   /// 3
    insert(&head,2,2);   /// 2,3

    insert(&head,1,1);   /// 1,2,3
    insert(&head,7,1);
    insert(&head,11,1);
    insert(&head,8,5);   /// 1,7,2,3
    insert(&head,9,5);
    struct node* temp=head;
    while(temp)
    {
        printf(" %d ",temp->data);
        temp=temp->next;

    }
    printf("\n head in main  after insertion is %d",head);

}

Answer 1

为什么我们不能在 while 循环中使用 temp!=NULL 而不是 temp->next!=NULL (?)

因为循环后面的行有temp->next;@Jonathan Leffler。

沿着链表进行插入的通常目标是知道前一个节点的指针，以便更新其.next成员。

---

代码有 2 个功能问题

比较错误的优先级

// while(temp->next!=NULL&&temp->priority <newnode->priority ){
while(temp->next!=NULL&&temp->next->priority <newnode->priority ){
//                          ^^^^^^    

== 时比较错误

// while(temp->next!=NULL&&temp->next->priority <newnode->priority ){
while(temp->next!=NULL&&temp->next->priority <= newnode->priority ){
//                          ^^^^^^           ^^

---

也可以使用%p 来打印void *，而不是"%d" 来打印指针。

// printf("head in insert is %d",*addofhead);
printf("head in insert is %p",(void *) (*addofhead));

---

对此非常有用的是创建一个辅助函数来打印数据。每次插入后都可以轻松调用它以缩小问题范围。

void pq(const struct node* p) {
  while (p) {
    printf("(data %d,pri %d) ", p->data,  p->priority);
    p = p->next;
  }
  puts("");
}

---

我发现 OP 的 insert() 过于复杂。见@wildplasser。