🔧 Added class for parsing command line parameters

src/CMakeLists.txt

@@ -13,6 +13,7 @@ set(SOURCE_FILES
         util/SignalMediator.hxx
         util/Singleton.hxx
         util/Meta.hxx
+        util/ParseCli.{hxx,cxx}
         util/Exception.{hxx,cxx}
         util/OSystem.{hxx,cxx}
         services/Randomizer.{hxx,cxx}

src/Game.cxx

@@ -78,7 +78,7 @@ void Game::initialize()
   LOG(LOG_DEBUG) << "Initialized Game Object";
 }
 
-void Game::run(bool SkipMenu)
+void Game::run()
 {
   Camera::instance().centerScreenOnMapCenter();
 

src/Game.hxx

@@ -44,7 +44,7 @@ public:
     * starts running the game
     * @param SkipMenu if the main menu should be skipped or not
     */
-  virtual void run(bool SkipMenu = false);
+  virtual void run();
 
   /** @brief ends the game
     * shuts down the game

src/engine/basics/Settings.hxx

@@ -166,6 +166,15 @@ struct SettingsData
 
   /// Write errors to a log file
   bool writeErrorLogFile;
+
+  // ==================================
+  // Command line options
+  // ==================================
+
+  /// Sets a different video driver
+  std::string videoDriver = "Default";
+
+  bool skipMenu = false;
 };
 
 /**

src/main.cxx

@@ -5,11 +5,13 @@
 #include "Exception.hxx"
 #include "LOG.hxx"
 #include "engine/WindowManager.hxx"
+#include "SimpleOpt.h"
+#include "ParseCli.hxx"
 
 #include <SDL.h>
 #include <SDL_ttf.h>
 
-bool initialize(const char *videoDriver)
+bool initialize()
 {
   if (SDL_Init(0) != 0)
   {
@@ -18,6 +20,10 @@ bool initialize(const char *videoDriver)
     return false;
   }
 
+  const char *videoDriver = nullptr;
+  if (Settings::instance().videoDriver != "Default")
+    videoDriver = Settings::instance().videoDriver.c_str();
+
   if (SDL_VideoInit(videoDriver) != 0)
   {
     LOG(LOG_ERROR) << "Unknown video driver " << videoDriver;
@@ -46,51 +52,31 @@ bool initialize(const char *videoDriver)
 
 int protected_main(int argc, char **argv)
 {
-  (void)argc;
-  (void)argv;
-
-  LOG(LOG_INFO) << VERSION;
-
-  // add commandline parameter to skipMenu
-  auto has_args = [argv, argc](const std::string &param)
-  {
-    for (int i = 1; i < argc; ++i)
-      if (param == argv[i])
-        return i;
-
-    LOG(LOG_DEBUG) << "Unknown game option " << param;
-    return 0;
-  };
-
-  bool skipMenu = has_args("--skipMenu");
-  uint32_t videoOpt = has_args("--video");
-  const char *videoDriver = nullptr;
-  if (videoOpt)
-  {
-    videoDriver = argv[videoOpt + 1];
-  }
+  ParseCli cli;
+  if (!cli.ProcessCommandLine(argc, argv))
+    return EXIT_FAILURE;
 
   LOG(LOG_DEBUG) << "Launching Cytopia";
 
   Cytopia::Game game;
 
   LOG(LOG_DEBUG) << "Initializing Cytopia";
 
-  if (!initialize(videoDriver))
+  if (!initialize())
     return EXIT_FAILURE;
   else
     LOG(LOG_DEBUG) << "DONE Cytopia";
 
   bool startGame = true;
-  if (!skipMenu)
+  if (!Settings::instance().skipMenu)
   {
     startGame = mainMenu();
   }
 
   if (startGame)
   {
     LOG(LOG_DEBUG) << "Running the Game";
-    game.run(skipMenu);
+    game.run();
   }
 
   LOG(LOG_DEBUG) << "Closing the Game";
@@ -117,4 +103,4 @@ int main(int argc, char **argv)
   }
 
   return EXIT_FAILURE;
-}
+}

src/util/ParseCli.cxx

@@ -0,0 +1,90 @@
+#include "ParseCli.hxx"
+#include "LOG.hxx"
+#include "Singleton.hxx"
+#include "Settings.hxx"
+
+// option identifiers
+enum
+{
+  OPT_HELP,
+  OPT_SKIPMENU,
+  OPT_VIDEODRIVER
+};
+
+// option array
+CSimpleOpt::SOption cmdline_options[] = {{OPT_SKIPMENU, ("--skipMenu"), SO_NONE},
+                                         {OPT_VIDEODRIVER, ("--video"), SO_REQ_SEP},
+                                         {OPT_HELP, ("--help"), SO_NONE},
+                                         {OPT_HELP, ("-h"), SO_NONE},
+                                         SO_END_OF_OPTIONS};
+
+bool ParseCli::ProcessCommandLine(int argc, char *argv[])
+{
+  CSimpleOpt args(argc, argv, cmdline_options);
+  bool success = true;
+  while (args.Next())
+  {
+    if (args.LastError() != SO_SUCCESS)
+    {
+      LOG(LOG_ERROR) << GetLastErrorText(args.LastError()) << " " << args.OptionText();
+      success = false;
+    }
+
+    switch (args.OptionId())
+    {
+    case OPT_HELP:
+      ShowUsage();
+      return false;
+    case OPT_SKIPMENU:
+      Settings::instance().skipMenu = true;
+      break;
+    case OPT_VIDEODRIVER:
+      if (args.OptionArg())
+      {
+        Settings::instance().videoDriver = args.OptionArg();
+      }
+      else
+      {
+        LOG(LOG_ERROR) << "videoDriver not set";
+        ShowUsage();
+        return false;
+      }
+      break;
+    default:
+      ShowUsage();
+    }
+  }
+
+  return success;
+}
+
+void ParseCli::ShowUsage()
+{
+  LOG(LOG_INFO) << "Usage: Cytopia [OPTIONS]";
+  LOG(LOG_INFO) << "\t--help (this)";
+  LOG(LOG_INFO) << "\t--skipMenu (Skips the main menu)";
+  LOG(LOG_INFO) << "\t--video <videoDriver> (Sets a different video driver)";
+}
+
+std::string ParseCli::GetLastErrorText(int a_nError)
+{
+  switch (a_nError)
+  {
+  case SO_SUCCESS:
+    return "Success";
+  case SO_OPT_INVALID:
+    return "Unrecognized option";
+  case SO_OPT_MULTIPLE:
+    return "Option matched multiple strings";
+  case SO_ARG_INVALID:
+    return "Option does not accept argument";
+  case SO_ARG_INVALID_TYPE:
+    return "Invalid argument format";
+  case SO_ARG_MISSING:
+    return "Required argument is missing";
+  case SO_ARG_INVALID_DATA:
+    return "Invalid argument data";
+  default:
+    return "Unknown error";
+  }
+}

src/util/ParseCli.hxx

@@ -0,0 +1,18 @@
+#ifndef CYTOPIA_PARSECLI_HXX
+#define CYTOPIA_PARSECLI_HXX
+
+#include "SimpleOpt.h"
+#include <string>
+
+class ParseCli
+{
+public:
+  bool ProcessCommandLine(int argc, char **argv);
+
+private:
+  std::string GetLastErrorText(int a_nError);
+
+  void ShowUsage();
+};
+
+#endif //CYTOPIA_PARSECLI_HXX