something wrong with my program when I end the file....

[jimmy_walker]

I need help...I can create the file...write to the file...but when I end the file...it gives me endless cursors until I close the window...when I go to the read program it it gives me endless replays of the last item listed in the file.......

#include <stdio.h>
int main()
{ 
   int StudentSSN;     /* Social Security Number */
   char StudentName [ 30 ]; /* Name up to 30 characters */
   double StudentAverage;  /* Average Grade */
   FILE *cfPtr;     /* cfPtr = clients.dat file pointer */
   /* fopen opens file. Exit program if unable to create file  */
   if ( ( cfPtr = fopen( "student.dat", "w" ) ) == NULL ) {
      printf( "File could not be opened\n" );
   } /* end if */
   else { 
      printf( "Enter the StudentSSN,StudentName, and STudentAverage.\n" );
      printf( "Enter EOF to end input.\n" );
      printf( "? " );
      scanf( "%d%s%lf", &StudentSSN, StudentName, &StudentAverage );
      /* write Student SSN, Student Name and Student Average into file with fprintf */
      while ( !feof( stdin ) ) { 
         fprintf( cfPtr, "%d %s %.2f\n", StudentSSN, StudentName, StudentAverage );
         printf( "? " );
         scanf( "%d%s%lf", &StudentSSN, StudentName, &StudentAverage );
      } /* end while */
      
      fclose( cfPtr ); /* fclose closes file */
   } /* end else */
   return 0; /* indicates successful termination */
} /* end main */

[hk_mp5kpdw]

How do you "end the file"? What do you type at the prompt to generate EOF?

I'm guessing that your EOF signal - whatever it is - is not being evaluated as true by your feof call. Instead, whatever you've done instead causes the stream to go into an error state because the scanf call fails (it cannot interpret what you've entered). Since the feof call still returns false you continually enter the loop and the scanf keeps failing but you still keep writing to the file in the meantime.

I'd suggest changing the loop do something more like this:

while (  3 ==  scanf( "%d%s%lf", &StudentSSN, StudentName, &StudentAverage ) )
{
    fprintf( cfPtr, "%d %s %.2f\n", StudentSSN, StudentName, StudentAverage );
} /* end while */

That will continue to loop until an error condition happens, either EOF or failure to convert the proper number of elements.