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()的调用。

Answer 2

另一种（较短的）方法是使用Set.of：

public final static Set SET_OF_CONSTANTS = Collections.unmodifiableSet(
    Set.of(CONSTANT_1, CONSTANT_2, CONSTANT_3));

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

2 个答案: