c# - 处理意外枚举值的首选方法是什么？

Question

假设我们有一个接受枚举值的方法。在该方法检查该值是否有效后，它会切换可能的值。所以问题是，在验证值范围之后，处理意外值的首选方法是什么？

例如:

enum Mood { Happy, Sad }

public void PrintMood(Mood mood)
{
    if (!Enum.IsDefined(typeof(Mood), mood))
    {
        throw new ArgumentOutOfRangeException("mood");
    }

    switch (mood)
    {
        case Happy: Console.WriteLine("I am happy"); break;
        case Sad:   Console.WriteLine("I am sad"); break;
        default: // what should we do here?
    }

处理 default 情况的首选方法是什么？

• 发表评论，如//永远不会发生
• Debug.Fail()(或Debug.Assert(false))
• 抛出新的 NotImplementedException()(或任何其他异常)
• 其他一些我没有想到的方法

Answer 1

我想上面的大部分答案都是有效的，但我不确定是否有任何答案是正确的。

正确答案是，您很少切换到 OO 语言，这表明您在 OO 方面做错了。在这种情况下，这是您的 Enum 类存在问题的完美指示。

您应该只调用 Console.WriteLine(mood.moodMessage())，并为每个状态定义 moodMessage。

如果添加了新状态——您的所有代码都应自动适应，则不应出现任何失败、抛出异常或需要更改的情况。

编辑:对评论的回复。

在您的示例中，要成为“良好的 OO”，文件模式的功能将由 FileMode 对象控制。它可以包含一个具有“打开、读取、写入...”操作的委托(delegate)对象，每个 FileMode 都不同，因此 File.open("name", FileMode.Create) 可以实现为(抱歉不熟悉API):

open(String name, FileMode mode) {
    // May throw an exception if, for instance, mode is Open and file doesn't exist
    // May also create the file depending on Mode
    FileHandle fh = mode.getHandle(name);
    ... code to actually open fh here...
    // Let Truncate and append do their special handling
    mode.setPosition(fh);
}

这比尝试使用开关更简洁...(顺便说一下，这些方法将是包私有(private)的并且可能委托(delegate)给“模式”类)

当 OO 做得很好时，每个方法看起来就像几行真正易于理解的简单代码——太简单了。您总是感觉有一些杂乱的大“奶酪核”将所有小的 Jade 米片对象聚集在一起，但您永远找不到它——它一直都是 Jade 米片...