两个成员枚举与布尔值

Question

这个问题在我的脑海里已经有一段时间了,对不起,如果它看起来是主观的.在公共属性中使用bool和在数据对象中使用构造函数存在一些缺点.请考虑以下代码作为示例.

使用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方法更详细(您可以仅从代码中推断枚举的用法)
• 可以扩展枚举以支持更多状态(对API特别有用)
• 枚举需要更多的输入(虽然大大减少了)
• 枚举不能在条件中使用(即if(r.bookable)),尽管我理解这是微不足道的.

我错过了什么,完全不合适吗？我不知道为什么这会让我这么烦恼,也许我对自己的好处太强迫了!

Answer 1

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

比较BookingStatus.Bookable和true,当然你想了解更多的阅读BookingStatus.Bookable.

也像fforw所提到的那样,如果将来你可能需要添加更多选项,枚举会更容易改变.

Answer 2

是否有可能在未来可能有超过最初的两个选项,为枚举添加第三个选项是更少的工作然后将所有bool更改为枚举.

Answer 3

对于财产我可能会同意。不过，在“干净代码”中，（正确地）表明 sbool在用作参数时隐藏意图。例如，格式化方法可能如下所示：

public void Format(
   bool bold,
   bool italic,
   bool underline
) {
   ...
}

调用时看起来像：

Format(true, false, true);

相反，它会更具可读性：

public enum Bold { Yes, No }
public enum Italic { Yes, No }
public enum Underline { Yes, No }

public void Format(
   Bold bold,
   Italic italic,
   Underline underline
) {
   ...
}

这使得调用看起来像：

Format(Bold.Yes, Italic.No, Underline.Yes);

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

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

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

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

这也可以通过使用参数名称来改进：

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

Answer 4

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