сравнение строки с перечислением

Я анализирую следующий фрагмент кода с помощью инструмента статического анализа под названием FindBugs.сравнение строки с перечислением

if(str.equals(enum.SOMEVALUE)) {// do something};

где str - это строка, а перечисление - перечисление. Инструмент генерирует следующее предупреждение для этого кода и указывает

Этот метод вызывает equals (Object) на двух ссылках разных типов классов без общих подклассов. Согласно договору equals(), объекты разных классов всегда должны сравниваться как неравные; поэтому, согласно контракту, определенному java.lang.Object.equals (Object), результат этого сравнения всегда будет ложным во время выполнения.

если заменить выше строки кода с этим:

if(str.equals(enum.SOMEVALUE.toString())) {// do something};

то предупреждение disappears.But Я не уверен, если предупреждение о том, что инструмент генерирует действительно так и ли я ее исправление правильный путь ? потому что я видел такие сравнения раньше и, похоже, работает правильно.

источник

2012-05-11 comatose

Ваше первое сравнение в основном неправильное. Вы сравниваете объекты совершенно разных типов (String и Enum), и они никогда не могут быть равными. intellij даже дает мне предупреждение здесь. Он компилируется только потому, что equals() принимает Object, а не String.

Второе сравнение: .

Хотя JavaDoc is a bit harsh on name() метод, я бы на самом деле использовать его рекомендации в случае, если данный перечислимого имеет toString() перекрываться:

if(str.equals(FooEnum.SOMEVALUE.name()))

источник

2012-05-11 14:49:18

Большое спасибо за советы по использованию имени(), что делает мой текущий подход к полю id в подобных ситуациях устаревшим. – Bananeweizen

Обратите внимание, что объекты разных типов, безусловно, могут быть равны в соответствии с equals(). И FindBugs, и IntelliJ ошибочны.В стандартном java-интерфейсе есть пример, где это выполняется: реализации реализации интерфейса List равны, сравнивая элементы, даже для разных подклассов List. – herman

@herman: вы правы, но 'equals()' должно быть * симметрией *, что легко сделать с помощью реализаций «List» (все они основаны на 'AbstractList'), но не в случае OP. –

Я предложил бы использовать:

if (SomeEnum.SOMEVALUE == SomeEnum.valueOf(str)) { 

}

источник

2012-05-11 14:49:20 Bozho

Обратите внимание, что это вызовет исключение, если 'str' не представляет никакого значения' SomeEnum' –

, это было бы полезно использовать в операторах switch –

Я думаю, заменяющий константа для toString() может быть правильным, я бы изменил его на .name(), хотя потому, что toString нужно переоценить.

источник

2012-05-11 14:50:10

Насколько я знаю, вы находитесь на правильном пути.

if(str.equals(enum.SOMEVALUE.toString())) {// do something};

Все должно быть в порядке.

источник

2012-05-11 14:50:27 sarwar026

вы можете попробовать

enum.SOMEVALUE.name()

как это

Возвращает имя этого перечисления постоянной, так же, как заявил в своем заявлении перечислений.

источник

2012-05-11 14:50:40

сравнение строки с перечислением

ответ

Смежные вопросы