Hai điều tra thành viên so với giá trị boolean

Question

Câu hỏi này có trong tâm trí tôi một thời gian, xin lỗi nếu nó xuất hiện chủ quan. Có một số nhược điểm trong việc sử dụng bool trong các thuộc tính công cộng và các hàm tạo cho các đối tượng dữ liệu. Hãy xem xét mã sau đây làm ví dụ.

Sử dụng bool:

public class Room 
{ 
    public string Name { get; set; } 
    public bool Bookable { get; set; } 

    public Room(string name, bool bookable); 
} 

và việc sử dụng các lớp này

Room r = new Room ("101", true); 

Đây là phù hợp chức năng, có tuy nhiên một cách khác để thực hiện nó:

Sử dụng enum :

public enum BookingStatus 
{ 
    Bookable, 
    NotBookable 
} 

public class Room 
{ 
    public string Name { get; set; } 
    public BookingStatus Bookable { get; set; } 

    public Room(string name, BookingStatus bookable); 
} 

và việc sử dụng các lớp

Room r = new Room ("101", BookingStatus.Bookable); 

này Đối với tôi hai xuất hiện chức năng tương đương, có một số ưu điểm và nhược điểm của mỗi tuy nhiên:

• Khi thiết lập các thuộc phương pháp Enum là hơn verbose (bạn có thể phỏng đoán việc sử dụng enum từ mã một mình)
• Các số đếm có thể được mở rộng để hỗ trợ các trạng thái khác (đặc biệt hữu ích cho một API)
• Các số đếm yêu cầu nhập nhiều hơn đáng kể (mặc dù giảm rất nhiều)
• Không thể sử dụng các số đếm trong điều kiện (tức là nếu (r.bookable)), mặc dù tôi đánh giá cao điều này là tầm thường để giải quyết.

Tôi có thiếu thứ gì đó, hoàn toàn không có dấu? Tôi không chắc tại sao điều này lại gây lỗi cho tôi rất nhiều, có lẽ tôi cũng quá OCD vì lợi ích của chính tôi!

Answer 1

Trong cuốn sách Refactoring, Martin Fowler giải thích lý do tại sao anh ấy nghĩ rằng enums là một mùi mã, và tôi chỉ có thể đồng ý. Trong ví dụ của bạn, cách tiếp cận tốt hơn là tạo lớp Tóm tắt trong phòng:

public abstract class Room 
{ 
    public string Name { get; set; } 

    public abstract bool Bookable { get; } 
} 

Sau đó, bạn có thể tạo ra lớp BookableRoom và NonBookableRoom.

public class BookableRoom : Room 
{ 
    public override bool Bookable 
    { 
     get { return true; } 
    } 
} 

public class NonBookableRoom : Room 
{ 
    public override bool Bookable 
    { 
     get { return false; } 
    } 
} 

Answer 2

Ih có bất kỳ cơ hội mà trong tương lai có thể có nhiều hơn hai lựa chọn ban đầu, thêm một lựa chọn thứ ba để một enum là ít hơn rất nhiều công việc sau đó thay đổi tất cả bool để enum.

Answer 3

Đơn giản chỉ vì khả năng đọc và hiểu mã, tôi sẽ chuyển sang enum thay vì boolean.

So sánh BookingStatus.Bookable và true, tất nhiên bạn sẽ hiểu thêm đọc BookingStatus.Bookable.

Cũng giống như những gì fforw đã đề cập, trong trường hợp trong tương lai bạn có thể cần thêm nhiều tùy chọn hơn, enum sẽ dễ thay đổi hơn.

Answer 4

Đối với tài sản, tôi có thể đồng ý. Trong 'Mã sạch', mặc dù, nó được chỉ định (chính xác) rằng mục đích ẩn bool s khi được sử dụng làm tham số.Ví dụ, một phương pháp định dạng có thể trông giống như:

public void Format(bool pBold, bool pItalic, bool pUnderline) 
{ ... } 

Khi gọi này trông giống như:

Format(true, false, true); 

Thay vào đó nó sẽ được nhiều dễ đọc hơn như:

public enum BOLD { BOLD, NOT_BOLD } 
public enum ITALIC { ITALIC, NOT_ITALIC } 
public enum UNDERLINE { UNDERLINE, NOT_UNDERLINE } 

public void Format(BOLD pBold, ITALIC pItalic, UNDERLINE pUnderline) 
{ ... } 

Mà sau đó làm cho gọi điện thoại trông giống như:

Format(BOLD.BOLD, ITALIC.NOT_ITALIC, UNDERLINE.NOT_UNDERLINE); 

(Hãy nhớ rằng đây chỉ là một ví dụ về cách các tham số bool ẩn ý định; . Có những cách có lẽ tốt hơn để thực hiện các mã trên)

Trong trường hợp cụ thể của bạn, hãy chú ý sự khác biệt giữa hai constructor gọi:

// enum; intent is clear 
Room r = new Room ("101", BookingStatus.Bookable); 

// bool; true what? 
Room r = new Room("101", true);