При сравнении перечислений из разных источников, например, из следующего кода, GCC выдает предупреждения. Есть ли способ избежать этих предупреждений без приведения c-стиля?
struct Enumerator
{
enum { VALUE = 5 };
};
template<int V>
struct TemplatedEnumerator
{
enum { VALUE = V };
};
if(Enumerator::VALUE == TemplatedEnumerator<5>::VALUE)
{
...
}
И GCC выдает предупреждение следующего типа:
GCC: warning: comparison between 'enum ...' and 'enum ...'
Простой ответ в вашем случае: не используйте enum, используйте встроенный static const int:
struct Enumerator
{
static int const VALUE = 5;
};
template<int V>
struct TemplatedEnumerator
{
static int const VALUE = V;
};
В этом частном случае это эквивалентно, и все компиляторы последних нескольких лет должны относиться к этому так (я точно знаю, что все основные).
См. также: static const Member Value vs. Member enum: какой метод лучше и почему?
Я считаю, что вы можете предоставить свои собственные операторы сравнения, но сначала вам нужно будет назвать перечисления:
struct Enumerator
{
enum Values { VALUE = 5 };
};
template<int V>
struct TemplatedEnumerator
{
enum Values { VALUE = V };
};
template <int V>
bool operator==(Enumerator::Values lhs, typename TemplatedEnumerator<V>::Values rhs)
{
return static_cast<int>(lhs) == static_cast<int>(rhs);
}
template <int V>
bool operator==(typename TemplatedEnumerator<V>::Values rhs, Enumerator::Values lhs)
{
return static_cast<int>(lhs) == static_cast<int>(rhs);
}
Интересно, что Visual Studio на самом деле не предупреждает о сравнении двух типов перечислителей, а скорее предупреждает о постоянном значении в операторе if — точно так же, как если бы вы сделали это:
if (true) {...}
Если есть N перечислений, не нужно ли будет N * (N - 1) операторов сравнения? Это может оказаться очень много.
Не могли бы вы просто использовать неявное преобразование перечислений в целые и сделать:
bool equals(int lhs, int rhs)
{
return lhs == rhs;
}
Тогда вы можете просто сделать это в своем коде:
if(equals(Enumerator::VALUE, TemplatedEnumerator<5>::VALUE))
{
...
}
Кроме того, вы можете проверить приведение enum-int: оператор или функция. Принятый ответ говорит, что вам не следует использовать неявное преобразование, но правильно названная функция, такая как equals, должна сделать код очень читабельным и удобным в сопровождении.
Не делай этого.
Предупреждение есть, потому что вы не должны смешивать разные перечисления в первую очередь.
Для констант вы можете использовать объявление const.
Вы упускаете очевидный ответ. Нет, не используйте приведение в стиле c. Используйте приведения в стиле C++:
if(static_cast<int>(Enumerator::VALUE) == static_cast<int>(TemplatedEnumerator<5>::VALUE))
{
}
Совершенно безопасно, вполне приемлемо. Два перечисления не являются одним и тем же типом, поэтому если в вашем коде есть что-то подозрительное, это не приведение, а попытка сравнить два разных типа. Но до тех пор, пока вы это делаете, гипс будет моим предпочтительным способом сделать это. Да, вы можете определить оператор сравнения, но приведение типов дает понять, что вы сравниваете два технически несвязанных типа.
