フラグ列挙型の賛否両論は何ですか？

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; } } 

と。

どちらがベストプラクティスであり、それぞれのアプローチの賛否両論があればどうでしょうか？

EDIT：特にパフォーマンスに関して2つのアプローチの間に実用的な違いがあるかどうかを知りたいと思っています。私は、（Microsoftのガイドラインにない限り）コーディングスタイルに関する意見を求めることは望ましくない。なぜなら、質問が非構造的なものとして閉じられるからだ。ありがとう。それは非常に少なくとも

Answer 1

コードをより読みやすくなりますので、

Answer 2

後には、あなたの第二の例は、よりよい意味を持っているし、コード内のビットの意味を示し、ベストプラクティスです。そのビットが何のために使われているかのコード内にいくつかの文書があります。

それ以外の場合は、最初の例に基づいて、基本的には魔法（ビット）番号を詰まらせるのでコメントを追加する必要があります。たとえあなたがこのコードを6ヶ月間維持しているとしても、どのビット5が使用されたのかを覚えるのは難しいかもしれません。

Answer 3

.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です。