Thread: Reading data from Text File

I am trying to read records from a text file. What is the best way to do this? Here is what I did in my program:

fgets(sentence,500,myfile);

sscanf(sentence,"%s%s%s%s%s%s%s%s",tempId,tempPric e,tempDate,myOffice,tempSold,tempCon,tempBed,tempA dd);

I used fgets() to get the target line of text from the text file and save it in my array "sentence". Then i use sscanf to get each piece of data, the problem is that this only works for about 4 of the data members, then it skips one and records the rest. Is this some sort of input overflow? I can't figure out how to get this to work. Is this even the best way to read a text file?

Thanks, but my code is really long and I really only need to know if anyone knows a better way to read data from text file and why sscanf would skip data. I think it has something to do with overflow.

Check sscanf() return value?
Post the relevant code. where's the input file? post the sample.

Originally Posted by DarrinTE

Thanks, but my code is really long and I really only need to know if anyone knows a better way to read data from text file and why sscanf would skip data. I think it has something to do with overflow.

We only need the short code section where you're intializing the variables and calling scanf(), not the whole program.

The request for some sample lines from the file so we could see how it's formatted.

fscanf() and sscanf() are perfectly fine ways to read formatted data from a file. But they are very picky about correct formatting and target variable types.

Frankly, if the 1 line you did post is accurate... I think it has to do with missing commas in your variable list...

Code:

sscanf(sentence,"%s%s%s%s%s%s%s%s",tempId,tempPric, e,tempDate,myOffice,tempSold,tempCon,tempBed,tempA, dd);

You're overthinking again, CommonTater. I'm sure "tempPric e" was supposed to be "tempPrice", and "tempA dd" was supposed to be "tempAdd". Counting the number of formatters and number parameters. Plus the variable names now make sense.

I'm surprised the compiler let that pass.

Originally Posted by nonoob

You're overthinking again, CommonTater.

Sigh... seeing your version, I think you're right....

I'm sure "tempPric e" was supposed to be "tempPrice", and "tempA dd" was supposed to be "tempAdd". Counting the number of formatters and number parameters. Plus the variable names now make sense.

Your answer makes a lot more sense than mine...

I'm surprised the compiler let that pass.

As am I...

On programming forums, you always need to put your code, between code tags. Just go into the advanced editing window, highlight your code, and click on the # icon top center right on the top bar of the window.

Now your code will be kept looking like code, and much easier for everyone to read/study.

ALWAYS use code tags around your code.