strcmp（） - コードがコンパイルされても問題なく動作しているようです。

Question

私は演算子（+、 - 、/、*のいずれか）をユーザーに与えようとしています。彼/彼女はこれを行い確実にするために、私はこのコードを書いた：

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



int main(void) 
{ 
char operator; 

printf("Enter operator: (+, -, *, /) \n"); 

do { scanf("%c", &operator); } 
while ((strcmp(&operator, "+") != 0) || (strcmp(&operator, "-") != 0) || (strcmp(&operator, "*") != 0) || (strcmp(&operator, "/") != 0)); 
} 

何が起こって終わると、ループは、私が正しい演算子を入力しても、延々と続くされます。どんな助けもありがとうございます。感謝:)

編集：（固定コード）

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



int main(void) 
{ 
char operator; 

printf("Enter operator: (+, -, *, /) \n"); 

    do { scanf(" %c", &operator); } 
while ((strcmp(&operator, "+") != 0) && (strcmp(&operator, "-") != 0) && (strcmp(&operator, "*") != 0) && (strcmp(&operator, "/") != 0)); 

} 

Answer 1

関数strcmpは、文字ではなくゼロで終了する文字列を受け取ります。そのため、

strcmp(&operator, "+") 

の使用は未定義の動作の原因です。

あなたのコードは、私も&&に||を変更お知らせ

while ((operator != '+') && ...) 

ような単純なものでした。

" %c"のように、"%c"の前にスペースが必要です。入力ループが繰り返されると、入力バッファに残っているnewlineがクリアされるようになります。

EDIT：あなたは右の補正を行っているようには見えませんが、私は

do { 
    scanf(" %c", &operator); 
} while (operator != '+' && operator != '-' && operator != '*' && operator != '/'); 

Answer 2

変数オペレータに次のよう

char operator[2] = { '\0' }; 

そして

do { scanf("%c ", operator); } 
while ((strcmp(operator, "+") != 0) && (strcmp(operator, "-") != 0) && (strcmp(operator, "*") != 0) && (strcmp(operator, "/") != 0)); 
} 

が、その代わりに考慮してくださいように使用を宣言しますたくさんのを使用するstrcmpあなたは1つの機能を使用することができましたstrchr