避免使用循环进行验证。
我建议使用valueOf
. 此方法内置于枚举中,可考虑用于编译时优化。
这类似于实现一个静态Map<String,EnumType>
来优化查找,这是您可以考虑的另一个考虑因素。
缺点是您必须使用异常处理机制来捕获非枚举值。
例子
public enum DataType {
//...
static public boolean has(String value) {
if (value== null) return false;
try {
// In this implementation - I want to ignore the case
// if you want otherwise ... remove .toUpperCase()
return valueOf(value.toUpperCase());
} catch (IllegalArgumentException x) {
// the uggly part ...
return false;
}
}
}
另请注意,通过上述类型实现,您的代码在调用时看起来更清晰。你的主要现在看起来像:
public void main(){
String filter = "SIZE";
String action = "DELETE";
// ...
if (Filter.has(filter) && Action.has(action)) {
// Appropriate action
}
}
然后提到的另一个选项是使用静态地图。您也可以采用这种方法来缓存基于其他属性的所有类型的索引。在下面的示例中,我允许每个枚举值都有一个别名列表。在这种情况下,查找索引将不区分大小写,强制为大写。
public enum Command {
DELETE("del","rm","remove"),
COPY("cp"),
DIR("ls");
private static final Map<String,Command> ALIAS_MAP = new HashMap<String,Command>();
static {
for (Command type:Command.values()) {
ALIAS_MAP.put(type.getKey().toUpper(),type);
for (String alias:type.aliases) ALIAS_MAP.put(alias.toUpper(),type);
}
}
static public boolean has(String value) {
return ALIAS_MAP.containsKey(value.toUpper());
}
static public Command fromString(String value) {
if (value == null) throw new NullPointerException("alias null");
Command command = ALIAS_MAP.get(value);
if (command == null) throw new IllegalArgumentException("Not an alias: "+value);
return command;
}
private List<String> aliases;
private Command(String... aliases) {
this.aliases = Arrays.asList(aliases);
}
}