双链表插入分段错误-C

Question

我有一个家庭作业要写“先到先得”和“循环模拟”并比较它们。我开始创建一个链表来制作事件列表。我制作了插入和打印列表函数，但无法使其工作。我的朋友告诉我使用双链表，所以我重新创建了链表，并试图使插入函数工作。我修复了大部分问题，但现在找不到插入函数的错误。当我比较是否插入指针数据和当前指针数据时，While循环语句出现分段错误。我已经阅读了我的C语言书，书中的代码与我的代码相似，并且还通过了双链表分段错误-C问题，但仍然无法得到我的错误。

这是我的输出

-bash-3.2$ ./main 3 100
please enter the process-id, arrival-time and cpu-time.
1 2 3 4
something
something
DONE1
DONE2
DONE3
DONE4
Segmentation fault

由于test4（DONE4）已打印，而下一个测试未打印，这意味着我在这部分中有一个错误

while((currentptr->arrivaltime < newpointer->arrivaltime) && currentptr != NULL)

你能帮帮我吗？我的代码有什么问题？

struct event {
   struct event *next;
   int processid;
   int arrivaltime;
   int cputime;
};
typedef struct event Event;
typedef Event *eventPtr;

Event create_node(int processid, int arrivaltime, int cputime);
void add_event(Event *newpointer, eventPtr *eventlist);
void print_node(Event node);
void print_eventlist(Event *eventlist);
Event get_nextevent(Event *eventlist);

int main(int argc, char* argv[])
{
int sourcenum,a,b,c,i;
Event tempevent;
eventPtr eventlist = NULL;
print_eventlist(&(*eventlist));
char* sources,timeSlice;
if(argc != 3)
printf("Proper usage is: main.c sources time-slice\n");
else
{
    sourcenum = atoi(argv[1]);
    timeSlice = atoi(argv[2]);
    for(i = 0; i < sourcenum ; i++)
    {
          //print_eventlist(&(*eventlist));
          printf("please enter the process-id, arrival-time and cpu-time.\n");
          scanf("%d %d %d",&a, &b, &c);
          printf("something\n");
          tempevent = create_node(a,b,c);
          print
          printf("something\n");
          add_event(&tempevent, &eventlist);
    }

}
return 0;
}


void add_event(Event *newpointer, eventPtr *eventlist)     //insert
{
 printf("DONE1\n");
 eventPtr currentptr, prevptr;
 printf("DONE2\n");
 prevptr = NULL;
 printf("DONE3\n");
 currentptr = *eventlist;
 printf("DONE4\n");
 while((currentptr->arrivaltime < newpointer->arrivaltime) && currentptr != NULL)                  /*on every loop previous and current pointer gets upgraded*/
 {
         printf("DONEW\n");
         prevptr = currentptr;
         currentptr = currentptr->next;   
 }
 printf("DONEW\n");
 if(prevptr != NULL)
 {
     printf("DONEW\n");
     prevptr->next = newpointer;
     newpointer->next = currentptr;
 }
 else
 {
     printf("DONEW\n");
     newpointer->next = *eventlist;                      //?????
     *eventlist = newpointer;
 }
 printf("DONE\n");
}

Answer 1

eventPtr eventlist = NULL;

add_event(&tempevent, &eventlist);
{
    currentptr = *eventlist;  //currentptr == NULL at this point
    ...............
    while(currentptr->arrivaltime ....)  // Segmentation
}

在代码中，“eventlist=NULL”传递到“add_event”。这就是分割的原因。

我认为在“添加事件”中使用“事件列表”之前，应该先执行“malloc（）”以获取“事件列表”的条目。

Answer 2

你说问题很可能是在

while((currentptr->arrivaltime < newpointer->arrivaltime) && currentptr != NULL)

您想要测试currentptr=NULL表示它可能为NULL。但是如果是这样的话，你的早期症状就会消失。首先测试NULL，然后进行第二部分比较…：

while(currentptr != NULL && (currentptr->arrivaltime < newpointer->arrivaltime))