Thread: Need a bit of help with a default

hey im writing a program for a currency converter but my default doesnt seem to be working the way i'd like it to... i no its only something small (i hope) but if anyone could help it would be much appreciated...

i started with a while loop and i have my main menu with the choices 1-5 and then a switch statement with everything else in it... and the very end i have a default saying "please enter only number 1-5"
BUT when the user choose 5 it says "thank you for using the currency converter" and then i want it to end but instead my default statement reappears...


any ideas anyone?

case 5:
{
printf("Thank you for using Orla's Currency Converter!\n\n");
printf("Have a nice day!");
}
default:
printf("Please type number 1-5 only");




}//end of switch
}//while end
}//end of main

everything else works fine...only the default at the end...

could it be because of the loop?

my code is really long....wat are tags?

#include<stdio.h>
main()
{
int choice;
int num;
float euro;
float sterling;
float krone;
float yen;
float doll;

printf("\n\n\nWELCOME TO ORLA'S CURRENCY CONVERTER!\n\n");

while (num >=1 && num <=5);
{

printf("Please select the number of the country which currency you wish to use.\n\n");
printf("1. Britain\n");
printf("2. Denmark\n");
printf("3. Japan\n");
printf("4. Usa\n"); // Main menu selection
printf("5. Exit Program\n\n");

rewind(stdin);
scanf("%d", &num);
choice=0;
switch (num)
{
case 1:
//If Britain is picked then this will be displayed
printf("You have chosen Britain.\n\n");
printf("Please select an option:\n\n 1.Euro to Sterling\n\n 2.Sterling to Euro\n\n"); // will display the 2 options

rewind(stdin);
scanf("%d", &num);

while (choice==0)
{
if (num ==1)
{
printf("You have chosen Euro to Sterling\n");
printf("\nPlease enter the amount which you would like to convert:\n\n");

scanf("%f", &euro);
sterling = euro*.70;
printf("The amount in Sterling is £%5.2f\n\n\n",sterling);
choice =1;
}

else if (num ==2)
{
printf("You have chosen Sterling to Euro\n");
printf("\nPlease enter the amount which you would like to convert:\n\n");

scanf("%f", &sterling);
euro = sterling*1.42;

printf("The amount in euro is %5.2f\n\n\n",euro);
choice=2: }
else
{
printf("Error!\nPlease press 1 or 2 only!\n\n");
rewind(stdin);
scanf("%d",&num); }
}
break;

#include<stdio.h>
main()
{
int choice;
int num;
float euro;
float sterling;
float krone;
float yen;
float doll;

printf("\n\n\nWELCOME TO ORLA'S CURRENCY CONVERTER!\n\n");

while (num >=1 && num <=5);
{

printf("Please select the number of the country which currency you wish to use.\n\n");
printf("1. Britain\n");
printf("2. Denmark\n");
printf("3. Japan\n");
printf("4. Usa\n"); // Main menu selection
printf("5. Exit Program\n\n");

rewind(stdin);
scanf("%d", &num);
choice=0;
switch (num)
{
case 1:
//If Britain is picked then this will be displayed
printf("You have chosen Britain.\n\n");
printf("Please select an option:\n\n 1.Euro to Sterling\n\n 2.Sterling to Euro\n\n"); // will display the 2 options

rewind(stdin);
scanf("%d", &num);

while (choice==0)
{
if (num ==1)
{
printf("You have chosen Euro to Sterling\n");
printf("\nPlease enter the amount which you would like to convert:\n\n");

scanf("%f", &euro);
sterling = euro*.70;
printf("The amount in Sterling is £%5.2f\n\n\n",sterling);
choice =1;
}

else if (num ==2)
{
printf("You have chosen Sterling to Euro\n");
printf("\nPlease enter the amount which you would like to convert:\n\n");

scanf("%f", &sterling);
euro = sterling*1.42;

printf("The amount in euro is %5.2f\n\n\n",euro);
choice=2: }
else
{
printf("Error!\nPlease press 1 or 2 only!\n\n");
rewind(stdin);
scanf("%d",&num); }
}
break;

then the next 3 case statements are the same

untill the 5th

case 5:
{
printf("Thank you for using Orla's Currency Converter!\n\n");
printf("Have a nice day!");
}
default:
printf("Please type number 1-5 only");




}//end of switch
}//while end
}//end of main

Originally Posted by itsme86

You need to add a break at the end of your last case statement (before the default). Without the break you'll fall through to the next case.

thanks so much.... that fixed it

i knew it was only something small

You have clearly not done what quzah told you. Put [/code] at the end of your code, and [code] before it.