Visual Studio 在“case variable:”之后标记代码无法访问

Question

我有一个包含多种情况的 switch 语句：

ConversionState state = ConversionState.Start; // enum
for(int i = 0; i < source.Length; i ++)
{
    switch(state)
    {
        case ConversionState.Start:
            state = ConversionState.Name; // <-- warning here
            name += source[i]; // source is a string
            break;
        case ConversionState.Name:
            if(source[i] == ' ') // <-- warning here
            {
                name = name.ToLower();
                if(name[0] == '/')
                    name = name.SubString(1);
                state = ConversionState.Between;
            }
            else
                name += source[i];
            break;
        case ConversionState.Between: // no code in this case statement, yet to be implemented
            break; // <-- warning here
    }
}

我在三个标记的行上收到警告：“检测到无法访问的代码”。案例之后的第一个语句不能无法访问，不是吗？我的问题是：

1. 是我的代码有问题，还是警告有误？

2. 如果 VS2015 认为代码 sn-p 无法访问，当我在优化编译时它会被删除吗？如果没有，我应该忽略这个警告吗？

3. 此警告是否意味着标记的行不可达，或者整个案例不可达？

PS：我知道当前的代码可以在没有 switch 的情况下重写，这会解决问题，但是将来添加的代码将使它 - 非常 - 更容易使用 switch 来维护。

编辑（由 amit dayama 要求）：

private enum ConversionState
{
  Start, Between, Name, Argument, Switch
}

枚举位于具有包含原始发布代码的方法的类中。

编辑 2： 此代码所在方法的第一行是throw new NotImplementedException();。显然，这使得 Visual Studio 将方法中每个 switch 中每个 case 的第一行标记为不可访问，但没有其他内容，而且有趣的是，不是异常之后的整个代码。

Answer 1

我已经检查了您的代码，并且所有内容都使用 VS2012 在我这边编译和构建。我现在VS2015有一些问题，错误列表窗口中有很多误报。

所以问题 1：在我看来，您的代码是正确的。

问题 2：不完全确定，但相信 id 不会将其删除以进行优化。我可能是错的，但警告的原因是您解决了它而不是编译器。当我为你找到正确答案时，我会添加评论。

问题 3：一行不是整个案例。

PS：也许您可以尝试将花括号添加到您的一些 if 和 else 语句中。我知道它们不是一个衬垫所必需的，但它们可能会导致 VS2015 中的特定行为

Answer 2

您将在switch 语句中手动检查的变量设置为ConversionState.Start

这很简单，因为在switch 语句之前没有代码可以改变它，Visual Studio 会将其他情况标记为无法访问的代码。

如果我没记错的话，Visual Studio（或您使用的插件？）看不到用于switch 语句的变量在这种情况下正在被更改，从而引发警告。 并回答其余问题

如果 VS2015 认为代码 sn-p 无法访问，是否将其删除 当我进行优化编译时？如果没有，我应该忽略 这个警告？

据我所知，警告就是警告。它不会干扰代码的最终结果。它会警告您可能有问题。

此警告是否意味着标记的行无法到达，或者 整个案子都达不到？

整个案例都无法到达。在这种情况下，控制流永远不会得到，所以里面的所有代码都不会被执行。

编辑在代码审查方面，您的代码确实令人困惑。您使用特定案例来运行 source 集合中的第 0 个元素。为什么不在第 0 个元素上执行该代码并从下一个元素开始循环？

Answer 3

这对我来说似乎很奇怪。 首先，我不知道 ConversationState 的属性是什么。 如果它来自 Lync API，则属性似乎错误，实际是 Active、Parked、Inactive 和 Terminated。

所以我会做一个这样的开关盒

ConversionState state = ConversationState.Active;
for(int i = 0; i < source.Length; i ++)
{
  switch(state)
  {
    case ConversationState.Active:
      state = ConversionState.Name; 
      name += source[i]; 
      break;
    case ConversationState.Inactive:
      if(source[i] == ' ') 
      {
        name = name.ToLower();
        if(name[0] == '/')
          name = name.SubString(1);
        state = ConversionState.Between;
      }
      else
        name += source[i];
      break;
}

问题似乎您使用的案例形式错误。如果我错了，请纠正我。