Thread: Line matching

An important element of this is: how much of a role can you play in the structure of the file? It sounds as if the answer is "none" because it is output by some other program not written by you, but I just want to check.

Originally Posted by KBriggs

I though about using fgets to read in each line, but I have never worked with strings like that so I wouldn't know how to test the result properly. Is there a library function that can tell you if everything in a given string is a number with no decimal place?

Very likely, using fgets or fread is the way to go. I would not bother looking for some special library to do this -- learning to implement it won't be any easier than just learning to parse a string, which if you have any desire to use the C language for programming anything, you should really get comfortable with strings.

One thing I have done for tasks like this is to write a custom "get line" type function using fread:

Code:

int fileline_base (int fp, char *line) {
      int c=0;
      char byte;
      while (fread(&byte, 1, 1, fp) {
           if (byte == ' ')   etc [ character by character processing ]
           line[c] = byte; c++;
      }
      line[c] = '\0';  /* must */
      return (some informative value, eg, the type of line)
}

I called this "fileline_base" because it is possible you will want to have different functions to process different kinds of lines *if* you can predict what kind of line is coming next based on the file structure. Otherwise, you could use fileline_base to get the line, but use the character by character processing to identify the "kind" of line, and pass it on to (eg) fileline_timestamp, which might break it up and put appropriate details into a struct.

There's a lot of useful things that working byte by byte makes possible. You could use fgets and then process the line the same way in a loop, but it is probably more efficient to do it on the initial read.

You could also use fgets() with sscanf(). That would allow you to distinguish one kind of line from another -- if the first sscanf fails, move on to the next one. That will be much less prone to error (and more flexible) than if you just fscanf the file.

Is there something I have to link at compile time to include sscanf? I #included <stdio.h> but I get an error saying:

/tmp/ccuKnQDb.o: In function `countTimeSteps':
chargestate.c.text+0x33): undefined reference to `sccanf'
collect2: ld returned 1 exit status

EDIT: LOL, a sad face in an error message


Also, will this work? I am unsure of my error-checking:

Code:

int countTimeSteps(FILE *input)
{
  char *line;
  int check,x,N_timeSteps;
  N_timeSteps = 0;
  
  while (fgets(line, MAXNUM, input) != NULL)
  {
    check = sccanf(input,"%d",&x);
    if (check != 0 && check != EOF)
    {
      N_timeSteps++;
    }
  }
  return N_timeSteps;
}

Spell it correctly: sscanf.

Remember to check spelling if you get an odd linker error like this.

Heh, silly me

No, it doesn't work - it gives me 0, and I get a warning:

chargestate.c: In function ‘countTimeSteps’:
chargestate.c:15: warning: passing argument 1 of ‘sscanf’ from incompatible pointer type


Which I don't know how to intepret

EDIT: it is reading in the lines, because adding a puts(line) prints the output file just fine...

Originally Posted by KBriggs

right now, the code will count any line that starts with an integer. Is there a way to specify that it count only lines that contain ONLY a single integer, rather than lines the simply begin with one? For the current case it doesn't matter, but it would help make the code a little more robust.

By "integer" you mean one number (eg, 234123) and not a single digit? Because a single digit will be simple.

Otherwise, depending on what else could be in the string, you could add a second parameter to sscanf and check the number of items returned.


BTW, you can use

Code:

sscanf("%*d");

The * means ignore, so you don't need x. Since you are not keeping the number, you could also test this mo' better:

Code:

if ((line[0] >= '0') && (line[0] <= '9'))

The ASCII values of digits are guaranteed to be sequential.