Bayrak enumunu kullanmanın artıları ve eksileri nelerdir?

Question

Donanımdan birkaç bit alanı alıyorum.

Kodum aslen:

public readonly byte LowByte; 

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

Sonra bayrakları enum hatırladı ve değiştirmeyi düşünüyorum: böylece

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

ve.

En iyi uygulama hangisidir ve her bir yaklaşımın artıları ve eksileri nelerdir?

DÜZENLEME: İki yaklaşım arasında, özellikle performans açısından pratik farkların olup olmadığını bilmek istiyorum. Kodlama stilleri üzerine fikir istemek istemiyorum (Microsoft yönergelerinden gelmediği sürece). Teşekkürler. o En azından

Answer 1

kodunuzu daha okunaklı hale getirdiği için

Answer 2

daha sonra ikinci örnek daha iyi semantiğini vardır ve kodundaki bit anlamını gösterir, en iyi uygulamadır. Bitin ne için kullanıldığına dair kod içinde bazı belgeler vardır.

Aksi takdirde, ilk örneğinize dayanarak, temelde sihirli (bit) sayıları ikiye katladığınız için yorum eklemeniz gerekir, bu da kodu özellikle daha aşina olmayan başka bir kişi tarafından okunmasını zorlaştırır. Kendinizi altı ay boyunca bu kodu koruyacak olsanız bile, 5'in ne için kullanıldığını hatırlamak zor olabilir.

Answer 3

.NET 4.0 kullanıyorsanız, bir enum'un belirli bir bit içerip içermediğini kontrol etmek için HasFlag yöntemini kullanabilirsiniz. Bu, önceki kontrol yönteminden daha da okunabilir hale getirir.

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

Daha fazla bilgi için MSDN.