Я могу что-то упустить, но объявление LinkedHashmap:
private final LinkedHashMap<anEnum, anInteger> linkedHashMap;
и позже:
linkedHashMap = new LinkedHashMap<anEnum, anInteger>();
приводит к:
«Карты с ключами, которые являются значениями перечисления, должны быть заменены на EnumMap» (SQUID: 1640)
Поскольку «LinkedEnumMap» отсутствует, это объявление следует игнорировать.
Дело в том, что использование EnumMap является хорошей практикой, поэтому SonarQube говорит вам об этом.
Но вы явно решили использовать LinkedHashMap, который отслеживает порядок вставки.
Таким образом, это сообщение можно интерпретировать как:
Другими словами: имейте в виду, что такие инструменты — это всего лишь инструменты. Они предоставляют сообщения, которые помогают вам принимать обоснованные решения. Когда вы решили, что ваша текущая реализация в порядке - тогда не меняйте свой код только потому, что какой-то инструмент выдает такие "советы".
Специализированная реализация карты для использования с ключами типа enum. Все ключи в карте перечисления должны исходить из одного типа перечисления, который указывается явно или неявно при создании карты. Карты Enum внутренне представлены в виде массивов. Это представление чрезвычайно компактно и эффективно.
В принципе, это не ошибка, это хорошая рекомендация. Каждый Enum имеет свой порядок, который исходит из порядка объявления. Вы можете увидеть это в массиве, который возвращает values().
EnumMap сохраняет этот порядок и фактически сохраняет значения в массиве.
С другой стороны, вы можете решить, какая коллекция лучше для ваших целей. Я считаю, что сочетание EnumMap и List может быть вам интересно, но помните, что Sonar — это всего лишь намек. Вы можете перенастроить свой сонар и указать, чтобы он не показывал этот случай.
