文字列からキーワードを読み込むときにif文をリファクタリングする

Question

私はプログラミングには比較的新しいですが、だからこそ長い間lurkerされています。

私は現在大学で勉強していますが、入力と出力が16進数でユーザーに表示される非常に基本的な電卓であるプログラムの作成に少し問題があります。

ほとんどのコードは良いですが、この特定のコードをリファクタリングする方法があるかどうか疑問に思っていたので、if文の大きなブロックを使用してオペレータそれに対応する関数に入りますか？

これは私が何をしたか、そして私がリファクタリングする必要があるもののスニペットです。

個人的には、私は入れ子にするか、おそらくこのためにループを利用するように感じますが、私は確かに知るための十分な知識がありません！

ありがとうございました。

int main() { 
    printf ("Type in an expression: "); 
    scanf ("%x %s %x", &a, op, &b); 
    if (strcmp (op, "add") == 0) 
    { 
     add(a, b); 
    } 
    if (strcmp (op, "sub") == 0) 
    { 
     sub(a, b); 
    } 
    if (strcmp (op, "and") == 0) 
    { 
     band (a, b); 
    } 
    if (strcmp (op, "or") == 0) 
    { 
     bor (a, b); 
    } 
    /* ...... and so on .....*/ 

} 

Answer 1

else ifを使用してください。パースのこの種を行うには、「プロ」の方法は、（flexなど）レクサーとパーサジェネレータである（opマッチ"add"場合、それはまた"or"（など）に一致した場合、それはテストする最も確かに無意味です。）

（bisonなど）を使用すると、すべての可能性に同時に対応するCコードを生成することができますが、それらは習得に少し時間がかかります。単純なものの場合は、if elseとstrcmpの方がいいです。