Use one instead of five state variables to keep track of character creation progress

apps/openmw/mwgui/window_manager.cpp

@@ -33,11 +33,6 @@ WindowManager::WindowManager(MyGUI::Gui *_gui, MWWorld::Environment& environment
   , createClassDialog(nullptr)
   , birthSignDialog(nullptr)
   , reviewDialog(nullptr)
-  , nameChosen(false)
-  , raceChosen(false)
-  , classChosen(false)
-  , birthSignChosen(false)
-  , reviewNext(false)
   , gui(_gui)
   , mode(GM_Game)
   , nextMode(GM_Game)
@@ -47,6 +42,8 @@ WindowManager::WindowManager(MyGUI::Gui *_gui, MWWorld::Environment& environment
 {
     showFPSCounter = fpsSwitch;
 
+    creationStage = NotStarted;
+
     //Register own widgets with MyGUI
     MyGUI::FactoryManager::getInstance().registerFactory<DialogeHistory>("Widget");
 
@@ -187,7 +184,7 @@ void WindowManager::updateVisible()
         std::string sName = getGameSettingString("sName", "Name");
         nameDialog->setTextLabel(sName);
         nameDialog->setTextInput(playerName);
-        nameDialog->setNextButtonShow(nameChosen || reviewNext);
+        nameDialog->setNextButtonShow(creationStage >= NameChosen);
         nameDialog->eventDone = MyGUI::newDelegate(this, &WindowManager::onNameDialogDone);
         nameDialog->open();
         return;
@@ -198,7 +195,7 @@ void WindowManager::updateVisible()
         if (raceDialog)
             removeDialog(raceDialog);
         raceDialog = new RaceDialog(environment);
-        raceDialog->setNextButtonShow(raceChosen || reviewNext);
+        raceDialog->setNextButtonShow(creationStage >= RaceChosen);
         raceDialog->setRaceId(playerRaceId);
         raceDialog->eventDone = MyGUI::newDelegate(this, &WindowManager::onRaceDialogDone);
         raceDialog->eventBack = MyGUI::newDelegate(this, &WindowManager::onRaceDialogBack);
@@ -232,7 +229,7 @@ void WindowManager::updateVisible()
         if (pickClassDialog)
             removeDialog(pickClassDialog);
         pickClassDialog = new PickClassDialog(environment);
-        pickClassDialog->setNextButtonShow(classChosen || reviewNext);
+        pickClassDialog->setNextButtonShow(creationStage >= ClassChosen);
         pickClassDialog->setClassId(playerClass.name);
         pickClassDialog->eventDone = MyGUI::newDelegate(this, &WindowManager::onPickClassDialogDone);
         pickClassDialog->eventBack = MyGUI::newDelegate(this, &WindowManager::onPickClassDialogBack);
@@ -256,7 +253,7 @@ void WindowManager::updateVisible()
         if (birthSignDialog)
             removeDialog(birthSignDialog);
         birthSignDialog = new BirthDialog(environment);
-        birthSignDialog->setNextButtonShow(birthSignChosen || reviewNext);
+        birthSignDialog->setNextButtonShow(creationStage >= BirthSignChosen);
         birthSignDialog->setBirthId(playerBirthSignId);
         birthSignDialog->eventDone = MyGUI::newDelegate(this, &WindowManager::onBirthSignDialogDone);
         birthSignDialog->eventBack = MyGUI::newDelegate(this, &WindowManager::onBirthSignDialogBack);
@@ -266,7 +263,6 @@ void WindowManager::updateVisible()
 
     if (mode == GM_Review)
     {
-        reviewNext = false;
         if (reviewDialog)
             removeDialog(reviewDialog);
         reviewDialog = new ReviewDialog(environment);
@@ -476,16 +472,17 @@ void WindowManager::onNameDialogDone()
         removeDialog(nameDialog);
     }
 
-    bool goNext = nameChosen; // Go to next dialog if name was previously chosen
+    // Go to next dialog if name was previously chosen
-    nameChosen = true;
+    if (creationStage == ReviewNext)
-
-    if (reviewNext)
         setGuiMode(GM_Review);
-    else if (goNext)
+    else if (creationStage >= NameChosen)
         setGuiMode(GM_Race);
     else
+    {
+        creationStage = NameChosen;
         setGuiMode(GM_Game);
     }
+}
 
 void WindowManager::onRaceDialogDone()
 {
@@ -497,16 +494,17 @@ void WindowManager::onRaceDialogDone()
         removeDialog(raceDialog);
     }
 
-    bool goNext = raceChosen; // Go to next dialog if race was previously chosen
+    // Go to next dialog if race was previously chosen
-    raceChosen = true;
+    if (creationStage == ReviewNext)
-
-    if (reviewNext)
         setGuiMode(GM_Review);
-    else if (goNext)
+    else if(creationStage >= RaceChosen)
         setGuiMode(GM_Class);
     else
+    {
+        creationStage = RaceChosen;
         setGuiMode(GM_Game);
     }
+}
 
 void WindowManager::onDialogueWindowBye()
 {
@@ -755,7 +753,8 @@ void WindowManager::onClassQuestionChosen(int _index)
 
 void WindowManager::onGenerateClassBack()
 {
-    classChosen = true;
+    if(creationStage < ClassChosen)
+        creationStage = ClassChosen;
 
     if (generateClassResultDialog)
         removeDialog(generateClassResultDialog);
@@ -766,20 +765,21 @@ void WindowManager::onGenerateClassBack()
 
 void WindowManager::onGenerateClassDone()
 {
-    bool goNext = classChosen; // Go to next dialog if class was previously chosen
-    classChosen = true;
-
     if (generateClassResultDialog)
         removeDialog(generateClassResultDialog);
     environment.mMechanicsManager->setPlayerClass(generateClass);
 
-    if (reviewNext)
+     // Go to next dialog if class was previously chosen
+    if (creationStage == ReviewNext)
         setGuiMode(GM_Review);
-    else if (goNext)
+    else if (creationStage >= ClassChosen)
         setGuiMode(GM_Birth);
     else
+    {
+        creationStage = ClassChosen;
         setGuiMode(GM_Game);
     }
+}
 
 
 void WindowManager::onPickClassDialogDone()
@@ -795,16 +795,17 @@ void WindowManager::onPickClassDialogDone()
         removeDialog(pickClassDialog);
     }
 
-    bool goNext = classChosen; // Go to next dialog if class was previously chosen
+    // Go to next dialog if class was previously chosen
-    classChosen = true;
+    if (creationStage == ReviewNext)
-
-    if (reviewNext)
         setGuiMode(GM_Review);
-    else if (goNext)
+    else if (creationStage >= ClassChosen)
         setGuiMode(GM_Birth);
     else
+    {
+        creationStage = ClassChosen;
         setGuiMode(GM_Game);
     }
+}
 
 void WindowManager::onPickClassDialogBack()
 {
@@ -849,16 +850,17 @@ void WindowManager::onCreateClassDialogDone()
         removeDialog(createClassDialog);
     }
 
-    bool goNext = classChosen; // Go to next dialog if class was previously chosen
+    // Go to next dialog if class was previously chosen
-    classChosen = true;
+    if (creationStage == ReviewNext)
-
-    if (reviewNext)
         setGuiMode(GM_Review);
-    else if (goNext)
+    else if (creationStage >= ClassChosen)
         setGuiMode(GM_Birth);
     else
+    {
+        creationStage = ClassChosen;
         setGuiMode(GM_Game);
     }
+}
 
 void WindowManager::onCreateClassDialogBack()
 {
@@ -878,14 +880,15 @@ void WindowManager::onBirthSignDialogDone()
         removeDialog(birthSignDialog);
     }
 
-    bool goNext = birthSignChosen; // Go to next dialog if birth sign was previously chosen
+    // Go to next dialog if birth sign was previously chosen
-    birthSignChosen = true;
+    if (creationStage >= BirthSignChosen)
-
-    if (reviewNext || goNext)
         setGuiMode(GM_Review);
     else
+    {
+        creationStage = BirthSignChosen;
         setGuiMode(GM_Game);
     }
+}
 
 void WindowManager::onBirthSignDialogBack()
 {
@@ -918,7 +921,7 @@ void WindowManager::onReviewActivateDialog(int parDialog)
 {
     if (reviewDialog)
         removeDialog(reviewDialog);
-    reviewNext = true;
+    creationStage = ReviewNext;
 
     switch(parDialog)
     {

apps/openmw/mwgui/window_manager.hpp

@@ -100,14 +100,6 @@ namespace MWGui
     BirthDialog *birthSignDialog;
     ReviewDialog *reviewDialog;
 
-    // Which dialogs have been shown, controls back/next/ok buttons
-    bool nameChosen;
-    bool raceChosen;
-    bool classChosen;
-    bool birthSignChosen;
-    bool reviewNext;
-    ///< If true then any click on Next will cause the summary to be shown
-
     // Keeps track of current step in Generate Class dialogs
     unsigned generateClassStep;
     // A counter for each specialization which is increased when an answer is chosen, in order: Stealth, Combat, Magic
@@ -300,6 +292,19 @@ namespace MWGui
     void onReviewDialogDone();
     void onReviewDialogBack();
     void onReviewActivateDialog(int parDialog);
+
+    enum CreationStageEnum
+    {
+        NotStarted,
+        NameChosen,
+        RaceChosen,
+        ClassChosen,
+        BirthSignChosen,
+        ReviewNext
+    };
+
+    // Which state the character creating is in, controls back/next/ok buttons
+    CreationStageEnum creationStage;
   };
 
   template<typename T>