Question

我正在为Visual Studio C＃控制台中的课程编写文本冒险，并决定使用switch语句而不是if-elses链，因为goto情况有多大（到目前为止它已经非常有效））。我知道交换机本身的每个案例都必须是一个常量，但我想知道这是否也延伸到使用goto情况。例如，我有：

switch (location)
{
case 1:
  break;
case 2:
  break;
case 3:
  break;
//I have 10 cases, each representing a location such as "Orc Cave", I just cut it down for brevity
default:
  break;
}

我希望能够输入一个整数变量，然后转到那个整数，我有以下内容来实现：

string travel2 = Console.ReadLine();//inputs a integer representing each location
int travel2A = Convert.ToInt32(travel2);
if (1<=travel2A && travel2A<=10)
{
    goto case(travel2A);
}
else{
    goto case(2);//current location
}

一切正常但是有一个“预期值不变”的警告会破坏案例（travel2A）。是否可以通过一些调整使goto case输入变量，或者只是switch语句的限制？如果是后者，我可以做一个if-elses链，但在我看来，输入变量更方便。非常感谢任何帮助！谢谢！

Answer 1

普通goto的标签需要在源代码中拼写出来。它们不是被评估的表达式;它们只是标识符。 goto case的标签必须是常量表达式，在编译时评估，而不是在运行时评估。

我会提醒你不要任何严重依赖任何形式的“转到”的解决方案。对任何一种“goto”都有一种流行的偏见;相信它们不够优雅，使你的代码难以理解。虽然C＃经过精心设计，但最糟糕的“goto”滥用是不可能或不太可能的，但这种偏见确实存在一些真相。（C＃只允许在同一个块内或从内部块到其中一个包含块的getos。它永远不允许从外部块转到内部块，或者在两个没有嵌套关系的块之间。这大大降低了可能性“意大利面条代码”。）

我认为，因为你正在上一门课程，所以你才开始学习C＃。随着您的进步，您将学习比开启案例更优雅的控制流技术。

虽然我们批评您的代码：考虑使用int.TryParse而不是Convert.ToInt32，并确保处理用户输入非整数内容的情况。

Answer 2

见POST

基本上，开关不能在case语句中包含evaluate语句。必须对它们进行静态评估。

Answer 3

怎么样：

int travel2A = Convert.ToInt32(travel2);
if( travel2A < 1 || travel2A > 10 )
    travel2A = 2;

switch( travel2A ) { ... }

Answer 4

编译器错误A constant value is expected说明了一切。

在调试模式下运行时，可以很好地演示在C＃中无法实现这一点的原因（VB Select操作略有不同）。每当遇到switch语句时，您都会注意到代码将跳转到所选的大小写。虽然通过跳过几个比较获得了显着的性能提升，但它限制了灵活性。

我过去常常采用的一种方法是使用自治方法进行递归。因此，您将使用可重新调用goto case(travel2a)语句的方法调用，而不是使用switch。

Action<int> foo = null;
foo = (i) =>
{
    switch (i)
    {
        case 1:
            Console.WriteLine("1");
            break;
        case 2:
            Console.WriteLine("1");
            break;
        case 3:
            Console.WriteLine("1");
            break;
        default:
            //goto case (i%3);
            foo(i % 3);
            break;
    }
};

foo(4);

Answer 5

您可以将case代码组织为一种方法，只需调用它：

switch (location)
{
    case 1:
        gocave();
        break;
    case 2:
        gocave();
        break;
    case 3:
        donotgocave();
        break;
}

或将复杂逻辑拆分为基元：

bool gocave = false;
bool eatmushroom = false;
switch (location)
{
    case 1:
        gocave = true;
        break;
    case 2:
        gocave = true;
        eathmushroom = true;
        break;
    case 3:
        break;
}
if(gocave) {...}
if(eatmushroom) {...}

使用goto case（变量）;在C＃开关

5 个答案: