Do ...論理演算子を使用したwhileループ

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

あなたは持っている：whileの条件で||の

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

使用することは間違っています。

operatorの値が'+'であるとします。その後、whileがあると評価：

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

while (true); 

と評価されたどんなにoperatorの値が何であるか、それらのサブ式の少なくとも3つがtrueに評価されません。したがって、条件式は常にtrueと評価されます。

||の代わりに&&を使用する必要があります。

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

コードをより明確にするための1つの可能な方法は次のとおりです。

あなたが使用して isValidOperatorでコードを短くすることができます

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

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

：追求

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