Почему здесь бесконечный цикл? (печать связанного списка)

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; из цикла.Вы хотите указать на первого ученика в комнате только один раз.