7

我经常遇到一种验证模式，其中一些变量必须包含前缀数量的值之一才能有效。 

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 术语链是否可以提高性能，也许还可以代码可读性。

4

2 回答 2

15

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

没有太大的区别，但我个人会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.

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

最后，注意whenx是一个字符串，下面的可能是不正确的：

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

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

于 2012-12-11T12:44:00.720 回答
2

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

public final static Set SET_OF_CONSTANTS = Collections.unmodifiableSet(
    Set.of(CONSTANT_1, CONSTANT_2, CONSTANT_3));
于 2018-12-19T12:28:41.093 回答