Pcornat/BillySheet
1
0
Fork 0
Code Issues Pull requests Releases Activity

Compare commits

base: Pcornat:8a5b2dd927c69241519459a6863239598a49932f
Branches Tags
Pcornat:main
...
compare: Pcornat:f9f973b51ebf4f90e6fc13ba1b5854df62f88242
Branches Tags
Pcornat:main

2 commits
8a5b2dd927 ... f9f973b51e

Author SHA1 Message Date
Pcornat
f9f973b51e
 		Inline variable 2026-01-25 21:33:53 +01:00
Pcornat
4f694d0bbe
 		No exception when trying to erase an object not in sheet 2026-01-25 21:33:46 +01:00
3 changed files with 22 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

12
Unit testing/billy_objects_tests.cpp
View file

@ -153,6 +153,18 @@ TEST_CASE("[D] Pushing popping BillyObjects", "[pushpop][0]") {
}
}
TEST_CASE("[D] Double erase no throw", "[pushpop][1]") {
BillyObjects gestionnaire{};
CharacterSheet sheet;
REQUIRE(sheet.get_objects().empty());
REQUIRE_NOTHROW(gestionnaire.insert_object(sheet, weapons::Sword));
REQUIRE(gestionnaire.insert_object(sheet, weapons::Sword));
REQUIRE_NOTHROW(gestionnaire.erase_object(sheet, weapons::Bow));
REQUIRE_FALSE(gestionnaire.erase_object(sheet, weapons::Bow));
}
TEST_CASE("[D] Printing Billy's objects", "[printing]") {
for (const auto &object: BillyObjects::all_objects) {
const std::unique_ptr<GenericObject> obj = test::get_obj(object);

2
include/billy_objects.hpp
View file

@ -52,7 +52,7 @@ namespace character {
[[nodiscard]] bool insert_object(CharacterSheet &sheet, const billyEnums objType) noexcept;
void erase_object(CharacterSheet &sheet, const billyEnums objToErase) noexcept;
bool erase_object(CharacterSheet &sheet, const billyEnums objToErase) noexcept;
[[nodiscard]] const std::plus<std::uint32_t> &get_plus_operation() const { return plus; }

17
src/billy_objects.cpp
View file

@ -57,9 +57,9 @@ namespace character {
return false;
}
void BillyObjects::erase_object(CharacterSheet &sheet, const billyEnums objToErase) noexcept {
if (!sheet.objects.empty()) {
const billyObjects obj{ sheet.objects[objToErase].release() };
bool BillyObjects::erase_object(CharacterSheet &sheet, const billyEnums objToErase) noexcept {
if (const auto it = sheet.objects.find(objToErase); it != std::end(sheet.objects)) {
const billyObjects obj{ it->second.release() };
sheet.objects.erase(objToErase);
sheet.available_objects.insert(objToErase);
@ -69,13 +69,15 @@ namespace character {
auto &local_chance = static_cast<Characteristic &>(sheet.chance);
std::visit(overloaded{
[&](const weapons &arg) { --sheet.nb_weapons; },
[&](const equipments &arg) { --sheet.nb_equipments; },
[&](const tools &arg) { --sheet.nb_tools; }
[&](const weapons &) { --sheet.nb_weapons; },
[&](const equipments &) { --sheet.nb_equipments; },
[&](const tools &) { --sheet.nb_tools; }
},
objToErase);
change_carac(obj, sheet, local_habilete, local_adresse, local_endurance, local_chance, minus);
return true;
}
return false;
}
void BillyObjects::change_carac(const billyObjects &arg,
@ -103,8 +105,7 @@ namespace character {
return std::get_if<weapons>(&node.first) != nullptr;
}
);
if (const bool is_there_bow = sheet.get_objects().find(weapons::Bow) != sheet.get_objects().cend();
count_weapons > 1 || is_there_bow) {
if (count_weapons > 1 || sheet.get_objects().find(weapons::Bow) != sheet.get_objects().cend()) {
sheet.habilete.materiel -= it_dagger->second->add_materiel(sheet.get_habilete().type);
}
}