C help required

Quote:

---

Originally Posted by MK27

It's easier to pinpoint a problem in this procedure if you use fgets() to get a line from the file and sscanf() on the line to extract your data. So for example:

Code:

---

char line[64];  /* should be long enough for this, right? */
float data;
FILE *fh;  /* obviously this should be fopend, etc. */
while ((fgets(line,8,8,fh))) {
    sscanf(line, "%f", &data);
}

---

The outer layer of brackets is a truth test the same as this:

Code:

---

while (fgets(line,8,8,fh)!=NULL) {

---

Since fgets returns NULL at end-of-file.

Once you have everything worked out you can always change it back to the straight fscanf().

For counting the lines add this to the loop:

Code:

---

sum++;

---

---

Yes, the maxiumum length of a file will be 60 rows.
However, my program will not compile at this section

Code:

---

while (fgets(line,8,8,fh)!=NULL) {

---

there are too many arguments to function fgets.

Also using

Code:

---

while (fscanf(fp,"%*[^\n]\n",& number)!=EOF)
sum++;

---

I am sure that it recognises the correct number of lines in the file, but it does not store it in 'number' correctly, nor does it increment the sum either.

I think you've gotten turned around on what fscanf does. For one thing, the * suppresses assignment, so by definition nothing will get stored in number. (This is probably a good thing, since it will try to store a bunch of characters in that address, as opposed to a float, which I think is what number was.) Secondly sum is indeed getting incremented unless your file is empty. (No really, I just tried it here on this machine.)

Quote:

---

Originally Posted by rmc

there are too many arguments to function fgets.

---

Whoops! Sorry sorry sorry... I was thinking about fread(), which takes a number of chunks and a size for each chunk. fgets() just takes a size, so that should be 64 (it only reads up to 63 tho, because it leaves room for a null terminator).

fscanf() returns the number of successful "assignments" (ie, the number of variables written into), so you could try a check on that:

Code:

---

int x;
x=fscanf(.....
printf("fscanf returned %d\n",x);

---

This will tell you whether "&number" got written into. If the answer is 0, your template does not work with your material. That's why I'm recommending fgets with sscanf, because it is FAR EASIER to make fgets work than fscanf, and you won't have to mess around with \n in the template (which I could be wrong, but I don't think that works anyway).