Thread: while(true) loop trouble

I'm trying to create a simple calculator that keeps prompting the user for input until the user enters the standard input ("number" "operator") and the operator = 'E'.

I know in java I'd just use a while(true) loop (as I did in here), but I get the error "true undeclared." I'm new to C and am not sure what I'm doing wrong .. thanks for any help!

Here is my code:

Code:

#include <stdio.h>
#include <stdlib.h>

main() {

  double num; double accum;
  char op;

  printf("Initialize your Accumulator with data of the form 'number' 'S' which sets the Accumulator \
to the value of your number\n");
  scanf("%f %c", &num, &op);

  while (1) {

    if (op == 'S') {
    accum = num;
}

    else if (op == '+') {
      accum = accum+num;
    }

    else if (op == '*') {
      accum = accum*num;
    }

    else if (op == '/') {
      accum = accum/num;
    }

    else if (op == '-') {
      accum = accum-num;
}

    else if ((num == 0) && (op == '/')) {
      printf("Can not divide by 0.\n");
}


    else if (op == '-') {
      accum = accum-num;
}

    else if ((num == 0) && (op == '/')) {
      printf("Can not divide by 0.\n");
}

    else if (op == 'E') {
      printf("Value in the Accumulator = %d \n", accum);
      printf("End of Calculations.\n");
      exit(EXIT_SUCCESS); /*exits program if user enters E*/
    }

    else {
      printf("Unknown operator.\n");
    }

    printf("Value in the Accumulator = %f \n", accum);
    scanf("%f %c", &num, &op);

  }

Originally Posted by CommonTater

Try it as ... while (1)

uhh no, that doesn't work.



Edit: to clarify, I just need the user to keep being prompted to enter a new value. The output should look like this:

Initialize your Accumulator with data of the form 'number' 'S' which sets the Accumulator to the value of your number
123123 S /*user enters this*/
Value in the Accumulator = 123123.000000
234 + /*user enters this*/
Value in the Accumulator = 123357.000000
123.5234523432 * /*user enters this*/ ..etc

In one of your scanf's you are correctly using %d and in the rest you are incorrectly usinf %f. That's probably the main issue.
Note that you are using different data types for accum and num where it would probably make sense for them to be the same.
Note that the same line, plus the printf appear in about 7 places in that code. The best thing to do there would be to remove the duplication. There should only be one scanf line in the whole program. Once you sort out where that one line should go and remove the duplication, you'll have a simpler program and you'll be much happier with the clearer code.

Very good. You're doing well. Sometimes you do want the thing before the loop as well, but in thise case you can get away with just one. Notice that the last thing in the loop is the same a the thing done just before it. As the user running the program, they can't tell the difference between something executed at the end of the loop and something executed at the start of the loop (provided it always loops of course). So by moving it to the start of the loop you can remove the other scanf since you'd then have two in a row first time through.

As for the real problem I'm quite puzzled because the code shown doesn't look like it can cause that.
Can you give an example of what you typed in?
Also, perhaps it isn't recompiling the program and instead just running the code from some time ago. You could try changing the text and see if it is different when you run it again, just to be sure.