如果我有一个这样的enum:
public enum Letter {
A,
B,
C,
//...
}
随机挑选一个的最佳方法是什么?它不需要是生产质量的防弹，但一个相当均匀的分布会很好。
我可以这样做
private Letter randomLetter() {
int pick = new Random().nextInt(Letter.values().length);
return Letter.values()[pick];
}
但是有没有更好的办法呢?我觉得这个问题之前已经解决了。
当前回答
同意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)];
}
}
其他回答
结合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>>。
我唯一的建议是缓存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));
}
}
enum形状颜色{ 蓝色, 黄色的, 红色, 绿色, 白色 ｝
Random random=new Random();
ShapeColor[] values=ShapeColor.values();
int size=values.length;
return values[random.nextInt(size)];
在枚举上实现一个随机函数更容易。
public enum Via {
A, B;
public static Via viaAleatoria(){
Via[] vias = Via.values();
Random generator = new Random();
return vias[generator.nextInt(vias.length)];
}
}
然后你从类中调用它，像这样
public class Guardia{
private Via viaActiva;
public Guardia(){
viaActiva = Via.viaAleatoria();
}
一行
return Letter.values()[new Random().nextInt(Letter.values().length)];
