Проблема с односвязным циклом

[Joose]

Привет. Есть некоторая функция, написанная на Си, которая считывает в односвязный список структур некоторые данные, в результате чего каждый элемент списка имеет вид:

строка
число
указатель на следующее

Код всей программы приводить не буду, опишу только функцию, которая выполняет считывание. start - глобальная переменная, имеющая смысл начала списка, в которую записывается самая первая по алфавиту карточка, и в последующем значение в start меняться не должно:

void loadDict(const char *name_file)
{
    int freq = 0;
    char word[25];
    Card *newPtr, *current = NULL;
    FILE *file;
    start = NULL;                                   /*Очистить заглавную карточку, т.к. возможно нахождение в ней ненужного мусора*/
    file = fopen(name_file, "r");

    while(!feof(file))
    {
        newPtr = (Card*)malloc(sizeof(Card));       /*Выделение памяти для создания очередной карточки*/
        fscanf(file, "%s%d", word, &freq);          /*Считать из файла очередное слово и число его вхождений*/
        newPtr->name = word;                        /*Записать считанное слово во временное хранилище*/
        newPtr->ent = freq;                         /*Записать считанное число в временное хранилище*/
        newPtr->next = NULL;                        /*Указателю на следующее присвоить NULL*/

        if(start == NULL)                           /*Если это первое слово, то записать структуру из хранилища в первый элемент списка*/
        {
            start = newPtr;
            printf("%-15s\t%3d\n", start->name, start->ent);
        }
        
        else                                        //Иначе это второе и более слово, нужно записать его в другую карточку
        {
            current = newPtr;
            printf("%-15s\t%3d\n", current->name, current->ent);
            current = current->next;
        }

    }

            printf("%-15s\t%3d\n", start->name, start->ent);
    fclose(file);
}

Проблема в том, что значение в карточке start меняется на протяжении выполнения всего цикла по считыванию из файла. Хотя в условии указано, что считывать в start необходимо только один раз, когда start == NULL. По сути мне необходимо, чтобы в start хранилась первая карточка списка, указатель на следующее из start указывал на current. Дальнейшая вставка карточек должна выполняться при помощи current, не передвигая start. Помогите разобраться, пожалуйста.

[Joose]

Ошибся в названии темы, подразумевался односвязный список

[kineziz]

Тут в коде есть непонятка:

newPtr = (Card*)malloc(sizeof(Card));       /*Выделение памяти для создания очередной карточки*/
        fscanf(file, "%s%d", word, &freq);          /*Считать из файла очередное слово и число его вхождений*/
        newPtr->name = word;                        /*Записать считанное слово во временное хранилище*/
        newPtr->ent = freq;                         /*Записать считанное число в временное хранилище*/
        newPtr->next = NULL;                        /*Указателю на следующее присвоить NULL*/

        if(start == NULL)                           /*Если это первое слово, то записать структуру из хранилища в первый элемент списка*/
        {
            start = newPtr;
            printf("%-15s\t%3d\n", start->name, start->ent);
        }
        
        else                                        //Иначе это второе и более слово, нужно записать его в другую карточку
        {
            current = newPtr;
            printf("%-15s\t%3d\n", current->name, current->ent);
            current = current->next;
        }

Вы инициализируете

newPtr->next

нулем, но затем вы приравниваете

current = current->next;

В итоге получается что строчка

current = current->next;

эквивалентна строке

current = NULL

По идеи должно быть так:

current->next = newPtr

Это уже логически верно, т.к. устанавливается связь между current и новым элементом

[BDA]

Попробуйте:

if(start == NULL)
{
    current = start = newPtr;
    printf("%-15s\t%3d\n", start->name, start->ent);
} else {
    current->next = newPtr;
    current = newPtr;
    printf("%-15s\t%3d\n", current->name, current->ent);
}