Breaking out of the loop

When the user enters a correct ISO setting , he is told that but the program then continues and states that an incorrect value was entered .

Code:

case '2':


fprintf(output,"%s please enter the ISO\n\n" ,name );
fgets(ISO,10,stdin); //Prompt the user for the ISO
p = strchr(ISO, '\n'); // This serves to remove the enter character which fgets registers . It eliminates the effect of \n.
		if ( p ) 
     			*p = 0;
	
				for(j=0;j<32;j++) // This loop will go through the 12 array elements.
				{
 					
					if(strcmp(ISO_VALID[j], ISO)==0)  
					// The strcmp will do a comparison of the string entered with allowed string values stored in the array)
					{

						fprintf(output,"\n\n	This ISO is on the list of possible values !\n\n");
						ISO_done=1;
						ISONumber = atof(&ISO[0]);
						getchar();
						break;
					}


				}
			
		fprintf(output," \n\n  Incorrect ISO value entered . Please try again !");
		fprintf(output, "The program will now return to main menu");
		break;

What should Iuse for it to break out of the loop back to the main menu .

I came up with this solution for the problem . It probably is't optimal but it works .

Code:

case '2':

if (ISO_done==1)
{
fprintf(output,"You have already entered the ISO value");
break;
}
{
fprintf(output,"%s please enter the ISO\n\n" ,name );
fgets(ISO,10,stdin); //Prompt the user for the ISO
p = strchr(ISO, '\n'); // This serves to remove the enter character which fgets registers . It eliminates the effect of \n.
		if ( p ) 
     			*p = 0;
	
				for(j=0;j<32;j++) // This loop will go through the 12 array elements.
				{
 					
					if(strcmp(ISO_VALID[j], ISO)==0)  
					// The strcmp will do a comparison of the string entered with allowed string values stored in the array)
					{

						fprintf(output,"\n\n	This ISO is on the list of possible values !\n\n");
						ISO_done=1;
						ISONumber = atof(&ISO[0]);
						ISO_completed=1;
						getchar();
						break;
					}


				}
		if (ISO_completed==1)
			{
			break;
			}
			{
			fprintf(output,"\n	Incorrect ISO entered ! Please try again\n"); // What happens if an incorrect aperture is entered.
   			
			break;  
			}
		}
                	break;

I have another problem , I'm using atof to convert 2 strings like so :

Code:

ApertureNumber= atof(&guess[0]);
ISONumber = atof(&ISO[0]);

Then the program processes these :

Code:

float exposure;
exposure=(ApertureNumber*ApertureNumber)/(256*ISONumber);
fprintf(output,"the exposure value is %f",exposure);
return 0;

If I understant right atof returns a double , so the above should work , yet I keep getting 0.00000

Originally Posted by ICool

I have another problem , I'm using atof to convert 2 strings like so :

Code:

ApertureNumber= atof(&guess[0]);
ISONumber = atof(&ISO[0]);

Then the program processes these :

Code:

float exposure;
exposure=(ApertureNumber*ApertureNumber)/(256*ISONumber);
fprintf(output,"the exposure value is %f",exposure);
return 0;

If I understant right atof returns a double , so the above should work , yet I keep getting 0.00000

1. Indenting is crap. Fix it.
2. Variables are named in a strange way. Try to be consistent, and consider looking up a naming convention for variables.
3. Why are you passing &guess[0] to it? Why not just guess? Same with ISO? Try changing it.

Code:

if ( p ) 
   *p = 0;

Is that the only thing you want to do if p != 0 or did you forget to use braces?

Originally Posted by cpjust

Code:

if ( p ) 
   *p = 0;

Is that the only thing you want to do if p != 0 or did you forget to use braces?

Just what I've been preaching for ages...