為什么這里有無限循環？ （鏈接列表打印）

Question

我正在對鏈表進行一些練習，這些是結構。

typedef struct roomList roomList;
typedef struct school school;
typedef struct studentList studentList;
roomList *getRoom(school* school, int class, int roomNr);

struct studentList{

    char *name;
    int class; 
    float grade;
    int roomNr;
    studentList *next;
    studentList *prev;
};


struct roomList{

    int nrOfStudents;
    int roomNr;
    studentList *students; //pointer to student list.
    roomList *next;
    roomList *prev; 
};



struct school{

    int totalStudents;
    roomList *Class[13]; //array of classes, each index contains rooms.
};

這是發生無限循環的地方，它是打印房間內所有學生的功能。

void printRoom(school *school, int class, int roomNr)
{
    roomList *room = getRoom(school, class, roomNr);
    studentList *student;

    if(room != NULL)
    {
        int i = 1;
        printf("Nr of students: %d\n", room->nrOfStudents);
        while(room->nrOfStudents != 0 && student != NULL)
        {
            student = room->students;
            printf("%d - \"%s\" ",i, student->name);
            student = student->next;
            i++;
        }
    }   
}

這就是我創造student

studentList *createStudent(int class, char *name, int roomNr)
{
    studentList *newNode;
    newNode = (studentList*)calloc(1, sizeof(studentList));
    newNode->class  = class;
    newNode->name   = (char*)malloc(strlen(name)+1);
    strcpy(newNode->name, name);
    newNode->roomNr = roomNr;
    newNode->grade  = 0;
    newNode->next   = newNode->prev = NULL;

    return newNode;
}

最后，這就是我將student插入room 。

void insertStudentToRoom(school* school, int class, int roomNr, char *name)
{
    roomList *room;
    room = getRoom(school, class, roomNr);
    studentList *newStudent;
    newStudent = createStudent(class, name, roomNr);

    if(room->students != NULL)
    {
        newStudent->next = room->students;
        room->students->prev = newStudent;
        room->students = newStudent;
        room->nrOfStudents++;
        school->totalStudents++;
    }
    else
    {
        room->students = newStudent;
        room->nrOfStudents++;
        school->totalStudents++;
    }
}

無限無限循環僅在我將一個以上的student插入一個room時才發生，而在只有一個學生的情況下退出就很好了，我試圖為while()摸索退出條件無濟於事。

Answer 1

    while(room->nrOfStudents != 0 && student != NULL)
    {
        student = room->students;
        printf("%d - \"%s\" ",i, student->name);
        student = student->next;
        i++;
    }

仔細看。 您永遠不會在循環中改變room 。 因此， student = room->students; 將在循環中的每個時間為student設置非常相同的值。 如果它在第一次之后沒有中斷，那么它將不會再中斷。

您可能想讓student = room->students; 跳出循環。 您只想指向房間中的第一個學生。