変数をif文の値のリストと比較するより読みやすい方法はありますか？

Question

ifステートメントで変数を同じ型の値のリストと比較し、それを読みやすくきれいにするにはどうすればよいですか？

たとえば、次のようなことがありますが、多くの値があり、わかりやすくしたいと考えています。

if ((lst->type == 'p' || lst->type == 'x' || lst->type == 'X' || 
    lst->type == 'o' || lst->type == 'O' || (lst->type == 'd' || 
    lst->type == 'i' || lst->type == 'D')) 

Answer 1

strchr機能を使用してこれを処理するための最も簡単な方法：strchrについて

const char *lookups = "pxXoOdiD"; 
if (strchr(lookups, lst->type)){ 
    // Do your thing here to handle the condition 
} 

詳しい情報を見つけることができます。

C文字列strの最初の文字の出現を指すポインタを返します。

Answer 2

ルックアップテーブルを使用するか、またはswitchのステートメントのいずれかを使用：

const char lookup[] = {'p', 'x', 'X', 'o', 'O', 'd', 'i', 'D'}; 
for (int i = 0; i < sizeof(lookup)/sizeof(*lookup); i++){ 
    if (lst->type == lookup[i]) { 
     // Your stuff 
     break; // Remember to break 
    } 
} 

switch (lst->type) { 
    case 'p': case 'x': case 'X': case 'o': 
    case 'O': case 'd': case 'i': case 'D': 
     // Your stuff 
}