仅当我打印了属性的值时，功能才能正常工作

Question

我正在尝试为大学创建一个库来处理来自 XML 文件的音乐分区。 我不想详细介绍，但我有一个函数可以创建分区 score，另一个函数可以添加单独的注释 score_add_note。

原型如下：

score_t* score(size_t tempo, size_t duration, size_t sig, size_t nbInstr)
void score_add_note(score_t* sc, size_t ins, size_t bar, pitch_t p, size_t start, size_t duration)

我还将 XML 文件转换为我制作的结构，在本例中称为 e。

有一次，我这样使用这些函数：

score_t* sc = score(atoi(e->at_head->value), atoi(e->at_head->next->value), atoi(e->at_head->next->next->value), atoi(e->at_head->next->next->next->value));
score_add_note(sc, 0, 0, 60, 0, 6);

属性是 XML 结构的一部分。

这不起作用，score_add_note 函数永远不会停止执行。

但是，如果我事先将属性中的值打印到 score 函数，它可以正常工作：

printf("%d\n", atoi(e->at_head->value));
printf("%d\n", atoi(e->at_head->next->value));
printf("%d\n", atoi(e->at_head->next->next->value));
printf("%d\n", atoi(e->at_head->next->next->next->value));

score_t* sc = score(atoi(e->at_head->value), atoi(e->at_head->next->value), atoi(e->at_head->next->next->value), atoi(e->at_head->next->next->next->value));
score_add_note(sc, 0, 0, 60, 0, 6);

是我犯了一些错误还是这是 C 中的一个错误。如果是这样，有什么办法可以解决吗？ 顺便说一句，我使用 gcc 作为我的编译器。

编辑： 如要求，这是score_add_note 函数

void score_add_note(score_t* sc, size_t ins, size_t bar, pitch_t p, size_t start, size_t duration){
    if(sc->system[ins].tab[bar] == NULL){
        sc->system[ins].tab[bar] = (bar_t*) malloc (sizeof(bar_t));
        sc->system[ins].tab[bar]->start = start;
        sc->system[ins].tab[bar]->duration = duration;
        sc->system[ins].tab[bar]->pitch = p;
    }else{
        note_t* note = sc->system[ins].tab[bar];
        while(1){
            if(note->next == NULL){
                note->next = (note_t*) malloc (sizeof(note_t));
                note = note->next;
                break;
            } 
            note = note->next;
        }
        note->start = start;
        note->duration = duration;
        note->pitch = p;
    } 
}

函数有可能陷入无限循环，但两种情况下的参数相同。

这是score 函数：

score_t* score(size_t tempo, size_t duration, size_t sig, size_t nbInstr){
    score_t* sc = (score_t*) malloc (sizeof(score_t));
    sc->tempo = tempo / 60;
    sc->duration = duration;
    sc-> time_sig = sig;
    sc->sys_size = nbInstr;
    sc->system = (staff_t*) malloc (nbInstr * sizeof(staff_t));
    for(int i = 0; i < nbInstr; i++){
        sc->system[i].tab = (bar_t**) malloc (duration * sizeof(bar_t*));
    }
    return sc;
}

这些是我创建的类型：

typedef struct note_s{
    size_t start;
    size_t duration;
    pitch_t pitch;
    struct note_s* next;
}note_t;


typedef note_t bar_t;

typedef struct staff_s{
    bar_t** tab;
}staff_t;


struct score_s{
    size_t tempo;
    size_t duration;
    size_t time_sig;
    size_t sys_size;
    staff_t* system;
};

Answer 1

这段代码

    note_t* note = sc->system[ins].tab[bar];
    while(1){
        if(note->next == NULL){
            note->next = (note_t*) malloc (sizeof(note_t));
            note = note->next;
            break;
        } 
        note = note->next;
    }

在链表上创建新节点时从不将note->next 设置为NULL。因此，您稍后在访问列表时会调用未定义的行为。

这样会更好：

    note_t* note = sc->system[ins].tab[bar];
    while (1) {
        if (note->next == NULL){
            note->next = malloc(sizeof *note->next);
            note = note->next;
            note->next = NULL;
            break;
        } 
        note = note->next;
    }