0

Hello,
I really don't know if this is more of a Mac of C++ question but.. I am doing a loading of a file with

short CSVload (char * filename) //called from main
{
//openning a file and checking if it exists
    FILE * datafile = NULL;
    if ((datafile=fopen (filename, "r"))==NULL)
    {
        printf ("\nError - no file found");
        return -1;
    }
//finding size of the file
    fseek (datafile,0,SEEK_END);
    int filesize = ftell (datafile);
    rewind (datafile);  
//allocating a string of a size of a file
 char * data;
    data = new char (filesize+2); //+2 just to be safe
//reading file (I am using getc but I simplified for this post) 
 fscanf (datafile,"%s", data);
//closing file
fclose (datafile);
//printing the file
 printf ("\nThe data is:\n%s",data);
//deleting data
delete [] data;
//returning success
    return 0;
}

and the program seems to be freezing every time I call this function.. I have asked somebody why this code doesn't work on OS 9.2 on G3 and the answer was that maybe I am allocating onto the system memory since OS9.2 doesn't protect it's memory like win2k or redhat 9.0 linux (my other systems). My compiler is a very old Code Warrior 7.0 Gold Edition (I usually don't use Mac's, nor this version of CW (I am only doing mac now because of my internship)).
If you can, please help,
Thank you in advance,
Ilya
:sad:

Edited by Dani: Formatting fixed

2
Contributors
1
Reply
3
Views
14 Years
Discussion Span
Last Post by subtronic
0

Alright, first of all... man you've really mixed C with C++ :) If you're going to use C you ought to use malloc() and free() and not new and elete. Anyway, try: char *data = (char *)malloc(sizeof(char) * (filesize + 2)); fscanf("%s",&data) (that's off the top of my head). Seriously though, use malloc and free from now on!!!!!

Sorry I really hate C++ and when it's mixed in with C is makes my heart frown

Edited by deceptikon: Fixed formatting

This topic has been dead for over six months. Start a new discussion instead.
Have something to contribute to this discussion? Please be thoughtful, detailed and courteous, and be sure to adhere to our posting rules.