diff --git a/include/billy_objects.hpp b/include/billy_objects.hpp index 8e3125d..2d2e7d2 100644 --- a/include/billy_objects.hpp +++ b/include/billy_objects.hpp @@ -10,46 +10,21 @@ #include #include #include +#include #include #include "characteristic/characteristic.hpp" +#include "generic_object.hpp" -// helper type for the visitor -template -struct overloaded : Ts ... { using Ts::operator()...; }; -template -overloaded(Ts...) -> overloaded; namespace character { class CharacterSheet; - enum class weapons : std::uint8_t { - Sword = 0, - Lance = 1, - Morgenstern = 2, - Bow = 3 - }; - - enum class equipments : std::uint8_t { - Chainmail = 4, - CookingPot = 5, - PamphletTourist = 6, - MedicKit = 7 - }; - - enum class tools : std::uint8_t { - Fourche = 8, - Dagger = 9, - RockClimbingKit = 10, - SackOfGrain = 11 - }; - class BillyObjects final { public: static constexpr std::size_t max_num_obj{ 3 }; - using billyObject = std::variant; - using container = ankerl::svector; + using container = std::unordered_map; - static constexpr std::array all_objects{ + static constexpr std::array all_objects{ weapons::Sword, weapons::Lance, weapons::Morgenstern, @@ -66,22 +41,7 @@ namespace character { static constexpr std::string_view json_key{ "billy_objects" }; - static constexpr std::string_view sword{ "Sword" }; - static constexpr std::string_view lance{ "Lance" }; - static constexpr std::string_view morgenstern{ "Morgenstern" }; - static constexpr std::string_view bow{ "Bow" }; - - static constexpr std::string_view chainmail{ "Chainmail" }; - static constexpr std::string_view cooking_pot{ "Cooking pot" }; - static constexpr std::string_view pamphlet_tourist{ "Touristic pamphlet" }; - static constexpr std::string_view medic_kit{ "Medic kit" }; - - static constexpr std::string_view fourche{ "Fourche" }; - static constexpr std::string_view dagger{ "Dagger" }; - static constexpr std::string_view rock_climbing_kit{ "Rock climbing kit" }; - static constexpr std::string_view sack_of_grain{ "Sack of grain" }; - - static std::string_view billy_object_to_string(const billyObject &object) noexcept; + static std::string_view billy_object_to_string(const billyObjects &object) noexcept; static void to_json(json &j, const BillyObjects::container &billy); @@ -91,47 +51,30 @@ namespace character { ~BillyObjects() noexcept = default; - [[nodiscard]] bool push_object(const billyObject &object, CharacterSheet &sheet) noexcept; + [[nodiscard]] bool insert_object(CharacterSheet &sheet, const billyEnums objType) noexcept; - void pop_object(CharacterSheet &sheet) noexcept; - - [[nodiscard]] static ankerl::svector check_conformity(const CharacterSheet &sheet) noexcept; + void erase_object(CharacterSheet &sheet, const billyEnums objToErase) noexcept; [[nodiscard]] const std::plus &get_plus_operation() const { return plus; } [[nodiscard]] const std::minus &get_minus_operation() const { return minus; } - private: - std::plus plus; - std::minus minus; - - static void change_carac_weapon(const weapons &arg, - CharacterSheet &sheet, - characteristic::Characteristic &localHabilete, - characteristic::Characteristic &localAdresse, - characteristic::Characteristic &localEndurance, - const std::function &operation) noexcept; - - static void change_carac_equipment(const equipments &arg, - CharacterSheet &sheet, - characteristic::Characteristic &localHabilete, - characteristic::Characteristic &localAdresse, - characteristic::Characteristic &localEndurance, - characteristic::Characteristic &localChance, - const std::function &primary, - const std::function &complement) noexcept; - - static void change_carac_tools(const tools &arg, - CharacterSheet &sheet, - characteristic::Characteristic &localHabilete, - characteristic::Characteristic &localAdresse, - characteristic::Characteristic &localEndurance, - characteristic::Characteristic &localChance, - const std::function &operation) noexcept; - static void check_dagger_conditions(const CharacterSheet &sheet, characteristic::Characteristic &localHabilete, const std::function &operation); + + private: + std::plus plus; + + std::minus minus; + + static void change_carac(const billyObjects &arg, + CharacterSheet &sheet, + characteristic::Characteristic &localHabilete, + characteristic::Characteristic &localAdresse, + characteristic::Characteristic &localEndurance, + characteristic::Characteristic &localChance, + const std::function &operation) noexcept; }; } diff --git a/src/billy_objects.cpp b/src/billy_objects.cpp index b4488e8..395600f 100644 --- a/src/billy_objects.cpp +++ b/src/billy_objects.cpp @@ -17,10 +17,16 @@ constexpr std::uint32_t toolsHash = const_hash("tools"); namespace character { using characteristic::Characteristic; - bool BillyObjects::push_object(const billyObject &object, CharacterSheet &sheet) noexcept { + + bool BillyObjects::insert_object(CharacterSheet &sheet, const billyEnums objType) noexcept { if (sheet.objects.size() < 3) { - sheet.objects.emplace_back(object); - sheet.available_objects.erase(object); + sheet.objects.emplace(objType, std::visit(overloaded{ + [](const weapons &arg) { return std::unique_ptr(std::make_unique(arg)); }, + [](const equipments &arg) { return std::unique_ptr(std::make_unique(arg)); }, + [](const tools &arg) { return std::unique_ptr(std::make_unique(arg)); }, + }, objType)); + const auto &object = sheet.objects[objType]; + sheet.available_objects.erase(objType); auto &local_habilete = static_cast(sheet.habilete); auto &local_adresse = static_cast(sheet.adresse); @@ -28,42 +34,21 @@ namespace character { auto &local_chance = static_cast(sheet.chance); std::visit(overloaded{ - [&](const weapons &arg) { - ++sheet.nb_weapons; - change_carac_weapon(arg, sheet, local_habilete, local_adresse, local_endurance, plus); - }, - [&](const equipments &arg) { - ++sheet.nb_equipments; - change_carac_equipment(arg, - sheet, - local_habilete, - local_adresse, - local_endurance, - local_chance, - plus, - minus); - }, - [&](const tools &arg) { - ++sheet.nb_tools; - change_carac_tools(arg, - sheet, - local_habilete, - local_adresse, - local_endurance, - local_chance, - plus); - }, - }, object); + [&](const weapons &arg) { ++sheet.nb_weapons; }, + [&](const equipments &arg) { ++sheet.nb_equipments; }, + [&](const tools &arg) { ++sheet.nb_tools; }, + }, objType); + change_carac(object, sheet, local_habilete, local_adresse, local_endurance, local_chance, plus); return true; } return false; } - void BillyObjects::pop_object(CharacterSheet &sheet) noexcept { + void BillyObjects::erase_object(CharacterSheet &sheet, const billyEnums objToErase) noexcept { if (!sheet.objects.empty()) { - const billyObject obj = sheet.objects.back(); - sheet.objects.pop_back(); - sheet.available_objects.insert(obj); + const billyObjects obj{ sheet.objects[objToErase].release() }; + sheet.objects.erase(objToErase); + sheet.available_objects.insert(objToErase); auto &local_habilete = static_cast(sheet.habilete); auto &local_adresse = static_cast(sheet.adresse); @@ -71,171 +56,49 @@ namespace character { auto &local_chance = static_cast(sheet.chance); std::visit(overloaded{ - [&](const weapons &arg) { - --sheet.nb_weapons; - change_carac_weapon(arg, sheet, local_habilete, local_adresse, local_endurance, minus); - }, - [&](const equipments &arg) { - --sheet.nb_equipments; - change_carac_equipment(arg, - sheet, - local_habilete, - local_adresse, - local_endurance, - local_chance, - minus, - plus); - }, - [&](const tools &arg) { - --sheet.nb_tools; - change_carac_tools(arg, - sheet, - local_habilete, - local_adresse, - local_endurance, - local_chance, - minus); - } - }, obj); + [&](const weapons &arg) { --sheet.nb_weapons; }, + [&](const equipments &arg) { --sheet.nb_equipments; }, + [&](const tools &arg) { --sheet.nb_tools; } + }, objToErase); + change_carac(obj, sheet, local_habilete, local_adresse, local_endurance, local_chance, minus); } } - void BillyObjects::change_carac_tools(const tools &arg, + void BillyObjects::change_carac(const billyObjects &arg, CharacterSheet &sheet, Characteristic &localHabilete, Characteristic &localAdresse, Characteristic &localEndurance, Characteristic &localChance, const std::function &operation) noexcept { - switch (arg) { - case tools::Fourche: - localHabilete.materiel = operation(localHabilete.materiel, 1); - localEndurance.materiel = operation(localEndurance.materiel, 3); - break; - case tools::Dagger: - sheet.critique = operation(sheet.critique, 6); - break; - case tools::RockClimbingKit: - localAdresse.materiel = operation(localAdresse.materiel, 1); - break; - case tools::SackOfGrain: - localEndurance.materiel = operation(localEndurance.materiel, 2); - localChance.materiel = operation(localChance.materiel, 2); - break; - } - } - - void BillyObjects::change_carac_equipment(const equipments &arg, - CharacterSheet &sheet, - Characteristic &localHabilete, - Characteristic &localAdresse, - Characteristic &localEndurance, - Characteristic &localChance, - const std::function &primary, - const std::function &complement) noexcept { - switch (arg) { - case equipments::Chainmail: - localHabilete.materiel = complement(localHabilete.materiel, 1); - localAdresse.materiel = complement(localAdresse.materiel, 1); - localEndurance.materiel = primary(localEndurance.materiel, 1); - sheet.armor = primary(sheet.armor, 2); - break; - case equipments::CookingPot: - localEndurance.materiel = primary(localEndurance.materiel, 2); - sheet.armor = primary(sheet.armor, 1); - break; - case equipments::PamphletTourist: - localChance.materiel = primary(localChance.materiel, 4); - break; - case equipments::MedicKit: - localChance.materiel = primary(localChance.materiel, 1); - break; - } - } - - void BillyObjects::change_carac_weapon(const weapons &arg, - CharacterSheet &sheet, - Characteristic &localHabilete, - Characteristic &localAdresse, - Characteristic &localEndurance, - const std::function &operation) noexcept { - switch (arg) { - case weapons::Sword: - localHabilete.materiel = operation(localHabilete.materiel, 4); - break; - case weapons::Lance: - localHabilete.materiel = operation(localHabilete.materiel, 3); - localAdresse.materiel = operation(localAdresse.materiel, 1); - break; - case weapons::Morgenstern: - localHabilete.materiel = operation(localHabilete.materiel, 1); - localEndurance.materiel = operation(localEndurance.materiel, 1); - sheet.damage = operation(sheet.damage, 1); - break; - case weapons::Bow: - localHabilete.materiel = operation(localHabilete.materiel, 3); - localAdresse.materiel = operation(localAdresse.materiel, 1); - break; - } + localHabilete.materiel = operation(localHabilete.materiel, arg->add_materiel(localHabilete.type)); + localAdresse.materiel = operation(localAdresse.materiel, arg->add_materiel(localAdresse.type)); + localEndurance.materiel = operation(localEndurance.materiel, arg->add_materiel(localEndurance.type)); + localChance.materiel = operation(localChance.materiel, arg->add_materiel(localChance.type)); + sheet.armor = operation(sheet.armor, arg->add_armor()); + sheet.damage = operation(sheet.armor, arg->add_damage()); + sheet.critique = operation(sheet.critique, arg->add_critique()); } void BillyObjects::check_dagger_conditions(const CharacterSheet &sheet, Characteristic &localHabilete, const std::function &operation) { - int count_weapons = 0; - bool is_there_bow = false; - std::for_each(sheet.objects.cbegin(), sheet.objects.cend(), [&](const billyObject &object) -> void { - if (const weapons *p = std::get_if(std::addressof(object)); p != nullptr) { - ++count_weapons; - if (*p == weapons::Bow) { - is_there_bow = true; - } + if (const auto it_dagger = sheet.objects.find(tools::Dagger); it_dagger != sheet.objects.cend()) { + const std::size_t count_weapons = std::count_if(sheet.objects.cbegin(), + sheet.objects.cend(), + [](container::const_reference node) { + return std::get_if(&node.first) != nullptr; + }); + const bool is_there_bow = sheet.objects.find(weapons::Bow) != sheet.objects.cend(); + if (count_weapons > 1 || is_there_bow) { + localHabilete.materiel = operation(localHabilete.materiel, + -it_dagger->second->add_materiel(localHabilete.type)); } - }); - if (count_weapons < 2 && !is_there_bow) { - localHabilete.materiel = operation(localHabilete.materiel, 1); } } - std::string_view BillyObjects::billy_object_to_string(const billyObject &object) noexcept { - return std::visit(overloaded{ - [](const weapons &arg) { - switch (arg) { - case weapons::Sword: - return sword; - case weapons::Lance: - return lance; - case weapons::Morgenstern: - return morgenstern; - case weapons::Bow: - return bow; - } - }, - [](const equipments &arg) { - switch (arg) { - case equipments::Chainmail: - return chainmail; - case equipments::CookingPot: - return cooking_pot; - case equipments::MedicKit: - return medic_kit; - case equipments::PamphletTourist: - return pamphlet_tourist; - } - }, - [](const tools &arg) { - switch (arg) { - case tools::Fourche: - return fourche; - case tools::Dagger: - return dagger; - case tools::RockClimbingKit: - return rock_climbing_kit; - case tools::SackOfGrain: - return sack_of_grain; - } - } - }, object); + std::string_view BillyObjects::billy_object_to_string(const billyObjects &object) noexcept { + return object->to_string(); } void BillyObjects::from_json(const json &j, BillyObjects::container &billy) { @@ -244,20 +107,21 @@ namespace character { const std::uint8_t value = element[1].get(); switch (key) { case weaponsHash: - billy.push_back(static_cast(value)); + billy.emplace(static_cast(value), std::make_unique(static_cast(value))); break; case equipmentHash: - billy.push_back(static_cast(value)); + billy.emplace(static_cast(value), + std::make_unique(static_cast(value))); break; case toolsHash: - billy.push_back(static_cast(value)); + billy.emplace(static_cast(value), std::make_unique(static_cast(value))); break; } } } void BillyObjects::to_json(json &j, const BillyObjects::container &billy) { - for (const auto &object: billy) { + for (const auto &[object_type, _]: billy) { std::visit(overloaded{ [&j](const weapons weapon) { j.emplace_back(std::pair{ weaponsHash, static_cast(weapon) }); @@ -268,27 +132,7 @@ namespace character { [&j](const tools tool) { j.emplace_back(std::pair{ toolsHash, static_cast(tool) }); } - }, object); + }, object_type); } } - - ankerl::svector BillyObjects::check_conformity(const CharacterSheet &sheet) noexcept { - ankerl::svector output; - std::transform(sheet.get_objects().cbegin(), - sheet.get_objects().cend(), std::back_inserter(output), - [&sheet](const billyObject &object) -> bool { - return std::visit(overloaded{ - [](const weapons weapon) { return false; }, - [](const equipments equipment) { return false; }, - [](const tools tool) { return false; }, - }, object); - }); - const int total = std::accumulate(sheet.get_objects().cbegin(), - sheet.get_objects().cend(), - 0, - [](const int a, const billyObject &object) -> int { - return 0; - }); - return output; - } } \ No newline at end of file