error in the scanf in the do-while loop

[ehab mohamed]

#include<stdio.h>
void main()
{
int i, n, month, day, calcday, calcmonth;
char flag='y';
char weekday[][10]={"Friday","Saturday","Sunday","Monday","Tuesday", "Wednesday","Thursday"};
int usermonth[]={31,28,31,30,31,30,31,31,30,31,30,31};
do
{
printf("Enter the date (Day, Month): ");
scanf("%d %d",&day ,&month);
calcday=day-1;
calcmonth=month-1;
for(i=0;i<calcmonth;i++)
{
calcday+=usermonth[i];
}
n=calcday%7;

printf("%d/%d/2010 is %s \n",day, month, weekday[n]);

printf("Do you want to continue (y/n)? ");

scanf("%c",&flag);
}

while(flag=='y');

}

[newb16]

Reproduced.
Well, if the question was "what to do" it depends on what do you need to achieve. If it doesn't matter, i'd continue with scanf("%c") till I get either 'y' or 'n'.

[gaurav kumar]

use getch() instead of scanf() it will work fine

[ehab mohamed]

i can't understand you mr.newb16
i want in each time scanf for the flag to know if it complete in the loop or get out

[ehab mohamed]

can u give me an example mr.gaurav kumar

[ehab mohamed]

i try it mr.gaurav kumar
and it still wrong
nothing change

[newb16]

If scanf resulted in 'flag' that is not 'y' nor 'n', continue calling scanf until you get the desired letter.

[donbock]

Change "while(flag=='y')" to "while(flag!='n')".
(This is just a restatement of newb16's suggestion.)

[newb16]

This will not work - it will receive \r left after previous scanf and will never quit. It need to discard it in a separate loop.

[ehab mohamed]

how mr.newb16 ??

[BillyTKid]

change

char flag='y'; -> char flag[2];
scanf("%c",&flag); -> fgets( flag,2,stdin );
while(flag=='y'); -> while(*flag=='y');