Thread: goto and Yes or No problem

Originally Posted by zenox_ruiz

Now the loop ends at 20. I made [Y/N] work. But I just need to let "Do you want to continue" reappear after the loop reaches 20.

It seems like you're mixing up the Y/N part in with your "getting the student grades" part

1. Make the program work one time without any kind of Y/N question.
2. Wrap the code with the following:

Code:

char x = 'Y';
while (x == 'Y') {
    /* ... */
    printf("Do you want to continue? [Y/N]");
    scanf(" %c",&x);
}

The part ... should be only the code that does the student grades part. Also the space in " %c" is required.

It worked without the Y/N

Code:

#include<stdio.h>
#include<conio.h>
#include<stdlib.h>
int main()
{
clrscr();
int z[20],zenox;
char x;
for(zenox=0;zenox<20;zenox++)
     {
     printf("Enter the grade for student %d:\n",zenox+1);
     scanf("%d",&z[zenox]);
     if((z[zenox]>100)||(z[zenox]<0))
          {
          printf("Invalid grade\n");
          zenox--;
          }
     if(z[zenox]>=90)
     printf("Excellent\n\n");
     else if(z[zenox]>=80)
     printf("Above Average\n\n");
     else if(z[zenox]>=70)
     printf("Average\n\n");
     else if(z[zenox]>=60)
     printf("Satisfactory\n\n");
     else
     printf("Failure\n\n");
     }
exit (0);
getch ();
return 0;
}

Then by wrapping it, it worked!

Code:

#include<stdio.h>
#include<conio.h>
#include<stdlib.h>
int main()
{
clrscr();
char x ='Y';
while (x=='Y') {
	{
	int z[20],zenox;
	for(zenox=0;zenox<20;zenox++)
		{
		printf("Enter the grade for student %d:\n",zenox+1);
		scanf("%d",&z[zenox]);


		if((z[zenox]>100)||(z[zenox]<0))
			{
			printf("Invalid grade\n");
			zenox--;
			}
		if(z[zenox]>=90)
		printf("Excellent\n\n");
		else if(z[zenox]>=80)
		printf("Above Average\n\n");
		else if(z[zenox]>=70)
		printf("Average\n\n");
		else if(z[zenox]>=60)
		printf("Satisfactory\n\n");
		else
		printf("Failure\n\n");
		}
	}
	printf("Do you want to continue? [Y/N]");
	scanf(" %c",&x);
}
exit(0);
getch ();
return 0;
}

But is there anyway that it could use either Y or y? I tried it this way

Code:

char x ='Y'||'y';
while ((x=='Y')||(x=='y')) {

But it didn't seem to work.

Originally Posted by zenox_ruiz

Code:

char x ='Y'||'y';
while ((x=='Y')||(x=='y')) {

But it didn't seem to work.

Your while condition is correct, but the assignment

char x = 'Y' || 'y';

is like saying this:

char x = (bool)('Y' || 'y');

It means the expression will evaluate to true or false, so treating x as a character from this point on doesn't work as expected.