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

Quote:

---

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?

Quote:

---

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...