在枚举值之间切换：C ++

如何解决在枚举值之间切换：C ++

| 希望这应该是一个简单的问题。我没有太多处理枚举，所以我不完全了解它们的工作原理。在我的程序中，我试图将播放器的工作存储在枚举中。我需要可以更改作业的功能。我正在尝试使用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语句中。     

解决方法

你的逻辑是错误的。

switch(job)

陈述将您带入案例陈述。在里面

case Landman

您立即返回Landman（因为在执行switch语句之前将作业设置为Landman，它将始终以您的编码方式返回Landman），这将使您完全从该函数返回。它从不尝试将作业更改为任何其他作业。请注意，您的break语句也不会执行，因为return会立即从此函数调用中返回您。您可能想要这样：

case Landman:
   job = geologist;
   return job;

等等等等。此外，您正在将默认的工作案例硬编码为Landman。您可能想要作为变量传递，或者从对象中读出Job的当前值在播放器表上的大小，然后根据其当前值进行调整。     ,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

走相同的路径。整个功能相当于ѭ11。