如果我有一个这样的enum:
public enum Letter {
A,
B,
C,
//...
}
随机挑选一个的最佳方法是什么?它不需要是生产质量的防弹，但一个相当均匀的分布会很好。
我可以这样做
private Letter randomLetter() {
int pick = new Random().nextInt(Letter.values().length);
return Letter.values()[pick];
}
但是有没有更好的办法呢?我觉得这个问题之前已经解决了。
当前回答
我想这个单行返回方法足够有效，可以用于这样一个简单的工作:
public enum Day {
SUNDAY,
MONDAY,
THURSDAY,
WEDNESDAY,
TUESDAY,
FRIDAY;
public static Day getRandom() {
return values()[(int) (Math.random() * values().length)];
}
public static void main(String[] args) {
System.out.println(Day.getRandom());
}
}
其他回答
我唯一的建议是缓存values()的结果，因为每次调用都会复制一个数组。另外，不要每次都创建一个Random。保持一个。除此之外，你做得很好。所以:
public enum Letter {
A,
B,
C,
//...
private static final List<Letter> VALUES =
Collections.unmodifiableList(Arrays.asList(values()));
private static final int SIZE = VALUES.size();
private static final Random RANDOM = new Random();
public static Letter randomLetter() {
return VALUES.get(RANDOM.nextInt(SIZE));
}
}
结合cletus和helios的建议，
import java.util.Random;
public class EnumTest {
private enum Season { WINTER, SPRING, SUMMER, FALL }
private static final RandomEnum<Season> r =
new RandomEnum<Season>(Season.class);
public static void main(String[] args) {
System.out.println(r.random());
}
private static class RandomEnum<E extends Enum<E>> {
private static final Random RND = new Random();
private final E[] values;
public RandomEnum(Class<E> token) {
values = token.getEnumConstants();
}
public E random() {
return values[RND.nextInt(values.length)];
}
}
}
编辑:哎呀，我忘记了有界类型参数，<E extends Enum<E>>。
让函数从数组中随机选取一个值可能是最简单的。这是更通用的，并且调用起来很简单。
<T> T randomValue(T[] values) {
return values[mRandom.nextInt(values.length)];
}
像这样叫:
MyEnum value = randomValue(MyEnum.values());
这里有一个使用shuffle和streams的版本
List<Direction> letters = Arrays.asList(Direction.values());
Collections.shuffle(letters);
return letters.stream().findFirst().get();
同意stephen C和helios的观点。从Enum中获取随机元素的更好方法是:
public enum Letter {
A,
B,
C,
//...
private static final Letter[] VALUES = values();
private static final int SIZE = VALUES.length;
private static final Random RANDOM = new Random();
public static Letter getRandomLetter() {
return VALUES[RANDOM.nextInt(SIZE)];
}
}
