Использование обобщений с коллекцией классов enum, реализующих один и тот же интерфейс - PullRequest
6 голосов
/ 28 декабря 2011

Я пытаюсь сделать обратный поиск на нескольких классах enum, реализующих тот же интерфейс Field, перебирая список Class es, используя Maps.uniqueIndex в Guava:

  Field valueOfSearchName = null;
  for (final Class<? extends Enum<?>> clazz : ImmutableList.of(
      EntityField.class,
      AddressField.class,
      PersonFunctionType.class)) {
    valueOfSearchName = Fields.valueOfSearchName(clazz, term.field()); // error
    if (valueOfSearchName != null) {
      // do something...
      break;
    }
  }

Я не хочу повторять один и тот же код (для создания индекса и поиска) во всех классах перечисления, поэтому я использую вспомогательный статический класс Fields, содержащий Fields.valueOfSearchName метод:

  public static <E extends Enum<E> & Field> Field valueOfSearchName(
      final Class<E> clazz, final String searchName) {
    // TODO: cache the index
    final ImmutableMap<String, E> index = Maps.uniqueIndex(
        EnumSet.allOf(clazz), GET_SEARCH_NAME_FUNCTION);
    return index.get(searchName);
  }

К сожалению, Eclipse показывает ошибку:

Bound mismatch: 
The generic method valueOfSearchName(Class<E>, String) of type Fields is not 
applicable for the arguments (Class<capture#1-of ? extends Enum<?>>, String).
The inferred type capture#1-of ? extends Enum<?> is not a valid substitute
for the bounded parameter <E extends Enum<E> & Field>

Проблема в Class<? extends Enum<?>> clazz в цикле for-each (не соответствует Field) , но я не знаю, как справиться с этим делом (очевидно, я не могу добавить & Field к clazz ).

Ответы [ 3 ]

4 голосов
/ 28 декабря 2011

Рассмотрим Class<? extends List<?>. Class<? extends List<?> имеет два символа подстановки, тогда как <E extends List<E>> Class<E> имеет только общий параметр. Бывший признается Class<ArrayList<String>>. Поэтому, не делая ничего особенного для перечислений, типы не совместимы.

Как исправить? Дополнительный слой косвенности!

public final class MetaEnum<E extends Enum<E>> {
    private final E clazz;
    public static <E extends Enum<E>> MetaEnum<E> of(E clazz) {
        return clazz;
    }
    private MetaEnum(E clazz) {
        this.clazz = clazz;
    }
    public E clazz() {
        return clazz;
    }
    // ...
}

for (final MetaEnum<?> meta : ImmutableList.of(
    MetaEnum.of(EntityField       .class),
    MetaEnum.of(AddressField      .class),
    MetaEnum.of(PersonFunctionType.class)
)) {
    Field valueOfSearchName = Fields.valueOfSearchName(
        meta.clazz(), term.field()
    );
    ...

(Обычный отказ от переполнения стека: не так много, как попытка компиляции.)

2 голосов
/ 02 января 2012

Вдохновленный Ответом Тома Хоутина Я создал класс-оболочку, содержащий Class es, но только с подписью <E extends Enum<E> & Field>:

public final static class FieldEnumWrapper<E extends Enum<E> & Field> {
  private final Class<E> clazz;
  private final ImmutableMap<String, E> index;

  public static <E extends Enum<E> & Field> 
      FieldEnumWrapper<E> of(final Class<E> clazz) {
    return new FieldEnumWrapper<E>(clazz);
  }

  private FieldEnumWrapper(final Class<E> clazz) {
    this.clazz = clazz;
    this.index = Maps.uniqueIndex(
        EnumSet.allOf(clazz), new Function<E, String>() {
          @Override
          public String apply(final E input) {
            return input.searchName();
          }
        });
  }

  public Class<E> clazz() {
    return clazz;
  }

  public Field valueOfSearchName(final String searchName) {
    return index.get(searchName);
  }
}

Сейчас:

for (final FieldEnumWrapper<?> fieldEnum : ImmutableList.of(
    FieldEnumWrapper.of(EntityField.class),
    FieldEnumWrapper.of(AddressField.class),
    FieldEnumWrapper.of(PersonFunctionType.class))) {
  valueOfSearchName = fieldEnum.valueOfSearchName("POD_I_OS_PARTNER");
  // ...

является типобезопасным и неуместным использованием статической фабрики FieldEnumWrapper:

FieldEnumWrapper.of(NotEnumAndFieldClass.class)

генерирует ошибку компиляции.

Более того, valueOfSearchName теперь является методом FieldEnumWrapper, что более логично, чем вспомогательный класс.

0 голосов
/ 30 декабря 2011

может быть что-то вроде этого:

import java.util.*;
class N {
    static int n;
}
interface HasField {
    int getField();
}
enum Color implements HasField {
    r, g, b;
    public int getField() {
        return field;
    }
    private int field = N.n++;
}
enum Day implements HasField {
    m, t, w, th, f, sa, su;
    public int getField() {
        return field;
    }
    private int field = N.n++;
}
    class Helper {
    Helper(Set<HasField> set) {
        for (HasField hasField : set)
            if (hasField instanceof Enum) {
                Enum<?> e = (Enum<?>) hasField;
                for (Object o : e.getDeclaringClass().getEnumConstants()) {
                    map.put(((HasField) o).getField(), (Enum<?>) o);
                }
            } else
                throw new RuntimeException(hasField + " is not an enum!");
    }
    final Map<Integer, Enum<?>> map = new TreeMap<Integer, Enum<?>>();
}
public class Main {
    public static void main(String[] args) {
        Set<HasField> set = new LinkedHashSet<HasField>();
        set.add(Color.r);
        set.add(Day.m);
        Helper helper = new Helper(set);
        for (int i = 0; i < N.n; i++)
            System.out.println(i + " " + helper.map.get(i));
    }
}
...