在枚举值之间切换:c++

Question

这应该是一个简单的问题。我没有接触过枚举，所以我不完全理解它们是如何工作的。在我的程序中，我试图将玩家的工作存储在enum中。我需要的功能，工作可以改变。我试图使用一个switch语句，但它并没有改变玩家的工作。

代码:

// if change is True, change the job. If false, simply return the current value
int GameModeState::changeJob(bool change)
{
    int job = Landman; // Default job is landman
    if (change == true)
    {
        switch(job)
        {
        case Landman:
            return job;
            break;
        case Geologist:
            return job;
            break;
        default:
            job = Landman;
            return job;
            break;
        }
    } 
    else 
    {
        return job;
    }
}
// when the player opens the stat sheet, it should change their job
void GameModeState::_statsheet()
 {
     changeJob(true);
 }

我做错了什么让工作改变?我想问题出在switch语句

Answer 1

你的逻辑错了。switch(job)语句将带您进入案例语句。在

case Landman

您立即返回Landman(因为您在执行switch语句之前将job设置为Landman，它将始终以您编码的方式返回Landman)，这将从该函数完全返回您。它从不试图将工作更改为任何其他工作。请注意，break语句也永远不会执行，因为return会立即从这个函数调用返回。你可能想要这样:

case Landman:
   job = geologist;
   return job;

等等。此外，您将默认作业用例硬编码给Landman。你可能想要传递一个变量，或者从一个对象中读取玩家表中Job的当前值，然后根据它的当前值进行调整。

Answer 2

switch语句是好的。你有一堆无用的break语句，但没有引起问题(除了使代码可读性降低)。

这就是问题所在:

// if change is True, change the job. If false, simply return the current value

job是一个局部变量，设置它的唯一效果是后面的return job;语句。你的评论应该是:

// if change is true, return the new job. If false, simply return the current value

返回job的新值，但丢弃返回值。

当然，您总是设置job = Landman，这意味着您总是通过switch的相同路径。整个函数相当于return Landman;