Flushing a line in an array...

I have a quick question...I'm sorting an array with only numbers in it, two per line. I wanna flush the first string of non-numbers and the other value in that line, if there is one of course, whether it was the first or last, and continue on with the sort. I started a while loop...

Code:

char flush (int count, FILE* fp)
{
    char c;
    printf("Line %d is invalid, skipping line.\n", count);
    while (((c = fgetc(fp)) != EOF && (c != '\n')));
    return c;
}

I'm wondering if, at the end I should I make c != '\n' and my count can't equal 2? Would that do it? Because if my count is 0, that should flush the line, or if it's 1, that should flush the line, and if its not already two, it gets sent to this flush function.

Am I trying to add too much to one loop?

Thanks, gator

Ok, so do you know the format of your data in the file? I am afraid I am not following exactly what you are trying to accomplish here.

c needs to be an int if you are trying to read EOF. Why don't you just use fgets+sscanf, and if your sscanf doesn't return 2, to just throw the line away (ignore it, and call fgets again).


Quzah.

Yes sir. I'm reading a text file and placing it into an array. I need to read two consecutive numbers and count those as a line. The first value read that is not a number invalidates that line and I have to toss it, but carry on with the scan. Since it's two per line, if the second value is not a number, I have to toss that line as well, including the number before it. Basically I have to toss the line if either of the two aren't both numbers, print an error, and contine reading the file. Believe it or not, it actually helps a little writing this down. I so wanna add more logic to the end of my flush code, but I don't know if it'll help if the second value in the line of two is not a number.

I'm throwing it away if it's a character, but not if it's the only value left at EOF. Does that sound right? I mean, I don't wanna lose the last value in a file if the file has an odd number index or the EOF is less than the max size.

Ever heard of scanf() and sscanf()?

(Google is your friend)

Originally Posted by CommonTater

Ever heard of scanf() and sscanf()?

(Google is your friend)

Of course...its a flush function from a scanf!!! It's during the scanf!!! What is it looking for during the scanf???

Quzah hit the nail on the head with this one. Take a closer look at what he is telling you to do:

Originally Posted by quzah

c needs to be an int if you are trying to read EOF. Why don't you just use fgets+sscanf, and if your sscanf doesn't return 2, to just throw the line away (ignore it, and call fgets again).

Quzah.

After considering this, take a look here. Play around with this and then post if you have any further questions.

Originally Posted by Cgmgator

Of course...its a flush function from a scanf!!! It's during the scanf!!! What is it looking for during the scanf???

Say for example you're looking for two integers...

Code:

if (fscanf(File,"%d %d",&a,&b) == 2)
  ++count;

If you check the return value of fscanf() you will get the number of successful conversions... anything but 2 would tell you not to increment your array pointers and carry on to the next line.

A variant of scanf(), sscanf() can be used with fgets() to break out a line at a time and then convert from the line...

Code:

do
  { fgets(buff, buffsize,file);
     if (sscanf(buff,"%d %d",&a[count],&b[count]) == 2)
       count++
   } while (! feof(file));

Of course you will need to flesh this out with error checking and bounds checking, but the general concept is there.

Awesome, I'll try it out! Thanks a lot!

You could also alternatively check the return of fgets so you wouldn't have to use feof:

Code:

while(fgets(buff,buffsize,file) != NULL){
     ........
}

Some reading for the OP:
feof FAQ
What is EOF?

Awesome, I'll try it out! Thanks a lot!