optional/mandatory reader for activity_level

To allow using optional/mandatory for types that need to read activity
level.

Remove some obsolete code too.

Author: ehughsbaird

src/activity_type.cpp

@@ -99,15 +99,7 @@ void activity_type::load( const JsonObject &jo )
     optional( jo, was_loaded, "do_turn_eoc", do_turn_EOC );
     optional( jo, was_loaded, "ignored_distractions", default_ignored_distractions_ );
     optional( jo, was_loaded, "based_on", based_on_ );
-
-    auto act_level_it = activity_levels_map.find( jo.get_string( "activity_level", "undefined" ) );
-    if( act_level_it == activity_levels_map.end() ) {
-        debugmsg( "activity_type '%s' has invalid activity_level '%s', defaulting to 'LIGHT_EXERCISE'",
-                  result.id().c_str() );
-        activity_level = activity_levels_map.at( "LIGHT_EXERCISE" );
-    } else {
-        activity_level = act_level_it->second;
-    }
+    mandatory( jo, was_loaded, "activity_level", activity_level, activity_level_reader{} );
 }
 
 void activity_type::load_all( const JsonObject &jo )

src/construction.cpp

@@ -32,6 +32,7 @@
 #include "flexbuffer_json.h"
 #include "game.h"
 #include "game_constants.h"
+#include "generic_factory.h"
 #include "input.h"
 #include "input_context.h"
 #include "inventory.h"
@@ -2269,14 +2270,8 @@ void load_construction( const JsonObject &jo )
         con.post_is_furniture = true;
     }
 
-    std::string activity_level = jo.get_string( "activity_level", "MODERATE_EXERCISE" );
-    const auto activity_it = activity_levels_map.find( activity_level );
-    if( activity_it != activity_levels_map.end() ) {
-        con.activity_level = activity_it->second;
-    } else {
-        jo.throw_error( string_format( "Invalid activity level %s in construction %s", activity_level,
-                                       con.str_id.str() ) );
-    }
+    optional( jo, false/*con.was_loaded*/, "activity_level", con.activity_level,
+              activity_level_reader{}, MODERATE_EXERCISE );
 
     if( jo.has_member( "pre_flags" ) ) {
         con.pre_flags.clear();

src/generic_factory.cpp

@@ -1,6 +1,7 @@
 #include "generic_factory.h"
 
 #include "catacharset.h"
+#include "game_constants.h"
 #include "output.h"
 #include "wcwidth.h"
 
@@ -75,3 +76,15 @@ float read_proportional_entry( const JsonObject &jo, std::string_view key )
     }
     return 1.0f;
 }
+
+float activity_level_reader::get_next( const JsonValue &jv ) const
+{
+    if( !jv.test_string() ) {
+        jv.throw_error( "Invalid activity level" );
+    }
+    auto it = activity_levels_map.find( jv.get_string() );
+    if( it == activity_levels_map.end() ) {
+        jv.throw_error( "Invalid activity level" );
+    }
+    return it->second;
+}

src/generic_factory.h

@@ -1826,6 +1826,12 @@ class text_style_check_reader : public generic_typed_reader<text_style_check_rea
         allow_object object_allowed;
 };
 
+class activity_level_reader : public generic_typed_reader<activity_level_reader>
+{
+    public:
+        float get_next( const JsonValue &jv ) const;
+};
+
 struct dbl_or_var;
 
 class dbl_or_var_reader : public generic_typed_reader<dbl_or_var>

src/move_mode.cpp

@@ -8,7 +8,6 @@
 #include "assign.h"
 #include "debug.h"
 #include "flexbuffer_json.h"
-#include "game_constants.h"
 #include "generic_factory.h"
 #include "translations.h"
 
@@ -57,11 +56,7 @@ void move_mode::load( const JsonObject &jo, std::string_view/*src*/ )
     assign( jo, "panel_color", _panel_color );
     assign( jo, "symbol_color", _symbol_color );
 
-    std::string exert = jo.get_string( "exertion_level" );
-    if( !activity_levels_map.count( exert ) ) {
-        jo.throw_error_at( id.str(), "Invalid activity level for move mode " + id.str() );
-    }
-    _exertion_level = activity_levels_map.at( exert );
+    mandatory( jo, was_loaded, "exertion_level", _exertion_level, activity_level_reader{} );
 
     mandatory( jo, was_loaded, "change_good_none", change_messages_success[steed_type::NONE] );
     mandatory( jo, was_loaded, "change_good_animal", change_messages_success[steed_type::ANIMAL] );

src/recipe.cpp

@@ -283,19 +283,7 @@ void recipe::load( const JsonObject &jo, const std::string &src )
     }
 
     // Mandatory: This recipe's exertion level
-    mandatory( jo, was_loaded, "activity_level", exertion_str );
-    // Remove after 0.H
-    if( exertion_str == "fake" ) {
-        debugmsg( "Depreciated activity level \"fake\" found in recipe %s from source %s. Setting activity level to MODERATE_EXERCISE.",
-                  id.c_str(), src );
-        exertion_str = "MODERATE_EXERCISE";
-    }
-    const auto it = activity_levels_map.find( exertion_str );
-    if( it == activity_levels_map.end() ) {
-        jo.throw_error_at(
-            "activity_level", string_format( "Invalid activity level %s", exertion_str ) );
-    }
-    exertion = it->second;
+    mandatory( jo, was_loaded, "activity_level", exertion, activity_level_reader{} );
 
     // Never let the player have a debug or NPC recipe
     if( jo.has_bool( "never_learn" ) ) {