Сравнение всех элементов структуры по определенным правилам

Question

У меня есть структура с 4 элементами внутри.

typedef struct 
    { 
    char colour; 
    char shape; 
    char nr; 
    char p; 
    }CARDS; 

Эта структура имеет много элементов внутри и я хочу, чтобы сравнить их 3 на 3 .. поэтому сравните, например, карты [1], карты [2], карты [3]. Если 3/4 элементов между 3-мя картами равны, тогда у нас есть SET так set = 1. Если элементы не равны, мы снова имеем set = 1.

Конечно, я мог бы просто написать много Если

If (cards[1].colour=cards[2].colour=cards[3].colour) 
    counter = counter +1; 
If (cards[1].shape=cards[2].shape=cards[3].shape) 
    counter=counter+1; 
If (cards[1].nr=cards[2].nr=cards[3].nr) 
    counter=counter+1; 
If (cards[1].p=cards[2].p=cards[3].p) 
    counter=counter+1; 

, а затем сделать, если заявление на прилавок

If (counter==3||counter==0) 
    set=1; 

Есть ли другие более элегантный способ сделать это?

Answer 1

Прежде всего, вы используете =, когда вы должны использовать ==, скомпилируйте с предупреждениями, чтобы увидеть это. Вы также пытаетесь сравнить три вещи одновременно. Вы не можете сделать if (a == b == c), но вместо этого должны делать if (a == b && a == c) или аналогичные.

== Истинный результат равен 1, и вы можете использовать это здесь, просто добавив его:

int count = 
    (cards[1].colour == cards[2].colour && cards[1].colour == cards[3].colour) 
    + (cards[1].shape == cards[2].shape && cards[1].shape == cards[3].shape) 
    + (cards[1].nr == cards[2].nr && cards[1].nr == cards[3].nr) 
    + (cards[1].p == cards[2].p && cards[1].p == cards[3].p); 

int set = count == 3 || count == 0; 

Answer 2

Вы могли бы сделать вашу структуру данных, более склонны к такого рода сравнения, например,

typedef enum { 
    COLOUR, 
    SHAPE, 
    NR, 
    P, 
    NUM_ATTRS 
} ATTRS; 

typedef struct { 
    char attrs[NUM_ATTRS]; 
} CARD; 

Это позволяет использовать простые циклы для проверки нескольких атрибутов нескольких карт, а не жестко закодированной явной логики.

Answer 3

Существует серьезная ошибка в состоянии, должно быть больше: Если 3/4 элементов между 3 карты

if (counter >= 3 || counter==0) 
    set=1;