VPF::связный список на Си - Форум программистов

kazansky

Дата 27.9.2012, 20:05 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 2
Регистрация: 6.3.2011

Репутация: нет
Всего: нет

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

Код



#include <stdio.h>
#include <stdlib.h>

 struct NODE
    {
        char letter;
        struct NODE *next;
        struct NODE *back;
    };

    typedef struct NODE Node;
    typedef Node *node;
    

    node createlist(FILE *filename)

{
    node current,previous;
    
    current = malloc (sizeof(node));
    current->next = malloc (sizeof(node));
    current->back=NULL;


    while(current->letter!=EOF)
    
        {
        
        current->letter = getc(filename);
        previous = current;
        current = previous->next;
        current->back = previous;
            if (current->letter!=EOF)
            current->next = malloc (sizeof(node));
            else        
            current->next = NULL;
        }

        

            return current;    }

        
    

void print(node list)

{ 
    while(list->back!=NULL)
    
        {    
        putchar(list->letter);
        list=list->back;    }


        
    }

    
    



int main(int argc, char *argv[])
{
    FILE *name;
    name = fopen("data.txt","r");
    node list = createlist(name);
    print(list);

    


 
    return 0;
}

bsa

Дата 27.9.2012, 21:38 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Модератор
Сообщений: 9185
Регистрация: 6.4.2006
Где: Москва, Россия

Репутация: 85
Всего: 196

kazansky, учись пользоваться отладчиком. Поверь мне, это очень просто smile

Начни с установки точки останова.

--------------------

Правильно заданный вопрос - половина ответа
Отзывы о работе модератора.

feodorv

Дата 28.9.2012, 03:21 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 2214
Регистрация: 30.7.2011

Репутация: 12
Всего: 45

Цитата(kazansky @ 27.9.2012, 21:05

)

while(current->letter!=EOF)

Здесь current->letter ещё не инициализирована.

Не нужно заранее заказывать память для current или current->next, а делать это в подходящий момент.

Цитата(kazansky @ 27.9.2012, 21:05

)

и выводить его на экран

В обратном порядке, перевёрнутый?

Цитата(bsa @ 27.9.2012, 22:38

)

учись пользоваться отладчиком.

Присоединяюсь smile

--------------------

Напильник, велосипед, грабли и костыли - основные инструменты программиста...

1 Пользователей читают эту тему (1 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Для новичков \| Следующая тема »