Thực hành tốt nhất để kiểm tra cho một lá cờ enum

Question

tôi nhận thấy hai mô hình này để kiểm tra cho một lá cờ enum:

[Flags] 
public enum PurchaseType 
{ 
    None = 0, 
    SalePrice = 2, 
    RegularPrice = 4, 
    Clearance = 8, 
    CreditCard = 16 
} 

public void Test() 
{ 
    PurchaseType type = PurchaseType.Clearance; 
    type |= PurchaseType.CreditCard; 

    // Practice 1 
    if ((type & PurchaseType.Clearance) == PurchaseType.Clearance) 
    { 
     // Clearance item handling 
    } 

    // Practice 2 
    if ((type & PurchaseType.CreditCard) != 0) 
    { 
     // Credit card item handling 
    } 
} 

Trong hai cách để kiểm tra cho một lá cờ enum, cái nào là tốt hơn hiệu suất wrt, dễ đọc, mã sức khỏe, và bất kỳ cân nhắc khác tôi nên thực hiện?

Cảm ơn, Mohammed

Answer 1

Net 4 giới thiệu một phương pháp HasFlag xác định liệu một hoặc nhiều lĩnh vực bit được thiết lập trong trường hợp hiện tại, đây là bởi đến nay các thực hành tốt nhất:

type.HasFlag(PurchaseType.CreditCard); // true 

Answer 2

tôi sẽ chọn cái đầu tiên:

if ((type & PurchaseType.Clearance) == PurchaseType.Clearance) 
{ 
    // Clearance item item handling 
} 

nguyên nhân rõ ràng Ngao rằng bạn đang kiểm tra cho Clearance loại hiện diện.

Answer 3

Cá nhân tôi sẽ luôn luôn thích có thể đọc rõ ràng về HasFlag.

Tuy nhiên, trong hai tùy chọn trong câu hỏi tôi nghĩ !=0 là an toàn hơn vì không có sự trùng lặp. Nếu bạn sử dụng thay thế của bạn thì tất cả quá dễ dàng khi mã hóa bảo trì để thay đổi một trong những lá cờ và quên thay đổi một lá cờ khác. Và sau đó bạn kết thúc với điều này

if ((type & PurchaseType.Clearance) == PurchaseType.CreditCard) 

Answer 4

Tôi muốn (type & PurchaseType.CreditCard) != 0 bởi vì nếu bạn muốn kiểm tra nhiều hơn một chút sau đó phía bên tay phải trở nên cồng kềnh. Tôi tin tưởng vào các hoạt động bit mà ở trên sẽ chỉ đúng nếu và chỉ khi bit (s) được thiết lập.