Simple Multi-Session Calculator: Switch Case and While Loop Problem in C

Hello Everyone,

I have a problem with my simple operations calculator code (using C, in Code::Blocks). I am required to use a while loop statement so the user can execute multiple step operations without re-opening the program. When I launch the program, I get through the first session fine, but when I'm on the second session, when entering the two operands and press enter (to calculate), it just gives me the return and say press any key to continue (exit). You could run it, if it helps anything. I am so close, I've been working on this problem for 1.5 days, I'm sure it is something simple, please help! Thanks!:biggrin:

Here is the code:

Code:

# include <stdio.h> int main() { int num1, num2; char op; int finished = 0; printf("Simple Operations Calculator\n"); printf(" \n"); printf("Enter your desired operation (+, -, *, /): "); scanf("%c", &op); printf("Enter your two desired numbers: "); scanf("%d%d", &num1, &num2); switch(op) { case '+': printf("num1 + num2 = %.2d\n", num1+num2); break; case '-': printf("num1 - num2 = %.2d\n", num1-num2); break; case '*': printf("num1 * num2 = %.2d\n", num1*num2); break; case '/': printf("num1 / num2 = %.2d\n", num1/num2); break; default: printf("Error! That is not an operation option."); break; } while(!finished){ printf("Enter your desired operation (+, -, *, /): "); scanf("%c", &op); getchar(); printf("Enter your two desired numbers: "); scanf("%d%d", &num1, &num2); getchar(); return 0; } }

your loop should include your calculation code... how else will it be repeated?

So put the switch case within the while loop as well?

Quote:

Originally Posted by vart

your loop should include your calculation code... how else will it be repeated?

I put the switch case in the while loop, but when I enter the two desired operands, it gives me the default, "Error! That is not an operation option!"

Code:

# include <stdio.h> int main() { int num1, num2; char op; int finished = 0; printf("Simple Operations Calculator\n"); printf(" \n"); printf("Enter your desired operation (+, -, *, /): "); scanf("%c", &op); printf("Enter your two desired numbers: "); scanf("%d%d", &num1, &num2); switch(op) { case '+': printf("num1 + num2 = %.2d\n", num1+num2); break; case '-': printf("num1 - num2 = %.2d\n", num1-num2); break; case '*': printf("num1 * num2 = %.2d\n", num1*num2); break; case '/': printf("num1 / num2 = %.2d\n", num1/num2); break; default: printf("Error! That is not an operation option."); break; } while(!finished){ printf("Enter your desired operation (+, -, *, /): "); scanf("%c", &op); getchar(); printf("Enter your two desired numbers: "); scanf("%d%d", &num1, &num2); switch(op) { case '+': printf("num1 + num2 = %.2d\n", num1+num2); break; case '-': printf("num1 - num2 = %.2d\n", num1-num2); break; case '*': printf("num1 * num2 = %.2d\n", num1*num2); break; case '/': printf("num1 / num2 = %.2d\n", num1/num2); break; default: printf("Error! That is not an operation option."); break; } getchar(); return 0; } }

Quote:

Originally Posted by critixal

I put the switch case in the while loop, but when I enter the two desired operands, it gives me the default, "Error! That is not an operation option!"

Code:

# include <stdio.h> int main() { int num1, num2; char op; int finished = 0; printf("Simple Operations Calculator\n"); printf(" \n"); printf("Enter your desired operation (+, -, *, /): "); scanf("%c", &op); printf("Enter your two desired numbers: "); scanf("%d%d", &num1, &num2); switch(op) { case '+': printf("num1 + num2 = %.2d\n", num1+num2); break; case '-': printf("num1 - num2 = %.2d\n", num1-num2); break; case '*': printf("num1 * num2 = %.2d\n", num1*num2); break; case '/': printf("num1 / num2 = %.2d\n", num1/num2); break; default: printf("Error! That is not an operation option."); break; } while(!finished){ printf("Enter your desired operation (+, -, *, /): "); scanf("%c", &op); getchar(); printf("Enter your two desired numbers: "); scanf("%d%d", &num1, &num2); switch(op) { case '+': printf("num1 + num2 = %.2d\n", num1+num2); break; case '-': printf("num1 - num2 = %.2d\n", num1-num2); break; case '*': printf("num1 * num2 = %.2d\n", num1*num2); break; case '/': printf("num1 / num2 = %.2d\n", num1/num2); break; default: printf("Error! That is not an operation option."); break; } getchar(); return 0; } }

Come on, could anyone help me?

The second time there will be a \n in the input buffer and the scanf("%c", &op ) will read that as op.
simple fix: put getchar(); after scanf("%d%d", &num1, &num2);

Kurt

Quote:

Originally Posted by ZuK

The second time there will be a \n in the input buffer and the scanf("%c", &op ) will read that as op.
simple fix: put getchar(); after scanf("%d%d", &num1, &num2);

Kurt

Thanks Kurt! I have another question.
The two sessions work, but after the second, it does not allow me to do another session.
I want to be able to do as many sessions as I want.
Do you know the solution to this?

Quote:

Originally Posted by critixal

Do you know the solution to this?

Don't return from inside the loop.
If you'd indent the code better the mistake would be obvious.
Kurt

Quote:

Originally Posted by ZuK

Don't return from inside the loop.
If you'd indent the code better the mistake would be obvious.
Kurt

Sorry about that, I just started C programming, well programming in general, on Thursday. I probably wouldn't have know that anyway. but I will try to improve my indenting skills for programming. Thanks! :)