java - 使用 static final unmodifiable Set as constant 来检查变量值

Question

我经常运行一种验证模式，在这种模式中，某些变量必须包含前缀数量的值之一才能有效。

PSEUDO CODE:
    IF x == CONSTANT_1 || X == CONSTANT_2 || ... || x == CONSTANT_N
    THEN X is valid

为了避免 OR 项链，我创建了一个静态最终不可修改集，其中包含所有常量:

public final static String CONSTANT_1 = *value* ;
public final static String CONSTANT_2 = *value* ;
...
public final static String CONSTANT_N = *value* ;

public final static Set SET_OF_CONSTANTS = Collections.unmodifiableSet(new HashSet(){
    private static final long serialVersionUID = 1L;
    {
        add(CONSTANT_1); 
        add(CONSTANT_2);
        ...
        add(CONSTANT_3);
    }
});

然后我按以下方式执行检查:

if(!SET_OF_CONSTANTS.contains(x)){ 
    //X NOT VALID 
}

我想知道这是否是一个好的编程实践，是否有任何替代方案，以及使用哈希表查询(理论上 O(1))而不是 OR 术语链是否确实可以提高性能或许还有代码可读性。

Answer 1

总的来说，我认为这是一种非常好的风格。

没有太大区别，但我个人会这样定义 SET_OF_CONSTANTS:

      public final static String CONSTANT_1 = "*value*";
      public final static String CONSTANT_2 = "*value*";
              ...
      public final static String CONSTANT_N = "*value*";

      public final static Set<String> SET_OF_CONSTANTS = Collections.unmodifiableSet(
        new HashSet<String>(Arrays.asList(
              CONSTANT_1, 
              CONSTANT_2,
                      ...
              CONSTANT_N
              )));

我不完全清楚您是否需要单独的 CONSTANT_1 常量，或者您是否可以简单地将这些值折叠到 SET_OF_CONSTANTS 中。

就性能而言，在我根据真实数据分析代码之前，我不会开始优化任何东西。

最后请注意，当x 是一个字符串时，以下可能是不正确的:

IF x == CONSTANT_1 || x == CONSTANT_2 || ... || x == CONSTANT_N

在这里，== 可能应该替换为对 equals() 的调用。