两个成员枚举与布尔值

Question

这个问题在我脑海里已经有一段时间了，如果它看起来很主观，很抱歉。在数据对象的公共属性和构造函数中使用 bool 有一些缺点。以下面的代码为例。

使用布尔：

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

    public Room(string name, bool bookable);
}

以及这个类的使用

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

这是适当的功能，但是还有另一种实现方式：

使用枚举：

public enum BookingStatus
{
    Bookable,
    NotBookable
}

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

    public Room(string name, BookingStatus bookable);
}

以及这个类的使用

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

在我看来，这两者在功能上是相同的，但各有优缺点：

• 设置属性时，Enum 方法更加冗长（您可以仅从代码中推断出 enum 的用法）
• 可以扩展枚举以支持更多状态（对 API 特别有用）
• 枚举需要更多的输入（尽管大大减少了）
• 枚举不能在条件句中使用（即 if (r.bookable)），尽管我知道这很容易解决。

我是否遗漏了什么，完全不合时宜？我不知道为什么这让我如此烦恼，也许我太强迫症了！

Answer 1

仅仅因为可读性和对代码的理解，我将使用枚举而不是布尔值。

比较BookingStatus.Bookable和true，当然你会明白更多阅读BookingStatus.Bookable。

也像 fforw 提到的那样，以防将来您可能需要添加更多选项，枚举会更容易更改。

Answer 2

我有可能在未来可能会比最初的两个选项更多，向枚举添加第三个选项比将所有 bool 更改为枚举要少得多。

Answer 3

对于房产，我可能会同意。但是，在“清洁代码”中，（正确地）表明bools 在用作参数时隐藏了意图。例如，格式化方法可能如下所示：

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

调用时如下所示：

Format(true, false, true);

相反，它会更具可读性：

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)
{ ... }

然后调用看起来像：

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

（请记住，这只是 bool 参数如何隐藏意图的示例；可能有更好的方法来实现上述代码。）

在您的特定情况下，请注意两个构造函数调用之间的区别：

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

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

Answer 4

在他的书Refactoring 中，Martin Fowler 解释了为什么他认为枚举是一种代码味道，我只能同意。在您的示例中，更好的方法是创建一个抽象 Room 类：

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

     public abstract bool Bookable { get; }
}

然后您可以创建派生的 BookableRoom 和 NonBookableRoom 类。

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

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