检查,如果所有的枚举值被处理

假设有一段代码:

enum Directions
{
    North,
    South,
    East,
    West
}

// (...)

switch (dir)
{
    case North : // Do sth
    case South : // Do sth
    case East : // Do sth
    case West : // Do sth
}

特别注意缺少default:指令。 现在假设,稍后有人为该枚枚举增加了四个值: NorthEastNorthWestSouthEastSouthWest 。 在这种情况下,引用的代码很可能会影响未定义的行为,因为没有安全措施来处理Directions枚举类型的可能扩展。

我有关于这件事的一些问题。

  • 程序员应该在这里实施安全措施,例如:

    default: throw new InvalidArgumentException("Unsupported enum value!");
    
  • 这个问题应该在单元测试中进行测试吗?

    // Naïve implementation, for example purposes only
    [ExpectedException(typeof(InvalidArgumentException))]
    [Test]
    void SomeTestMethod()
    {
        SomeFunction((Direction)-1);
    }
    

    如果是这样,你会如何实现这种类型的测试自动化(例如,找到与此枚举中的所有其他值不同的序数值,考虑枚举后备类型等)?

  • 在静态代码分析期间是否应该验证此事? 哪个工具可以提供这样的信息? 我在VS 2012 Pro中编写了简单的概念验证,运行代码分析,但IDE报告说'没有检测到代码分析问题'。

    代码:

    namespace ManagedConsoleSketchbook
    {
        class Program
        {
            enum MyEnum
            {
                One,
                Two,
                Three
            }
    
            static void Main(string[] args)
            {
                MyEnum e = MyEnum.One;
    
                switch (e)
                {
                    case MyEnum.One:
                        break;
                    case MyEnum.Two:
                        break;
                }
            }
        }
    }
    
  • 这个普遍问题是否有一个通用的方法? 例如。 也许我可以在C#中使用反射来编写复杂的测试方法,但在C ++中,确定这种潜在的代码完整性违反要比在C#或其他更高级别的语言中困难得多。


  • 为什么我会问在单元测试期间是否应该检查它? 这是因为 - 据我所知 - 单元测试应该验证,如果方法的行为像它应该(记录)一样。 在这种情况下,它应该接受四个枚举元素中的任何一个,它实际上是这样做的,所以所有的测试都应该通过。 在我看来,检查潜在的问题并不是单元测试过程的问题。 如果我想验证单元测试中潜在问题的所有可能性,我最终可能会为编写3行代码的方法编写数百个测试。

    另一方面,如果有人决定为枚举添加另一个值,那么它似乎是一个相当便宜(以资源消耗方式)的方式,确保该方法不会导致任何严重问题。


    1:是的,我经常这样做。

    2:是的。 我会补充一点:

    [ExpectedException(typeof(InvalidArgumentException))]
    [Test]
    void SomeTestMethod()
    {
        Direction testValue = (Direction)-1;
        Assert.IsFalse(Enum.IsDefined(typeof(Direction), testValue));
        SomeFunction((Direction)-1);
    }
    

    这个问题实际上是在VS-2012中处理的。 请注意,此测试在默认情况下是禁用的。

    启用警告后,您可以指示Vs将此特定警告视为错误。


  • 这将是理想的测试。
  • 我不知道一个可以做到的工具。
  • 最简单的方法是始终有一个默认条款。 当默认不应该做任何事情时写下如下内容:

    default:
        // No action required
        break;
    

    这样,程序员至少会考虑剩下的项目(如果有的话)。

  • 链接地址: http://www.djcxy.com/p/73381.html

    上一篇: Checking, if all enum values were processed

    下一篇: Mbunit Factory attribute with ExpectedException