printing from file problems

[Conor1992]

void ld_apps()
{
     FILE *fp;
     
      fp = fopen("G:/Uniwork/Semester2/ProceduralProgramming/Assignment/Appointment.txt", "rt");
        if (fp == NULL)  {
               printf("\nUnable to Open Input File\n");
               exit(EXIT_FAILURE);
               }
               
        else  {
              printf("File Open\n\n The Appointments are Shown Below\n\n");
              }
              
            while  ( !feof ( fp ))
            
              {  
            Appointments++  ;     
            printf("\n\nCurrent Number of Appointments: %d", Appointments);
            
              {
            fscanf(fp, "%s", PrAppArray[0].name);
            printf("\nName: %s", PrAppArray[0].name);
            
            fscanf(fp, "%s", PrAppArray[0].surname);
            printf("\nSurname: %s", PrAppArray[0].surname);
            
            fscanf(fp, "%f", &PrAppArray[0].starttime);
            printf("\nStart Time: %.2f", PrAppArray[0].starttime);
            
            fscanf(fp, "%f", &PrAppArray[0].endtime);
            printf("\nEnd Time: %.2f", PrAppArray[0].endtime);
            
            fscanf(fp, "%s", PrAppArray[0].place);
            printf("\nPlace: %s", PrAppArray[0].place);
            
            fscanf(fp, "%s", PrAppArray[0].whom);
            printf("\nWhom: %s", PrAppArray[0].whom);
            
            fscanf(fp, "%s", PrAppArray[0].date);
            printf("\nPlace: %s", PrAppArray[0].date);
            
            fscanf(fp, "%s", &PrAppArray[0].priority);
            printf("\nPriority: %s", PrAppArray[0].priority); 
         
     }  
     }       
      printf("\n");
      subm_app();
      return;
   
} // finish function

Basically this is the code ive used to pull the information from the file so that it writes all the records to the screen, that are in the text file.

but when it does, it prints every record twice, very frustrating.

could anyone please help?

i also have the code also which writes the information into the file, if its needed please say.

[anduril462]

You should never use feof to control a loop. Read why here. That should only cause the last record to get printed twice, not every record, but it's still a problem. Also, you have an extra set of braces between the printf in the loop and the end of the loop. It serves no purpose.

Does it print the two copies "collated"? That is, does it print A, B, C, A, B, C or A, A, B, B, C, C? Some sample output might be helpful. The writing code might be helpful too. Are you sure there's only one copy of each record written to the file? I see nothing in here that screams "print two copies".

[Conor1992]

My text file had a bit of a problem, ive now deleted it and remade the text file.

except i still face a problem but not the same one.

it prints A B C C. so the last record is printed twice now, the rest print in order.

even when i add a record C would become a single record and D would become double.

i.e. A B C C, add 1 record now becomes A B C D D

[anduril462]

Clearly you didn't read my first paragraph thoroughly or the link I provided (click the red text).

[Conor1992]

i kind of dont get it, so im ment to change the feof part to somethin like "while (fgets(buf, sizeof(buf), fp) != NULL)"

but to be 100% honest im no good with programing and wouldnt be albe to put that into my own progam

[CommonTater]

but to be 100% honest im no good with programing and wouldnt be albe to put that into my own progam

And you never will so long as you keep telling yourself that.

Half of success is attitude. Enter a new venture with a defeatist attitude and even with the best products, tools and people, it will fail.
On the other hand, go into it with an attitude that says "I can do this!" and just watch what happens.

[anduril462]

To follow up a little with Tater, yes, your attitude is crappy. There are some people who truly don't get it, but you do get it. Hell, you wrote the right answer in your post. You need to replace the while (!feof) that I said was wrong with that while (fgets) bit. Only you need to define buf, and change your fscanf to be sscanf. Read the docs on sscanf for usage details. You will have to change very little.