条件が多い場合

以下のようにif文の条件が
たくさん有る場合、もうすこしif文をすっきりさせる
書き方はないでしょうか？

#define AAA_1 1
#define AAA_2 2
#define AAA_3 3
#define AAA_4 4
#define AAA_5 5
#define AAA_6 6
#define AAA_7 7
#define AAA_8 8

int a;

if ( (a == AAA_1) ||
(a == AAA_2) ||
(a == AAA_3) ||
(a == AAA_4) ||
(a == AAA_7) ||
(a == AAA_8) ){
　　}
if ( (a == AAA_1) ||
(a == AAA_2) ||
(a == AAA_3) ||
(a == AAA_4) ||
(a == AAA_5) ||
(a == AAA_6) ){
　　}
if ( (a == AAA_1) ||
(a == AAA_3) ||
(a == AAA_4) ||
(a == AAA_5) ||
(a == AAA_8) ){
　　}

No.2 ベストアンサー 回答者： AsanoNagi 回答日時： 2013/12/18 09:37

配列をひとつ間において

int check_1[] = {AAA_1, AAA_2, AAA_3, AAA_4, AAA_7, AAA_8, -1};
int check_2[] = {AAA_1, AAA_2, AAA_3, AAA_4, AAA_5, AAA_6, -1);
int check_3[] = {AAA_1, AAA_2, AAA_3, AAA_4, AAA_5, AAA_8, -1);

Ｃの関数でこんなのを作って、
（src と、その後の、int のリスト（負の数で、リストの終端）を比較して、一致する物があれば、true を返す）

bool in(int src, int terget[])
{
for (int i = 0; terget[i] >= 0; i++)
{
if(src == terget[i] return true;
}
return fasle;
}

if(in(a, check_1)
{ }

if(in(a, check_2)
{ }

if(in(a, check_3)
{ }

Ｃの関数で、こんなのを作って、
（上と同じ関数の、可変引数関数版）

#include <stdarg.h>

bool in(int src, ...)
{
va_list args;
va_start(args, src);
while(1)
{
int terget = va_arg(args, int);
if(terget < 0)
{
va_end(args);
return false;
}

if (terget == src)
{
va_end(args);
return true;
}

}
}

if(in(a, AAA_1, AAA_2, AAA_3, AAA_4, AAA_7, AAA_8, -1)
{ }

if(in(a, AAA_1, AAA_2, AAA_3, AAA_4, AAA_5, AAA_6, -1)
{ }

if(in(a, AAA_1, AAA_2, AAA_3, AAA_4, AAA_5, AAA_8, -1)
{ }

最新のＣ＋＋だったら、もう少し簡単にできた気がする。

No.7 回答者： licht_gkk 回答日時： 2013/12/19 07:55

個人的にはNo2さんの回答が一番好みです。

専用の関数や宣言を増やしたくない場合、私は以下のように書いたりしています。
switch (a) {
case AAA_1:
case AAA_2:
case AAA_3:
case AAA_4:
case AAA_7:
case AAA_8:
//処理
break;
}
switch (a) {
case AAA_1:
case AAA_2:
case AAA_3:
case AAA_4:
case AAA_5:
case AAA_6:
//処理
break;
}
switch (a) {
case AAA_1:
case AAA_3:
case AAA_4:
case AAA_5:
case AAA_8:
//処理
break;
}

No.6 回答者： wormhole 回答日時： 2013/12/18 19:24

まずやるべきは、条件を整理できないか検討する事じゃないかと思います。

No.5 回答者： nak777r 回答日時： 2013/12/18 18:30

#define AAA_1 1

#define AAA_8 8
#define chk(A,B) (((A)<=AAA_8)&&((B)[((A)-(AAA_1))]=='1'))
if(chk(a,"11110011")){
}
if(chk(a,"11111100")){
}
if(chk(a,"11111001")){
}

No.4 回答者： foobar 回答日時： 2013/12/18 16:53

＃３さんが触れられてる switch文を使う、っというのもよさそうにおもいます。

でもって、処理の内容によるけど、三つのifをまとめて、（最初のifの中の処理を1、二番目のを２、三番目のを３っとすると）
aがAAA_1なら１，２，３を、AAA_2なら１，２、を、と言う具合に、分岐数を増やすほうが見通しがよくなるかもしれません。

No.3 回答者： ninoue 回答日時： 2013/12/18 09:41

case文、switch文、　if(...){...} else if(...){...} .... 構文等を使って下さい。

複数の同一の変数に対する連続したif文の内、一つだけしかifの条件が満足しない場合に、質問の書き方では条件を満たした後のif文は無駄ですし、プログラムも読み難くなります。

C OR C++ 条件分岐文 OR 制御文
C OR C++ case文　"else if"文　switch文
　　等とサーチして調べて下さい。
　　==>　
http://masudahp.web.fc2.com/cl/kiso/ck0205.html 等

No.1 回答者： hashioogi 回答日時： 2013/12/18 08:25

int a;

int b ;
b = 1 << (a - 1) ;

if (b & 0xcf) {
　　}
if (b & 0x3f) {
　　}
if (b & 0x9d){
　　}