使用标志枚举有哪些优点和缺点？

Question

我从硬件接收到几个位字段。

我的代码最初是：

public readonly byte LowByte;

public bool Timer { get { return (LowByte & 1) == 1; } }

然后我记得标志枚举并正在考虑将其更改为：

[Flags]
public enum LowByteReasonValues : byte
{
    Timer = 1,
    DistanceTravelledExceeded = 2,
    Polled = 4,
    GeofenceEvent = 8,
    PanicSwitchActivated = 16,
    ExternalInputEvent = 32,
    JourneyStart = 64,
    JourneyStop = 128
}

public readonly LowByteReasonValues LowByte;

public bool Timer { get { return (LowByte & LowByteReasonValues.Timer) == LowByteReasonValues.Timer; } }

等等。

哪种方法是最佳实践？每种方法的优缺点是什么？

编辑：我很想知道这两种方法之间是否存在任何实际差异，特别是在性能方面。我不想征求有关编码风格的意见（除非它来自 Microsoft 指南），因为这会使问题被认为没有建设性。谢谢。

Answer 1

后者是最佳实践，因为它使您的代码更具可读性

Answer 2

如果您使用的是 .NET 4.0，现在可以使用 HasFlag 方法来检查枚举是否包含特定位。这使得它比以前的检查方法更具可读性。

[Flags]
public enum LowByteReasonValues : byte
{
    Timer = 1,
    DistanceTravelledExceeded = 2,
    Polled = 4,
    GeofenceEvent = 8,
    PanicSwitchActivated = 16,
    ExternalInputEvent = 32,
    JourneyStart = 64,
    JourneyStop = 128
}

public readonly LowByteReasonValues LowByte;

public bool Timer 
{
  get 
  { 
    return (LowByte.HasFlag(LowByte.Timer));
  } 
}

有关 MSDN 的更多信息。

Answer 3

至少，您的第二个示例具有更好的语义并指示代码中位的含义。代码中有一些文档说明了该位的用途。

否则，根据您的第一个示例，您将需要添加注释，因为您基本上是在摆弄魔术（位）数字，这使得代码更难以阅读，尤其是对于另一个不熟悉它的人来说。即使您自己将在六个月后维护此代码，您也可能会发现很难记住第 5 位的用途。