Question

这应该是一个简单的问题。我没有处理过很多内容，所以我不完全了解它们是如何工作的。在我的程序中，我试图将玩家的工作存储在枚举中。我需要可以更改作业的功能。我正在尝试使用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语句之前将作业设置为Landman，它将始终按照你编码的方式返回Landman），这将完全从这个函数返回。它从不试图将工作转换为任何其他工作。请注意，您的break语句也永远不会执行，因为返回会立即从此函数调用返回。你可能想要这个：

case Landman:
   job = geologist;
   return job;

等等。此外，您正在将默认工作案例硬编码为Landman。您可能想要作为变量传递，或者从对象中读取，Job的CURRENT值在播放器工作表上，然后根据它的当前值进行调整。

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;。

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

2 个答案: