Thread: annoying validation issue with sscanf

i did something simliar:

Code:

 
if(sscanf(buff,"%s %f",name,&num) != '\n' )

it just prints out

Number of lines: 1
Number of lines: 2
Number of lines: 3
Number of lines: 4

it doesn't seem to skip the blank lines i added 2 lines of text and 2 blank lines

hmm you have a point there.

i got it to work:

Code:

  while(fgets(buff,sizeof(buff),fp)!=NULL)  // read a single line from the file till i get EOF
    {

        if(sscanf(buff,"%s %f",name,&num) != '\n' && sscanf(buff,"%s %f",name,&num) == 2 )  // getting the name and the number from the  buffer which was read from the file
            {
               if(!buff[i] || (!buff[i] && buff[i] == '\n')) continue;
               nooflines++;
               printf("%s %d", "\n Number of lines: ", nooflines);
               i++;
            }
        else
        {
            printf("\nline cannot be printed\n");
            exit(0);
        }
    }

one prob though, for some reason "line cannot be printed even though i have no blank lines etc.

whoops, i actually tried it without that bit but it still prints out "cannot be printed" i have the correct format for sscanf.

Code:

while(fgets(buff,sizeof(buff),fp)!=NULL)  // read a single line from the file till i get EOF
    {
        if(!buff[i] || (!buff[i] && buff[i] == '\n')) continue;
        if(sscanf(buff,"%s %f",name,&num) == 2 )  // getting the name and the number from the  buffer which was read from the file
            {
               nooflines++;
               printf("%s %d", "\n Number of lines: ", nooflines);
               i++;
            }
        else
        {
            printf("\nline cannot be printed\n");
            exit(0);
        }
    }

Works for me with a test.txt file that contains string<space>floating point<newline> lines, what is your input?

blah1 44.3
blah1 44.3

there's two blank lines at the bottom

Number of lines: 1
Number of lines: 2
line cannot be printed <- shouldn't be there :|

Why shouldn't it?

You are reading in each line, and you are using fscanf to read in a string and a double. Of course it's going to fail (not return 2) if you feed it a blank line instead of a string/double pair.

What is it you actually expect to happen? If you want to ignore blank lines, check if the fgets result string only contains a newline, then skip your fscanf if that is the case.

Yes, instad of it printing 'there's an error' i just want it to ignore empty lines are print nothing.

i know what you mean with changing the fgets line, but i cant get it to compile 'operands of != have illegal types 'pointer to char' and 'char'

Code:

    while(fgets(buff,sizeof(buff),fp)!=NULL && fgets(buff,sizeof(buff),fp)!='\n')  // read a single line from the file till i get EOF
    {

        if(sscanf(buff,"%s %f",name,&num) == 2)  // getting the name and the number from the  buffer which was read from the file
            {
                if(!buff[i] || (!buff[i] && buff[i] == '\n')) continue;

               nooflines++;
               printf("%s %d", "\n Number of lines: ", nooflines);
               i++;
            }
        else
        {
            printf("\nline cannot be printed\n");
            exit(0);
        }
    }

Comparing the return value of fgets to '\n' is ridiculous. fgets returns a pointer, either NULL or the pointer passed. The only thing you should be comparing to '\n' is buff[0], ie the first character of the result string.

And you should do this BEFORE, not after the sscanf, because you want to continue to the next line without reading it with sscanf.

you mean?

Code:

    while(fgets(buff,sizeof(buff),fp)!=NULL)  // read a single line from the file till i get EOF
    {
      if(!buff[i] || (!buff[i] && buff[i] == '\n')) continue;

      if(sscanf(buff,"%s %f",name,&num) == 2)
      {

               nooflines++;
               printf("%s %d", "\n Number of lines: ", nooflines);
        }
        else
        {
            printf("\nline cannot be printed\n");
            exit(0);
        }

        i++;

	}

Number of lines: 1
Number of lines: 2
line cannot be printed

No.

Code:

if(!buff[i] || (!buff[i] && buff[i] == '\n')) continue;

what on earth is that supposed to do? For a start, what is i? It makes no sense at all, and will always be false anyway. Why not just do:

Code:

if (buff[0] == '\n')
    continue;

Simple.

ahh so you actually meant buff[0] sorry, i misunderstood. I thought i had to increment the line as i work through the file. thanks.