3

I just started learning Python yesterday and one of the programs I'm trying to write is a calculator. This segment of code is where I'm having problems. The while loop is supposed to stop when the user inputs any of the math operators but it still asks for input even when the user enters one the correct characters.

What do I need to do to fix this.

    op = None                                 
    while (op != "-" or op != "+" or op != "*" or op != "/"):
        op = input("Enter operator (-, +, *, /):  ")
    print(op)

3 Answers 3

3
  op = None                                 
while op not in ["-","+","*","/"]:
    op = input("Enter operator (-, +, *, /):  ")
print(op)

or

op = None                                 
while (op != "-" and op != "+" and op != "*" and op != "/"):
    op = input("Enter operator (-, +, *, /):  ")
print(op)

your code isnt working because while "op" might be equal to one of your options, its not equal to all the others, and therefore continues the loop

Sign up to request clarification or add additional context in comments.

6 Comments

The not in technique is exactly what I was going to post, but I'd use a tuple rather than a list as tuples are faster.
For some reasons why, you can see here
informative, thankyou, so tuples are built about 3x faster than lists, and tuples made of literals (rather than variables) are built about 6x faster
Yep, lists are used for their mutability, but if it's not going to be changed, a tuple is better~
You can also use a string: while op not in "-+*/":
|
2

You don't want or between the conditions, you want and.

Comments

1

The other answers are great, but I wanted to show you a piece of code called the break statement. Here is a more detailed explanation, but basically, a break statement breaks out of an iterative loop like a while loop. Here is how your code would look if you used a break statement:

op = None                                 
while (op != "-" or op != "+" or op != "*" or op != "/"):
  op = input("Enter operator (-, +, *, /):  ")

  if op == "-" or op == "+" or op == "*" or op == "/":
    break

print(op)

1 Comment

This would be better shown with while True: imo~

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.