it-swarm.com.ru

Поиск значения перечисления с помощью Java 8 Stream API

Предположим, есть простое перечисление с именем Type, определенное так:

enum Type{
    X("S1"),
    Y("S2");

    private String s;

    private Type(String s) {
        this.s = s;
    }
}

Поиск правильного перечисления для данного s тривиально выполняется статическим методом с циклом for (предположим, что метод определен внутри enum), например:

private static Type find(String val) {
        for (Type e : Type.values()) {
            if (e.s.equals(val))
                return e;
        }
        throw new IllegalStateException(String.format("Unsupported type %s.", val));
}

Я думаю, что функциональный эквивалент этого, выраженный в Stream API, будет примерно таким:

private static Type find(String val) {
     return Arrays.stream(Type.values())
            .filter(e -> e.s.equals(val))
            .reduce((t1, t2) -> t1)
            .orElseThrow(() -> {throw new IllegalStateException(String.format("Unsupported type %s.", val));});
}

Как мы могли бы написать это лучше и проще? Этот код кажется принудительным и не очень понятным. Функция reduce() особенно кажется неуклюжей и злоупотребленной, так как она ничего не накапливает, не выполняет никаких вычислений и всегда просто возвращает t1 (при условии, что фильтр возвращает одно значение - если это явно не является катастрофой), не говоря уже о том, что t2 является излишним и сбивает с толку. Тем не менее, я не смог найти ничего в Stream API, которое просто каким-то образом возвращало бы T из Stream<T>.

Есть ли способ лучше?

31
quantum

Я бы использовал findFirst вместо этого:

return Arrays.stream(Type.values())
            .filter(e -> e.s.equals(val))
            .findFirst()
            .orElseThrow(() -> new IllegalStateException(String.format("Unsupported type %s.", val)));


Хотя Map может быть лучше в этом случае:

enum Type{
    X("S1"),
    Y("S2");

    private static class Holder {
        static Map<String, Type> MAP = new HashMap<>();
    }

    private Type(String s) {
        Holder.MAP.put(s, this);
    }

    public static Type find(String val) {
        Type t = Holder.MAP.get(val);
        if(t == null) {
            throw new IllegalStateException(String.format("Unsupported type %s.", val));
        }
        return t;
    }
}

Я узнал этот трюк из этого ответ . По сути, загрузчик классов инициализирует статические классы перед классом enum, что позволяет вам заполнить Map в самом конструкторе enum. Очень кстати !

Надеюсь, поможет ! :)

69
Alexis C.

Принятый ответ работает хорошо, но если вы хотите избежать создания нового потока с временным массивом, вы можете использовать EnumSet.allOf().

EnumSet.allOf(Type.class)
       .stream()
       .filter(e -> e.s.equals(val))
       .findFirst()
       .orElseThrow(String.format("Unsupported type %s.", val));
14
Pär Eriksson
Arrays.stream(Type.values()).filter(v -> v.s.equals(val)).findAny().orElseThrow(...);
4
sprinter

Как насчет использования findAny() вместо reduce?

private static Type find(String val) {
   return Arrays.stream(Type.values())
        .filter(e -> e.s.equals(val))
        .findAny()
        .orElseThrow(() -> new IllegalStateException(String.format("Unsupported type %s.", val)));
}
4
Todd

Я думаю, что второй ответ Алексис С. ( ответ Алексис С. ) является хорошим с точки зрения сложности. Вместо того, чтобы искать в O(n) каждый раз, когда вы ищите код, используя 

return Arrays.stream(Type.values())
        .filter(e -> e.s.equals(val))
        .findFirst()
        .orElseThrow(() -> new IllegalStateException(String.format("Unsupported type %s.", val)));

вы можете использовать O(n) время при загрузке класса, поместив все элементы на карту, а затем получить доступ к коду типа в постоянное время O(1), используя карту.

enum Type{
X("S1"),
Y("S2");

private final String code;
private static Map<String, Type> mapping = new HashMap<>();

static {
    Arrays.stream(Type.values()).forEach(type-> mapping.put(type.getCode(), type));
}

Type(String code) {
    this.code = code;
}

public String getCode() {
    return code;
}

public static Type forCode(final String code) {
    return mapping.get(code);
}

}

1
Bastien Escouvois

Я знаю, что этот вопрос старый, но я пришел сюда из дубликата. Мой ответ не является точным ответом на вопрос ОП о том, как решить проблему с помощью Java Streams. Вместо этого этот ответ расширяет Map-основанное решение, предложенное в принятом ответе , чтобы стать более (IMHO) управляемым.

Итак, вот оно: я предлагаю ввести специальный вспомогательный класс, который я назвал EnumLookup.

Предполагая, что перечисление Type написано немного лучше (содержательное имя поля + получатель), я вставляю в него константу EnumLookup, как показано ниже:

enum Type {

    X("S1"),
    Y("S2");

    private static final EnumLookup<Type, String> BY_CODE = EnumLookup.of(Type.class, Type::getCode, "code");

    private final String code;

    Type(String code) {
        this.code = code;
    }

    public String getCode() {
        return code;
    }

    public static EnumLookup<Type, String> byCode() {
        return BY_CODE;
    }
}

Использование становится (опять же, IMO) действительно читабельным:

Type type = Type.byCode().get("S1"); // returns Type.X

Optional<Type> optionalType = Type.byCode().find("S2"); // returns Optional(Type.Y)

if (Type.byCode().contains("S3")) { // returns false
    // logic
}

Наконец, вот код вспомогательного класса EnumLookup:

public final class EnumLookup<E extends Enum<E>, ID> {

    private final Class<E> enumClass;
    private final ImmutableMap<ID, E> valueByIdMap;
    private final String idTypeName;

    private EnumLookup(Class<E> enumClass, ImmutableMap<ID, E> valueByIdMap, String idTypeName) {
        this.enumClass = enumClass;
        this.valueByIdMap = valueByIdMap;
        this.idTypeName = idTypeName;
    }

    public boolean contains(ID id) {
        return valueByIdMap.containsKey(id);
    }

    public E get(ID id) {
        E value = valueByIdMap.get(id);
        if (value == null) {
            throw new IllegalArgumentException(String.format(
                    "No such %s with %s: %s", enumClass.getSimpleName(), idTypeName, id
            ));
        }
        return value;
    }

    public Optional<E> find(ID id) {
        return Optional.ofNullable(valueByIdMap.get(id));
    }

    //region CONSTRUCTION
    public static <E extends Enum<E>, ID> EnumLookup<E, ID> of(
            Class<E> enumClass, Function<E, ID> idExtractor, String idTypeName) {
        ImmutableMap<ID, E> valueByIdMap = Arrays.stream(enumClass.getEnumConstants())
                .collect(ImmutableMap.toImmutableMap(idExtractor, Function.identity()));
        return new EnumLookup<>(enumClass, valueByIdMap, idTypeName);
    }

    public static <E extends Enum<E>> EnumLookup<E, String> byName(Class<E> enumClass) {
        return of(enumClass, Enum::name, "enum name");
    }
    //endregion
}

Обратите внимание, что:

  1. Я использовал Guava ImmutableMap здесь, но вместо него можно использовать обычные HashMap или LinkedHashMap.

  2. Если вы помните об отсутствии отложенной инициализации в вышеупомянутом подходе, вы можете отложить сборку EnumLookup до первого вызова метода byCode (например, с помощью идиомы lazy-holder , как в принятом ответе )

1
Tomasz Linkowski

Вам нужен метод get для String s . В приведенном ниже примере это метод getDesc():

public static StatusManifestoType getFromValue(String value) {
    return Arrays.asList(values()).stream().filter(t -> t.getDesc().equals(value)).findAny().orElse(null);
}
0
Marcell Rico

Я пока не могу добавить комментарий, поэтому я публикую ответ в дополнение к вышеупомянутому answer , просто следуя той же идее, но используя подход Java 8:

public static Type find(String val) {
    return Optional
            .ofNullable(Holder.MAP.get(val))
            .orElseThrow(() -> new IllegalStateException(String.format("Unsupported type %s.", val)));
}
0
Thiago