【发布时间】：2011-07-17 19:37:21
【问题描述】：
我维护着一个大型文档存档，并且我经常使用位域来记录我的文档在处理期间或验证时的状态。我的遗留代码只是使用静态 int 常量，例如：
static int DOCUMENT_STATUS_NO_STATE = 0
static int DOCUMENT_STATUS_OK = 1
static int DOCUMENT_STATUS_NO_TIF_FILE = 2
static int DOCUMENT_STATUS_NO_PDF_FILE = 4
通过设置适当的标志，可以很容易地指示文档所处的状态。例如：
status = DOCUMENT_STATUS_NO_TIF_FILE | DOCUMENT_STATUS_NO_PDF_FILE;
由于使用静态常量的方法是不好的做法，并且因为我想改进代码，所以我希望使用 Enums 来实现相同的效果。有一些要求，其中之一是需要将状态作为数字类型保存到数据库中。因此需要将枚举常量转换为数值。以下是我的第一种方法，我想知道这是否是正确的方法？
class DocumentStatus{
public enum StatusFlag {
DOCUMENT_STATUS_NOT_DEFINED(1<<0),
DOCUMENT_STATUS_OK(1<<1),
DOCUMENT_STATUS_MISSING_TID_DIR(1<<2),
DOCUMENT_STATUS_MISSING_TIF_FILE(1<<3),
DOCUMENT_STATUS_MISSING_PDF_FILE(1<<4),
DOCUMENT_STATUS_MISSING_OCR_FILE(1<<5),
DOCUMENT_STATUS_PAGE_COUNT_TIF(1<<6),
DOCUMENT_STATUS_PAGE_COUNT_PDF(1<<7),
DOCUMENT_STATUS_UNAVAILABLE(1<<8);
private final long statusFlagValue;
StatusFlag(long statusFlagValue) {
this.statusFlagValue = statusFlagValue;
}
public long getStatusFlagValue(){
return statusFlagValue;
}
}
/**
* Translates a numeric status code into a Set of StatusFlag enums
* @param numeric statusValue
* @return EnumSet representing a documents status
*/
public EnumSet<StatusFlag> getStatusFlags(long statusValue) {
EnumSet statusFlags = EnumSet.noneOf(StatusFlag.class);
StatusFlag.each { statusFlag ->
long flagValue = statusFlag.statusFlagValue
if ( (flagValue&statusValue ) == flagValue ) {
statusFlags.add(statusFlag);
}
}
return statusFlags;
}
/**
* Translates a set of StatusFlag enums into a numeric status code
* @param Set if statusFlags
* @return numeric representation of the document status
*/
public long getStatusValue(Set<StatusFlag> flags) {
long value=0;
flags.each { statusFlag ->
value|=statusFlag.getStatusFlagValue()
}
return value;
}
public static void main(String[] args) {
DocumentStatus ds = new DocumentStatus();
Set statusFlags = EnumSet.of(
StatusFlag.DOCUMENT_STATUS_OK,
StatusFlag.DOCUMENT_STATUS_UNAVAILABLE);
assert ds.getStatusValue( statusFlags )==258 // 0000.0001|0000.0010
long numericStatusCode = 56;
statusFlags = ds.getStatusFlags(numericStatusCode);
assert !statusFlags.contains(StatusFlag.DOCUMENT_STATUS_OK);
assert statusFlags.contains(StatusFlag.DOCUMENT_STATUS_MISSING_TIF_FILE);
assert statusFlags.contains(StatusFlag.DOCUMENT_STATUS_MISSING_PDF_FILE);
assert statusFlags.contains(StatusFlag.DOCUMENT_STATUS_MISSING_OCR_FILE);
}
}
【问题讨论】：
-
为什么不能写：static int DOCUMENT_STATUS_NO_PDF_FILE = 1
-
@trashgod 链接似乎已死。 here 是另一个。
-
@SeyedJalalHosseini：另见第 32 项：使用 EnumSet 代替位域，引用 here。
-
@trashgod 每次编写 EnumSet 时，您都会创建一个放在堆上而不是堆栈上的对象
if (universe.length <= 64) return new RegularEnumSet<>(elementType, universe);。虽然它是使用本机的小对象，但它仍然是一个对象。这仅在您的目标是零垃圾和/或 gc blips 时才重要。在高性能循环中，您不想调用EnumSet.of而更喜欢 longs... 等等。但这是一个极端的边缘案例。