Question

我需要来自数组的EnumSet（通过varargs方法参数给出）。首先，我很惊讶EnumSet中没有varargs构造函数方法（有EnumSet#of(E first, E... rest)）。作为一种解决方法，我使用了以下变体：

EnumSet<Options> temp = EnumSet.copyOf(Arrays.asList(options));

然而，这会触发java.lang.IllegalArgumentException: Collection is empty。所以，现在我最终得到了以下内容，看起来有些荒谬：

EnumSet<Options> temp = options.length > 0 ? 
                        EnumSet.copyOf(Arrays.asList(options)) : 
                        EnumSet.noneOf(Options.class);

如果当然这可以转移到某种实用方法，但是，我仍然问自己，使用现有方法是否有更简单的方法？

Answer 1

只是另一种选择。使用EnumSet.of()：

，相同数量的代码，除了不需要转换为列表

EnumSet<Options> temp = options.length > 0 ? 
                        EnumSet.of(options[0], options) : 
                        EnumSet.noneOf(Options.class);

不用担心第一个元素会重复出现（无论如何都不会在一个集合中重复），也不会有性能上的提升或惩罚。

Answer 2

这是两行，但稍微复杂一点：

    EnumSet<Options> temp = EnumSet.noneOf(Options.class); // make an empty enumset
    temp.addAll(Arrays.asList(options)); // add varargs to it

对于没有你想要的构造函数的类，我不认为这比任何其他类型的变量声明更糟糕：

    SomeClass variable = new SomeClass(); // make an empty object
    variable.addStuff(stuff); // add stuff to it

Answer 3

您也可以使用Java8流和您自己的CollectionFactory执行此操作：

EnumSet<Options> temp = Arrays.stream(options)
        .collect(Collectors.toCollection(() -> EnumSet.noneOf(Options.class)));

Answer 4

最简单的解决方案是显而易见的解决方案：如果您可以访问方法的签名，那么您只需将EnumSet.of(T first, T ... more)的签名模仿为

void myMethod(Options first, Options ... rest) {
    EnumSet<Options> temp = EnumSet.of(first, rest);
}

由于省略号已经是方法参数列表的最后一个元素（因为它必须是），这不会改变任何调用代码，也不需要跳过循环。

Answer 5

番石榴有针对这些情况的工厂方法：它仍然需要调用Arrays.asList，但至少它是可读的。

import com.google.common.collect.Sets;

EnumSet<Options> temp = Sets.newEnumSet(Arrays.asList(options), Options.class);

EnumSet来自数组，最短的变体？

5 个答案: