做...同时循环逻辑运算符[重复]

Question

我从我的教科书上复制了一个简单的“计算器”示例，它接受2个值和一个运算符。但是，它不会检查输入的表达式是否有效。代码应该提示用户，直到输入正确的表达式。

我应该如何根据自己的风格来修正它？有什么更好的方法？

/*
Input: A simple expression with 2 values and 1 operator
Format: value1 (operator) value2
Ex: 5*5
Output: Answer
*/

#include <stdio.h>

int main (void)
{
float value1, value2;
char operator = ' ';

//Issue Area, the while section does not work
do
{
    printf ("Type in your expression:");
    scanf ("%f %c %f", &value1, &operator, &value2);
}
while ((operator != '+' || operator != '-' || operator != '*' || operator != '/'));

//This is fine, code is for an else...if example in textbook
if (operator == '+')
    printf ("%.2f\n",value1 + value2);
else if (operator == '-')
    printf ("%.2f\n",value1 - value2);
else if (operator == '*')
    printf ("%.2f\n",value1 * value2);
else if (operator == '/')
    printf ("%.2f\n",value1 / value2);

return 0;
}

Answer 1

你有：

do
{
   ... 
}
while ((operator != '+' || operator != '-' || operator != '*' || operator != '/'));

在的条件中使用||而是错误的。

假设运算符的值是'。然后，当计算为：

while ((false || true || true || true));

评估结果是

while (true);

无论运算符的值是什么，这些子表达式中至少有三个将计算为true。因此，条件将始终计算为true。

您需要使用

do
{
   ... 
}
while ((operator != '+' && operator != '-' && operator != '*' && operator != '/'));

使代码更清晰的一种可能方法是：

do
{
   ... 
}
while (!isValidOperator(operator));

哪里

int isValidOperator(char operator)
{
   return (operator == '+' ||
           operator == '-' ||
           operator == '*' ||
           operator == '/' );
}

您可以在isValidOperator中使用以下命令缩短代码：

int isValidOperator(char operator)
{
   return (operator != '\0' && strchr("+-*/", operator) != NULL);
}