Альтернативный синтаксис объединения проверки нескольких условий

Question

Я реализую простой оператор if в c, где я сравниваю значение integer 'tile' с четырьмя другими целыми числами: w, a, s и d.

Вот мой код:

if(tile == w || tile == a || tile == s || tile == d) 
{ 
    printf("legal\n"); 
    return true; 
} 

Хотя выше правильно, синтаксис утомительно. Есть ли более элегантный способ написания условия? «Плитка» является одним из следующих целых чисел ... »

Я новичок в программировании, поэтому прошу прощения, поскольку я подозреваю, что ответ очень очевиден. Однако мне не удалось найти его в другом месте.

Answer 1

Два варианта я могу думать ...

битовые

#define TILE_W 0x0001 
#define TILE_A 0x0002 
#define TILE_S 0x0004 
#define TILE_D 0x0008 

if (tile&(TILE_w|TILE_A|TILE_S|TILE_D)) 
{ 
    printf("legal\n"); 
    return true; 
} 

переключатель случая

switch (tile) 
{ 
case w: 
case a: 
case s: 
case d: 
    printf("legal\n"); 
    return true; 
default: 
    return false; 
} 

Answer 2

Раствор для улучшения может зависеть от значений, которые вы сравните с ,

Если w, a, s и d являются целыми числами, которые имеют последовательные значения (например, 10, 11, 12 и 13), Условный оператор может использовать условие для границ:

if(tile >= w && tile <= d) { printf("legal\n"); } 

Если значения несопоставимы (например, 6, 32, 142, 55), вы можете использовать switch .. case строительство, как

switch (tile) 
{ 
    case w: 
    case a: 
    case s: 
    case d: 
      printf("legal\n"); 
      break; 
    default: 
      printf("ILLEGAL\n"); 
      break; 
} 

Также вы можете использовать установки флага в одном или нескольких if в

int legal = 0; 

// checking can be in different places of code 
if (tile == w) 
    legal = 1; 
if (tile == a || tile == s) 
    legal = 1; 
if (tile == d) 
    legal = 1; 

if(legal) 
{ 
    printf("legal\n"); 
} 

И считают хранение w, a, s и d значения как массив допустимых значений, так что цикл может быть использован для проверки:

int valArr[] = {101, 151, 333, 7}; // you can add any number of values here 
int i; 
int legal = 0; 
for(i = 0; i < (sizeof(valArr)/sizeof(valArr[0])); i++) 
{ 
    if(valArr[i] == tile) 
    { 
     legal = 1; 
     break; 
    } 
} 
if(legal) 
{ 
    printf("legal\n"); 
} 
else 
{ 
    printf("ILLEGAL\n"); 
} 

Answer 3

Хотя eyalm's answer, может быть, один вы (, как вы уже упоминали, «я новичок в программировании»), что означает «написать код, который легче понять людям».

В то время как побитовое приближение короче, как правило, кажется, что труднее понять содержание, так как оно растет.

уборщик подход будет (в то время как она требует больше усилий, чтобы написать код), придерживаться синтаксиса if (a || b || c), или осенью, хотя switch случай. Это дает лучшую читаемость.

• В случае ваши варианты становится длиннее (, что вам может понадобиться горизонтальной прокрутки), вы можете рассмотреть вопрос о включении функции, чтобы получить требуемое значение проверено и использовать возвращаемое значение в состоянии в if заявлении.

Суть в том, что нет правильного или неправильного пути, только выберите способ, который сделает код более удобочитаемым и обслуживаемым.