为什么 switch-case 语句中不能有变量？

Question

这是我的代码：

bool Character::keyPress(char c)
{
    switch(c)
    {
        case up_key:
            move(0, -1);
            break;

        case down_key:
            move(0, 1);
            break;

        case left_key:
            move(-1, 0);
            break;

        case right_key:
            move(1,0);
            break;

        default:
            return false;
    }

    return true;
}

编译器抱怨：

error C2051: case expression not constant
error C2051: case expression not constant
error C2051: case expression not constant
error C2051: case expression not constant

在我的头文件中，我有：

protected:
    char up_key;
    char down_key;
    char right_key;
    char left_key;

我正在使用 Visual C++ 2008。

Answer 1

正如错误消息所述，case 表达式必须是常量。编译器在编译时将其构建为一个非常快速的查找表，如果这些值有可能随着程序运行而改变，则它无法做到这一点。

如果您确实需要它们是可变的，而不是恒定的，那么最好的选择是使用 if/else 语句。

Answer 2

代码替换这个又长又笨拙的代码

switch(c)
{
    case up_key:
        move(0, -1);
        break;

    case down_key:
        move(0, 1);
        break;

    case left_key:
        move(-1, 0);
        break;

    case right_key:
        move(1,0);
        break;

    default:
        return false;
}

用这样的

move( (c==right_key) - (c==left_key) , (c==down_key) - (c==up_key) );

：您可以用更简洁的单行代码替换这 17 行长的代码。

Answer 3

你不能，因为语言不是这样工作的。例如，如果 up_key、down_key、right_key 和 left_key 全部相等，会发生什么情况？

Answer 4

由于 switch 语句只能采用常量，因此您在阅读代码时就知道要比较的内容都是常量。另一方面，您可以使用 if 语句（或其他结构）来与变量进行比较：

if (c == up_key) {
    move(0, -1);
} else if (c == down_key) {
    move(0, 1);
} else ...

这提供了结构上的明显差异，可以极大地帮助那些在您之后阅读代码的人。想象一下，如果您必须查找每个 case 标签来查看它是否是变量？

Answer 5

我相信这是因为编译器生成了一个跳转表，其中的值被硬编码，尽管我可能是错的。表的生成方式不允许这样做。

Answer 6

由于其他答案已经涵盖了您收到错误的原因，因此这里有一种方法可以在四个方向之一上移动以响应按键：使用查找表而不是条件/开关。

设置部分：

std::map<char,pair<int,int> > moves;
moves[up_key] = make_pair(0, -1);
moves[down_key] = make_pair(0, 1);
moves[left_key] = make_pair(-1, 0);
moves[right_key] = make_pair(1, 0);

功能：

bool Character::keyPress(char c) {
    if (moves.count(c)) {
        pair<int,int> dir = moves[c];
        move(dir.first, dir.second);
        return true;
    } else {
        return false;
    }
}

Answer 7

//here is the full functional code snippet which can be compiled and run with most of C++  
//compiler/link ...console app was demoed but you can apply the code/logic to win32 app...
//if you have any problem, send me email to [email protected]

#include <iostream.h>
#include <map>
#include <conio.h>

class CkbdHanler{
  private:
    map<char,pair<int,int> > moves;
  protected:
    char up_key;
    char down_key;
    char right_key;
    char left_key;
  public:

CkbdHanler(char a,char b,char c,char d):up_key(a),
                                        down_key(b),
                                       right_key(c),
                                       left_key(d)
{
    moves[up_key] = make_pair(0, -1);
    moves[down_key] = make_pair(0, 1);
    moves[left_key] = make_pair(-1, 0);
    moves[right_key] = make_pair(1, 0);
 }

bool keyPress(char c){
    if (moves.count(c)) {
            pair<int,int> dir = moves[c];
            move(dir.first, dir.second);
            return true;
   } else return false;

}
void move(int i,int j){
   cout<<"(i,j)=("<<i<<","<<j<<")"<<endl;
  }
};

int main(int argc, char* argv[])
{
  CkbdHanler CmyKbdH('u','d','l','r');

  cout << "Hello C++... here is a demo of Map to replace switch-case" << endl;
  CmyKbdH.keyPress('d');
  cout << endl << "Press any key to continue...";

  getch();
  return 0;
}