Thread: Saving linked list in file

You're not allocating space for any of these things
- struct Link
- struct Frame
- struct Frame.name
- struct Frame.path

Code:

struct Frame *temp = malloc( sizeof(*temp) );
if ( temp ) {
    temp->name = malloc(MAX_NAME_SIZE);
    temp->path = malloc(MAX_PATH_SIZE);
    fscanf(pfile, "%s %s %d", temp->name, temp->path, &temp->duration);
    // now append temp to your list, by allocating a link and fixing up the frame/prev/next pointers.
}

Originally Posted by Yoni Goikhman

Code:

int writeListTo,File(const link_t *list);
int readListFromFile(const link_t *list);


int readListFromFile(const link_t *list){
    FILE *pfile = fopen("s.txt", "r");
    fseek(pfile, SEEK_SET, 0);
    if (pfile != NULL) {
        while (!feof(pfile)) {
            fscanf(pfile, "%s %s %d", &list->frame->name, &list->frame->path, &list->frame->duration);
                        
        }
        fclose(pfile);
        return 1;
    }
    else{
        printf("Error opening file");
        fclose(pfile);
    }
}

Don't work with a filepointer before you check it.
This means, do 'fseek' after check 'if (pfile != NULL)'.
Second, you don't need 'fseek', because if you open a file for reading, the position for read will be automaticly set at the beginning of the file.
Your function returns an 'int'.
If the file was opened without problems, you return '1' after closing the file.
But if the file was not opened, you output a error message (on stdout), close a file that was never opened and then? What returns the function?
All in all, you read the information allways in the same structure. At the end you have only one 'frame_t' with valid values.

Originally Posted by Yoni Goikhman

Code:

int writeListToFile(const link_t *list) {
    int c = 0;
    FILE *pfile = fopen("SavedList.txt", "w");
    if (pfile != NULL) {
        for (; list != NULL; list = list->next) {
            fprintf(pfile, "%s %s %d ", list->frame->name, list->frame->path, list->frame->duration);
        }
    }
    fclose(pfile);
    return 1;
}

You write all information in one line. There is no linefeed in your 'printf'.
I think this will be difficult to read in later (especially, your function 'readListFromFile' will not handle this format).
If the file was not opened correctly, you close a NULL pointer.
The function returns allways 1. Normally, the returned value reflect the success or fail of the function.