在枚舉值之間切換：C++

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 

您立即返回蘭德曼（因爲你執行你的switch語句之前設置工作，蘭德曼，它總是返回蘭德曼已將其編碼方式），它完全返回你從這個功能。它從不嘗試將工作轉換爲任何其他工作。請注意，您的break語句也永遠不會執行，因爲返回將立即從此函數調用中返回。您可能需要：

case Landman: 
    job = geologist; 
    return job; 

等等等等。此外，你很難將默認的工作案例編碼給蘭德曼。您可能想要作爲變量傳遞，或者從對象中讀出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;。