使用static final unmodifiable Set作为常量来检查变量值

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().