Mirrored_Repos/veloren
1
0
Fork 0
mirror of https://gitlab.com/veloren/veloren.git synced 2024-08-30 18:12:32 +00:00
Code Issues Packages Projects Releases Wiki Activity

Adress review

Browse Source 
* Make compatible_handndess check its own function and get back to
  `match`
This commit is contained in:
juliancoffee 2022-09-06 00:59:15 +03:00
parent 54a1adea31
commit 2b3cfe690d
1 changed files with 16 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
common/src/comp/inventory/item/modular.rs
View File

@ -389,6 +389,16 @@ pub enum ModularWeaponCreationError {
SecondaryComponentNotFound, SecondaryComponentNotFound,
} }
/// Check if hand restrictions are compatible.
///
/// If at least on of them is omitted, check is passed.
fn compatible_handndess(a: Option<Hands>, b: Option<Hands>) -> bool {
match (a, b) {
(Some(a), Some(b)) => a == b,
_ => true,
}
}
/// Generate all primary components for specific tool and material. /// Generate all primary components for specific tool and material.
/// ///
/// Read [random_weapon_primary_component] for more. /// Read [random_weapon_primary_component] for more.
@ -406,9 +416,7 @@ pub fn generate_weapon_primary_components(
.get(&(tool, material_id.to_owned())) .get(&(tool, material_id.to_owned()))
.into_iter() .into_iter()
.flatten() .flatten()
.filter(|(_comp, hand)| { .filter(|(_comp, hand)| compatible_handndess(hand_restriction, *hand))
hand_restriction == *hand || hand_restriction.is_none() || hand.is_none()
})
.map(|(c, h)| (c.duplicate(ability_map, msm), hand_restriction.or(*h))) .map(|(c, h)| (c.duplicate(ability_map, msm), hand_restriction.or(*h)))
.collect()) .collect())
} else { } else {
@ -442,9 +450,7 @@ pub fn random_weapon_primary_component(
.get(&(tool, material_id.to_owned())) .get(&(tool, material_id.to_owned()))
.into_iter() .into_iter()
.flatten() .flatten()
.filter(|(_comp, hand)| { .filter(|(_comp, hand)| compatible_handndess(hand_restriction, *hand))
hand_restriction == *hand || hand_restriction.is_none() || hand.is_none()
})
.collect::<Vec<_>>(); .collect::<Vec<_>>();
let (comp, hand) = primary_components let (comp, hand) = primary_components
@ -484,16 +490,10 @@ pub fn generate_weapons(
.get(&tool) .get(&tool)
.into_iter() .into_iter()
.flatten() .flatten()
.filter(|(_def, hand)| { .filter(|(_def, hand)| compatible_handndess(hand_restriction, *hand))
hand_restriction == *hand || hand_restriction.is_none() || hand.is_none() .filter(|(_def, hand)| compatible_handndess(comp_hand, *hand));
});
for (def, hand) in secondaries {
if comp_hand.is_some() && hand.is_some() && comp_hand != *hand {
// if handedness of components incompatible, skip
continue;
}
for (def, _hand) in secondaries {
let secondary = Item::new_from_item_base( let secondary = Item::new_from_item_base(
ItemBase::Simple(Arc::clone(def)), ItemBase::Simple(Arc::clone(def)),
Vec::new(), Vec::new(),
@ -532,9 +532,7 @@ pub fn random_weapon(
.get(&tool) .get(&tool)
.into_iter() .into_iter()
.flatten() .flatten()
.filter(|(_def, hand)| { .filter(|(_def, hand)| compatible_handndess(hand_restriction, *hand))
hand_restriction == *hand || hand_restriction.is_none() || hand.is_none()
})
.collect::<Vec<_>>(); .collect::<Vec<_>>();
let secondary_component = { let secondary_component = {