Я правильно сравниваю значения enum с оператором или оператором C++?

Question

Я написал кусок кода, который не похоже на работу в соответствии с требованиями:

typedef enum{none=0,apple,grape,orange} FRUIT; 

FRUIT first = rand()%4; 
FRUIT second = rand()%4; 
FRUIT third = rand()%4; 

Так что в моем состоянии, если, я могу иметь

if (first == (none | apple | grape | orange) && 
    second == apple && 
    third == (none | apple | grape | orange) 
{ 
    cout<<"Here"<<<endl; 
} 

Переменные first и third может иметь любой от яблоко, виноград, нет или оранжевый значения. Правильно ли условие if? Я не получаю желаемый результат, так как он вообще не входит в условие if.

Answer 1

В состоянии:

first == (none | apple | grape | orange) 

вы должны использовать логическое или (||) вместо побитового или (|). К сожалению, даже если вы изменили его:

first == (none || apple || grape || orange) 

правая часть этого условия будет оцениваться первым, что делает его (в данном случае), что эквивалентно:

first == true 

, которая до сих пор семантически отличается от того вероятно, имел в виду:

first == none || first == apple || first == grape || first == orange 

отметить также, что использование rand() не очень мудрая здесь. Вы можете попробовать использовать вместо std::random_shuffle:

FRUIT fruit[] = { none, apple, grape, orange, 
        none, apple, grape, orange, 
        none, apple, grape, orange}; 
srand(time(NULL)); 
std::random_shuffle(&fruit[0], &fruit[11]); 
FRUIT first = fruit[0]; 
FRUIT second = fruit[1]; 
FRUIT third = fruit[2]; 

просто не забудьте #include <algorithm> :)

Answer 2

Могу ли я сравнивать значения перечислений с оператором С ++ „или“ правильно?

Нет, вы должны написать:

first == none || first == apple || first == grape || first == orange