在枚举值之间切换: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的当前值在播放器表上的大小,然后根据其当前值进行调整。     

bab

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。     

要回复问题请先登录注册