[Worr]

I am writing a simple printing calculator that is suppose to accept input from a user in the following form: number operator
The calculation will be based on the letter the user inputs (e or E sets the number as the accumulator, t or T is suppose to terminate, a or A is addition, m or M is multiplication, etc.). The problem I am having is that it doesn't matter what number i put in it just prints out 0.00 and the loop never ends. The program should go through once do the calculation based off the operator and then wait for the users next input.

#include <stdio.h>
#include <math.h>

void main (){

	double accum;
	double number;
	char operator;

	scanf ("%lf%c", &number, &operator);
	while (operator != 't' && operator != 'T'){
		if (operator == 'e' || operator == 'E')
			printf ("= %0.2lf\n", accum = number);
		else if (operator == 'a' || operator == 'A')
			accum += number;
		else if (operator == 's' || operator == 'S')
			accum -= number;
		else if (operator == 'm' || operator == 'M')
			accum *= number;
		else if (operator == 'd' || operator == 'D')
			accum /= number;
		else if (operator == 'p' || operator == 'P'){
			accum = pow(accum, number);
		printf ("= %0.2lf\n", accum);}				
		else{		
			printf ("invalid command\n");
			scanf ("%lf%c", &number, &operator);}
		
   }
}

This post has been edited by Worr: 07 October 2009 - 01:31 PM

[Autocrat]

Quote

else
printf ("= %0.2lf\n", accum);
scanf ("%lf%c", &number, &operator);
}

When you have more than one statements, you use braces, so this should be:

	else {
		printf ("= %0.2lf\n", accum);
		scanf ("%lf%c", &number, &operator);
	}
   }

This post has been edited by Autocrat: 07 October 2009 - 10:45 AM

[taylorc8]

is operator a keyword ??

i would imagine you would get an error

[Worr]

taylorc8, on 7 Oct, 2009 - 09:51 AM, said:

is operator a keyword ??

i would imagine you would get an error

It compiles fine. Operator is a character the user inputs and each character represents the type of calculation the program should do. Sorry if this is not what you meant and also thanks for the quick responses guys. I fixed that problem Autocrat posted, but I'm still having the same issue which is the loop doesn't stop.

[aks29921]

use or instead of and in while statement
while (operator != 't' || operator != 'T')

[Worr]

Got it thanks.

This post has been edited by Worr: 07 October 2009 - 05:40 PM