Thread: A little stupid problem with my c code.

In a part of my program I am trying to use fscanf to read from a file, that has a format as following

14:37:37.825	0:00:00.020
14:37:37.845	0:00:00.020
14:37:37.865	0:00:00.020
14:37:37.885	0:00:00.020
14:37:37.905	0:00:00.020
..	..

I used the line

while(fscanf(fp,"%2d %c %2d %c %5.3f %2d %c %2d %c %5.3f",&h1,&c1,&h2,&c2,&h3,&h4,&c3,&h5,&c4,&h6)!=E OF)

trying to accomplish the job, but it didn't work. I will be very thankful if someone could suggest me a way out.

Originally Posted by Ironmaniac

In a part of my program I am trying to use fscanf to read from a file, that has a format as following

I used the line trying to accomplish the job, but it didn't work. I will be very thankful if someone could suggest me a way out.

Use fwrite() and fread() instead of fprintf and fscanf() respectively.

Originally Posted by quzah

"Doesn't work" doesn't mean anything.

This is inspiring, thanks.

Code:

fscanf( fp, "%d:%d:%f %d:%d:%f", ... )

Tried this...prints strange values.

Originally Posted by quzah

Also consider using fgets and then sscanf.

Will do that. Thank you.

Thanks chibi.coder. Will try fread() and fwrite().

Originally Posted by Ironmaniac

In a part of my program I am trying to use fscanf to read from a file, that has a format as following

Code:

while(fscanf(fp,"%2d %c %2d %c %5.3f %2d %c %2d %c %5.3f",&h1,&c1,&h2,&c2,&h3,&h4,&c3,&h5,&c4,&h6)!=E OF)

I used the line trying to accomplish the job, but it didn't work. I will be very thankful if someone could suggest me a way out.

1) don't use field sizes in scanf().
2) watch your spacing it's EOF not E OF

Scanf works by pattern matching so you need to create a format string that matches the format of your file...
that is -> number:number:number.number number:number:number.number
so...

Code:

fscanf(fp,"%d:%d:%d.%d %d:%d:%d.%d"...

Originally Posted by Ironmaniac

Thanks chibi.coder. Will try fread() and fwrite().

Don't waste your time. It won't solve your problem.

Originally Posted by quzah

That's a terrible idea.

When in doubt, write a small test case:

Code:

#include<stdio.h>
int main( void )
{
    char *p[] =
    {
        "14:37:37.825     0:00:00.020",
        "14:37:37.845     0:00:00.020",
        "14:37:37.865     0:00:00.020",
        "14:37:37.885     0:00:00.020",
        "14:37:37.905     0:00:00.020",
        NULL,
    };
    int x = 0;
    
    for( x = 0; p[ x ]; x++ )
    {
        int h1 = 0, m1 = 0;
        int h2 = 0, m2 = 0;
        float s1 = 0.0f, s2 = 0.0f;
        int rval = 0;
        
        rval = sscanf( p[ x ], "%d:%d:%f %d:%d:%f",
            &h1, &m1, &s1, &h2, &m2, &s2 );
    
        printf( "string: %s\nrval: %d\n", p[ x ], rval );
        printf( "h1: %d, m1: %d, s1: %f\nh2: %d, m2: %d, s2: %f\n",
            h1, m1, s1, h2, m2, s2 );
    }
    return 0;
}

Quzah.

Thank you Quzah. Thats a nice clean code.

Originally Posted by CommonTater

1) don't use field sizes in scanf().
2) watch your spacing it's EOF not E OF

Scanf works by pattern matching so you need to create a format string that matches the format of your file...
that is -> number:number:number.number number:number:number.number
so...

Code:

fscanf(fp,"%d:%d:%d.%d %d:%d:%d.%d"...

Thank you CommonTater . I tried correcting my program according to your suggestions. I removed the field sizes like you and Quzah pointed out. And it was certainly EOF like you pointed out. Now it still didn't work. I will request you all to please take a look at the following..
Okay here is
1.A chunk of my code after I have it edited by removing the field sizes in the scanf line . And

2.I have a little 7KB file attached to this post that could be used as the input file to this code

Code:

#include<stdio.h>
#include<stdlib.h>
main()
{     
    int h1,h2,h4,h5;
    float h3,h6;
        FILE *fp,*fp1;
    fp=fopen("in.txt","r");
    fp1=fopen("a.txt","w");


                    /*  11:57:25.138  0:00:00.020 */
  while(fscanf(fp,"%d %d %f %d %d %f",&h1,,&h2,&h3,&h4,&h5,&h6)!=EOF)
{


    if(h6>0.02 && h6<=0.1)
            { 
          fprintf(fp1,"%d:%d:%f\n",h1,h2,h3);     
          
}    }






        return(0);
}

So what is your current code and problem then? Be specific!

Code:

#include<stdio.h>
#include<stdlib.h>

main()
{  int n1,n2,n3;
    int h1,h2,h4,h5;
    float h3,h6;


    FILE *fp,*fp1,*fp2,*fp3;
    fp=fopen("in.txt","r");
    fp1=fopen("a.txt","w");
    fp2=fopen("b.txt","w");
    fp3=fopen("c.txt","w");
    n1=n2=n3=0;


       /*      11:57:25.138    0:00:00.020 */
       while(fscanf(fp,"%d:%d:%f %d:%d:%f",&h1,&h2,&h3,&h4,&h5,&h6)!=EOF)
    {
        if(h6>00.020 && h6<=1.00)
        {
        fprintf(fp1,"%d:%d:%5.3f\n",h1,h2,h3);
           n1=n1+1;
        }




        if(h6>1.00 && h6<=2.00)
        {
        fprintf(fp2,"%d:%d:%5.3f\n",h1,h2,h3);
           n2=n2+1;
        }


        if(h5*h6>60)
        {
        fprintf(fp3,"%d:%d:%5.3f\n",h1,h2,h3);
           n3=n3+1;
        }
    }
        fprintf(fp1,"---------  %d",n1);
        fprintf(fp2,"---------  %d",n2);
        fprintf(fp3,"---------  %d",n3);
}

Hello everyone, the above is the code I was looking for. I would like to thank everyone who tried to help me. Especially Quzah and CommonTatter. Both provided working solutions. But I found the one by Quzah appropriate for my case.
There were two problems that I had to figure out, namely
1. Getting scanf the right format to read from the file supplied. This was solved by Quzah and CommonTatter.
2.Initially I had named my input file 21marSV19.txt and that somehow made the program useless. I while experimenting re-named it to in.txt, and suddenly things fell in place on their own. Keep it simple, is my lesson.

And like Quzah said...Doesn't work doesn't mean anything. Cheers!

thank you rags-to-riches for your time and interest.