File won't write.....

[Diamonddrago]

I wanted this module to modify data on a file, but it doesn't seem to write into the file at all...pls help....

void ModEmp(void)
{       clrscr();

	int Target,EmpID,Found=0;
	char check,EmpName[40],DOA[9],DOP[9],DOB[9],Mstat;
	FILE *mem,*pay,*temp;

	temp=fopen("E:\\Proj\\Temp.dat","w");
	if((mem=fopen("E:\\Proj\\Member.dat","r"))==NULL)
	{	printf("File is Empty!");
		sleep(2);
	}

	else
	{	printf("Please Enter Employee ID to Modify	:	");
		fflush(stdin);
		scanf("%i",&Target);

		while (!feof(mem))
		{	fscanf(mem,"%i %[^/]%*c %[^/]%*c %[^/]%*c %[^/]%*c %c",&EmpID,EmpName,DOA,DOP,DOB,&Mstat);
			if(feof(mem))
			{	break;
			}
 //				if(Target!=EmpID)
					fprintf(temp,"%i %s/ %s/ %s/ %s/ %c\n",EmpID,EmpName,DOA,DOP,DOB,Mstat);
				    //	Found=0;

	   //		else
				Found=1;
				printf("\nEmployee ID			:	%04i\n",EmpID);
				printf("\nEmployee Name			:	%s\n",EmpName);
				printf("\nDate of Appoinment		:	%s\n",DOA);
				printf("\nDate of Probation Period	:	%s\n",DOP);
				printf("\nDate of Birth			:	%s\n",DOB);
				printf("\nMaritial Status		:	%c\n",Mstat);
	   //		}

		}

		if(Found==0)
		{	printf("Record not Found\n");
		}

		else if(Found==1)
		{	printf("\nNew Employee Name		:	");
			scanf("%s",EmpName);
			printf("\nNew Date of Appointment	:	");
			scanf("%s",DOA);
			gets(DOA);
			printf("\nNew Date of Probation Period	:	");
			scanf("%s",DOP);
			printf("\nNew Date of Birth		:	");
			scanf("%s",DOB);
			printf("\nNew Marital Status		:	");
			scanf("%c",&Mstat);

			fscanf(temp,"%i %[^/]%*c %[^/]%*c %[^/]%*c %[^/]%*c %c",&EmpID,EmpName,DOA,DOP,DOB,&Mstat);
		}
			fflush(stdin);
			fclose(mem);
			fclose(temp);
			remove("E:\\Proj\\Member.dat");
			rename("E:\\Proj\\Temp.dat","E:\\Proj\\Member.dat");
			printf("\nRecords Edited");
			sleep(2);

	}
	Employeemgt();

}

[hk_mp5kpdw]

#1.

fflush(stdin);

fflush is undefined for input streams.

#2.

while (!feof(mem))
{
    fscanf(mem,"%i %[^/]%*c %[^/]%*c %[^/]%*c %[^/]%*c %c", &EmpID, EmpName, DOA, DOP, DOB, &Mstat);
    if(feof(mem))
    {
        break;
    }

You manage to get around the problems using feof to control a loop by using a second test for feof after the read. Usually the better solution is to test the read operation itself within the conditional part of the while loop to see how many values were read and seeing if this matches your expectations:

while (fscanf(mem,"%i %[^/]%*c %[^/]%*c %[^/]%*c %[^/]%*c %c",&EmpID,EmpName,DOA,DOP,DOB,&Mstat) == 6)
{

#3. What's with the "//"? Are those meant to be there because it looks like some of them should not. You also have a place where it looks like you'd want to put in a bracket "{" (after an else perhaps?).

#4.

else if(Found==1)
{

    ...

    fscanf(temp,"%i %[^/]%*c %[^/]%*c %[^/]%*c %[^/]%*c %c",&EmpID,EmpName,DOA,DOP,DOB,&Mstat);
}

temp is open for writing. This is the place where you need to be using fprintf to write to temp, not fscanf trying to read from temp.

#5.

printf("\nNew Date of Appointment	:	");
scanf("%s",DOA);
gets(DOA);

Huh? First off, there are better choices than gets. Second, why are you reading into DOA twice?

May be other issues... I don't know.