Handling of complex depends parents, some cleanup

This commit is contained in:
Ivanowicz
2023-12-08 18:53:56 +01:00
parent debeb1bb5d
commit b551fc1ca4
2 changed files with 97 additions and 75 deletions

View File

@ -1,13 +1,21 @@
#include "..\script_component.hpp" #include "..\script_component.hpp"
/* /*
* Author: commy2, kymckay, LinkIsGrim * Author: commy2, kymckay, LinkIsGrim
* Get list of vehicle hitpoints to ignore * Get list of vehicle hitpoints to ignore and depends hitpoints
* *
* Arguments: * Arguments:
* 0: Vehicle <OBJECT> * 0: Vehicle <OBJECT>
* *
* Return Value: * Return Value:
* HitPoints to ignore <ARRAY> * 0: HitPoint indexes to ignore <ARRAY>
* 1: Depends hitpoints with parents <HASHMAP>
* <KEY>: Depends Index <NUMBER>
* <VALUE>: Parent Index <NUMBER>
* 2: Depends hitpoints with complex/multiple parents <HASHMAP>
* <KEY>: Depends Index <NUMBER>
* <VALUE>: Parent <ARRAY>
* 0: Parent Hitpoint <STRING>
* 1: Individual, valid parent hitpoints <ARRAY>
* *
* Example: * Example:
* [vehicle] call ace_repair_fnc_getSelectionsToIgnore * [vehicle] call ace_repair_fnc_getSelectionsToIgnore
@ -26,7 +34,7 @@ if (_type in _initializedClasses) exitWith {
}; //you return different amount of values each time }; //you return different amount of values each time
private _vehCfg = configOf _vehicle; private _vehCfg = configOf _vehicle;
private _hitpointGroups = getArray (_vehCfg >> QGVAR(hitpointGroups)); private _hitPointGroups = getArray (_vehCfg >> QGVAR(hitpointGroups));
private _turretPaths = ((fullCrew [_vehicle, "gunner", true]) + (fullCrew [_vehicle, "commander", true])) apply {_x # 3}; private _turretPaths = ((fullCrew [_vehicle, "gunner", true]) + (fullCrew [_vehicle, "commander", true])) apply {_x # 3};
(getAllHitPointsDamage _vehicle) params [["_hitPoints", []], ["_hitSelections", []]]; (getAllHitPointsDamage _vehicle) params [["_hitPoints", []], ["_hitSelections", []]];
@ -34,47 +42,39 @@ private _turretPaths = ((fullCrew [_vehicle, "gunner", true]) + (fullCrew [_vehi
([_vehicle] call FUNC(getWheelHitPointsWithSelections)) params ["_wheelHitPoints", "_wheelHitSelections"]; ([_vehicle] call FUNC(getWheelHitPointsWithSelections)) params ["_wheelHitPoints", "_wheelHitSelections"];
private _indexesToIgnore = []; private _indexesToIgnore = [];
private _dependsIndexMap = createHashMap;
private _processedSelections = []; private _processedSelections = [];
private _dependsIndexMap = createHashMap;
private _complexDependsMap = createHashMap;
{ {
private _selection = _x; private _selection = _x;
private _hitpoint = toLower (_hitPoints select _forEachIndex); private _hitPoint = toLower (_hitPoints select _forEachIndex);
private _isWheelOrTrack = _selection in _wheelHitSelections || {_hitpoint in _wheelHitPoints} || {_hitpoint in TRACK_HITPOINTS}; private _isWheelOrTrack = _selection in _wheelHitSelections || {_hitPoint in _wheelHitPoints} || {_hitPoint in TRACK_HITPOINTS};
if (_hitpoint isEqualTo "") then { // skip empty hitpoint if (_hitPoint isEqualTo "") then { // skip empty hitpoint
_indexesToIgnore pushBack _forEachIndex; _indexesToIgnore pushBack _forEachIndex;
continue continue
}; };
if (_isWheelOrTrack && {_selection in _processedSelections || {_selection isEqualTo ""}}) then { // skip duplicate or empty selection wheel/track if (_isWheelOrTrack && {_selection in _processedSelections || {_selection isEqualTo ""}}) then {
TRACE_3("Skipping duplicate Wheel/Track or empty selection",_hitpoint,_forEachIndex,_selection); TRACE_3("Skipping duplicate Wheel/Track or empty selection",_hitPoint,_forEachIndex,_selection);
/*#ifdef DEBUG_MODE_FULL
systemChat format ["Skipping duplicate wheel, hitpoint %1, index %2, selection %3", _hitpoint, _forEachIndex, _selection];
#endif*/
_indexesToIgnore pushBack _forEachIndex; _indexesToIgnore pushBack _forEachIndex;
_processedSelections pushBack _selection; _processedSelections pushBack _selection;
continue continue
}; };
if ("glass" in _hitpoint) then { // skip glass if ("glass" in _hitPoint) then {
TRACE_3("Skipping glass",_hitpoint,_forEachIndex,_selection); TRACE_3("Skipping glass",_hitPoint,_forEachIndex,_selection);
/*#ifdef DEBUG_MODE_FULL
systemChat format ["Skipping glass, hitpoint %1, index %2, selection %3", _hitpoint, _forEachIndex, _selection];
#endif*/
_indexesToIgnore pushBack _forEachIndex; _indexesToIgnore pushBack _forEachIndex;
_processedSelections pushBack _selection; _processedSelections pushBack _selection;
continue continue
}; };
if (_hitpoint select [0,7] isEqualTo "hitera_" || {_hitpoint select [0,4] isEqualTo "era_"} if (_hitPoint select [0,7] isEqualTo "hitera_" || {_hitPoint select [0,4] isEqualTo "era_"}
|| {_hitpoint select [0,8] isEqualTo "hitslat_"} || {_hitpoint select [0,5] isEqualTo "slat_"} || {_hitPoint select [0,8] isEqualTo "hitslat_"} || {_hitPoint select [0,5] isEqualTo "slat_"}
|| {_hitpoint select [0,9] isEqualTo "sideskirt"} || {_hitpoint select [0,6] isEqualTo "armor_"} || {_hitPoint select [0,9] isEqualTo "sideskirt"} || {_hitPoint select [0,6] isEqualTo "armor_"}
|| {_hitpoint select [0,3] isEqualTo "mud"}) then { // skip era/slat/sideskirt/armor/mudguards || {_hitPoint select [0,3] isEqualTo "mud"} || {_hitPoint select [0,9] isEqualTo "smoketube"}) then {
TRACE_3("Skipping ERA/Slat/Sideskirt/Armor/Mudguard HitPoint",_hitpoint,_forEachIndex,_selection); TRACE_3("Skipping ERA/Slat/Sideskirt/Armor/Mudguard/Smoketube HitPoint",_hitPoint,_forEachIndex,_selection);
/*#ifdef DEBUG_MODE_FULL
systemChat format ["Skipping ERA/SLAT, hitpoint %1, index %2, selection %3", _hitpoint, _forEachIndex, _selection];
#endif*/
_indexesToIgnore pushBack _forEachIndex; _indexesToIgnore pushBack _forEachIndex;
_processedSelections pushBack _selection; _processedSelections pushBack _selection;
continue continue
@ -83,75 +83,88 @@ private _processedSelections = [];
private _armorComponent = ""; private _armorComponent = "";
if (_selection == "") then { // some hitpoints have empty selection but defined armor component (mostly RHS) if (_selection == "") then { // some hitpoints have empty selection but defined armor component (mostly RHS)
{ {
private _turretHitpointCfg = ([_vehCfg, _x] call CBA_fnc_getTurret) >> "HitPoints"; private _turretHitPointCfg = ([_vehCfg, _x] call CBA_fnc_getTurret) >> "HitPoints";
private _hitpointsCfg = "configName _x == _hitpoint" configClasses _turretHitpointCfg; private _hitPointsCfg = "configName _x == _hitPoint" configClasses _turretHitPointCfg;
if (_hitpointsCfg isNotEqualTo []) exitWith { if (_hitPointsCfg isNotEqualTo []) exitWith {
TRACE_2("turret hitpoint configFound",_hitpoint,_x); TRACE_2("turret hitpoint config found",_hitPoint,_x);
_hitpointsCfg = _hitpointsCfg # 0; _hitPointsCfg = _hitPointsCfg # 0;
// only do turret hitpoints and stuff linked to visuals // only do turret hitpoints and stuff linked to visuals
if ( if (
(_hitpoint in ["hitturret", "hitgun"]) || (_hitPoint in ["hitturret", "hitgun"]) ||
{(getNumber (_hitpointsCfg >> "isGun")) == 1} || {(getNumber (_hitPointsCfg >> "isGun")) == 1} ||
{(getNumber (_hitpointsCfg >> "isTurret")) == 1} || {(getNumber (_hitPointsCfg >> "isTurret")) == 1} ||
{(getText (_hitpointsCfg >> "visual")) != ""} {(getText (_hitPointsCfg >> "visual")) != ""}
) then { ) then {
_armorComponent = getText (_hitpointsCfg >> "armorComponent"); _armorComponent = getText (_hitPointsCfg >> "armorComponent");
}; };
}; };
} forEach _turretPaths; } forEach _turretPaths;
if (_armorComponent == "") then { if (_armorComponent == "") then {
private _hitpointsCfg = "configName _x == _hitpoint" configClasses (_vehCfg >> "HitPoints"); private _hitPointsCfg = "configName _x == _hitPoint" configClasses (_vehCfg >> "HitPoints");
if (_hitpointsCfg isNotEqualTo []) then { if (_hitPointsCfg isNotEqualTo []) then {
_hitpointsCfg = _hitpointsCfg # 0; _hitPointsCfg = _hitPointsCfg # 0;
if ( if (
(getNumber (_hitpointsCfg >> "isGun")) == 1 || (getNumber (_hitPointsCfg >> "isGun")) == 1 ||
{(getNumber (_hitpointsCfg >> "isTurret")) == 1} || {(getNumber (_hitPointsCfg >> "isTurret")) == 1} ||
{(getText (_hitpointsCfg >> "visual")) != ""} {(getText (_hitPointsCfg >> "visual")) != ""}
) then { ) then {
_armorComponent = getText (_hitpointsCfg >> "armorComponent"); _armorComponent = getText (_hitPointsCfg >> "armorComponent");
}; };
}; };
}; };
}; };
if ((_selection == "") && {_armorComponent == ""}) then { if ((_selection == "") && {_armorComponent == ""}) then {
TRACE_3("Skipping no selection OR armor component",_hitpoint,_forEachIndex,_selection); TRACE_3("Skipping no selection OR armor component",_hitPoint,_forEachIndex,_selection);
/*#ifdef DEBUG_MODE_FULL
systemChat format ["Skipping no selection OR armor component, hitpoint %1, index %2, selection %3", _hitpoint, _forEachIndex, _selection];
#endif*/
_indexesToIgnore pushBack _forEachIndex; _indexesToIgnore pushBack _forEachIndex;
_processedSelections pushBack _selection; _processedSelections pushBack _selection;
continue continue
}; };
if (!(getText (_vehCfg >> "HitPoints" >> _hitpoint >> "depends") in ["", "0"])) then { // skip depends hitpoints, should be normalized by engine if (!(getText (_vehCfg >> "HitPoints" >> _hitPoint >> "depends") in ["", "0"])) then {
TRACE_3("Skipping depends hitpoint",_hitpoint,_forEachIndex,_selection); // Caches depends hitpoints and their parents
/*#ifdef DEBUG_MODE_FULL private _parentHitPoint = getText (_vehCfg >> "HitPoints" >> _hitPoint >> "depends");
systemChat format ["Skipping depends hitpoint, hitpoint %1, index %2, selection %3", _hitpoint, _forEachIndex, _selection]; _parentHitPoint = toLower _parentHitPoint;
#endif*/ private _parentHitPointIndex = _hitPoints findIf {_x == _parentHitPoint};
private _parentHitpoint = getText (_vehCfg >> "HitPoints" >> _hitpoint >> "depends");
_parentHitpoint = toLower _parentHitpoint;
private _parentHitpointIndex = _hitPoints findIf {_x == _parentHitpoint};
if (_parentHitpointIndex != -1) then { if (_parentHitPointIndex != -1) then {
_dependsIndexMap set [_forEachIndex, _parentHitpointIndex]; _dependsIndexMap set [_forEachIndex, _parentHitPointIndex];
} else { // multiple parents or incorrect parent TRACE_3("Depends hitpoint and parent index",_hitPoint,_forEachIndex,_parentHitPoint);
_indexesToIgnore pushBack _forEachIndex; } else {
private _groupIndex = _hitpointGroups findIf {_x # 0 == _hitpoint}; // Multiple parents or broken parents
if (_groupIndex != -1) then { _indexesToIgnore pushBack _forEachIndex;
ERROR_2("[%1] hitpoint [%2] is both a group-parent and an ignored depends and will be unrepairable",_type,_hitpoint); private _parentHitPoints = _parentHitPoint splitString "+*() ";
ERROR_1("group: %1",_hitpointGroups # _groupIndex); _parentHitPoints deleteAt (_parentHitPoints findIf {!(_x regexMatch "[a-z]")});
private _validComplexHitPoint = true;
{
private _currentHitPoint = _x;
private _hitPointsCfg = "configName _x == _currentHitPoint" configClasses (_vehCfg >> "HitPoints");
if (_hitPointsCfg isEqualTo []) then {
_validComplexHitPoint = false;
break;
};
} forEach _parentHitPoints;
if (_validComplexHitPoint) then {
TRACE_3("Skipping depends and setting complex parent",_hitPoint,_forEachIndex,_parentHitPoint);
_complexDependsMap set [_forEachIndex, [_parentHitPoint, _parentHitPoints]];
_dependsIndexMap set [_forEachIndex, _parentHitPointIndex];
} else {
TRACE_4("Skipping depends with broken complex parent",_hitPoint,_forEachIndex,_parentHitPoint,_parentHitPoints);
private _groupIndex = _hitPointGroups findIf {_x # 0 == _hitPoint};
if (_groupIndex != -1) then {
ERROR_2("[%1] hitpoint [%2] is both a group-parent and an ignored depends and will be unrepairable",_type,_hitPoint);
ERROR_1("group: %1",_hitPointGroups # _groupIndex);
};
}; };
}; };
_processedSelections pushBack _selection; _processedSelections pushBack _selection;
continue continue
}; };
if (ANY_OF(_hitpointGroups, ANY_OF(_x select 1, _x == _hitpoint))) then { // skip child hitpoints if (ANY_OF(_hitPointGroups, ANY_OF(_x select 1, _x == _hitPoint))) then {
TRACE_3("Skipping child hitpoint",_hitpoint,_forEachIndex,_selection); TRACE_3("Skipping child hitpoint",_hitPoint,_forEachIndex,_selection);
/*#ifdef DEBUG_MODE_FULL
systemChat format ["Skipping child hitpoint, hitpoint %1, index %2, selection %3", _hitpoint, _forEachIndex, _selection];
#endif*/
_indexesToIgnore pushBack _forEachIndex; _indexesToIgnore pushBack _forEachIndex;
_processedSelections pushBack _selection; _processedSelections pushBack _selection;
continue continue
@ -160,7 +173,7 @@ private _processedSelections = [];
_processedSelections pushBack _selection; _processedSelections pushBack _selection;
} forEach _hitSelections; } forEach _hitSelections;
_initializedClasses set [_type, [_indexesToIgnore, _dependsIndexMap]]; _initializedClasses set [_type, [_indexesToIgnore, _dependsIndexMap, _complexDependsMap]];
SETMVAR(GVAR(ignoredAndDependsInitializedClasses),_initializedClasses); SETMVAR(GVAR(ignoredAndDependsInitializedClasses),_initializedClasses);
[_indexesToIgnore, _dependsIndexMap] [_indexesToIgnore, _dependsIndexMap, _complexDependsMap]

View File

@ -1,7 +1,7 @@
#include "..\script_component.hpp" #include "..\script_component.hpp"
/* /*
* Author: commy2 * Author: commy2
* Used to normalize dependant hitpoints. May overwrite some global variables that are named like hitpoints or "Total" though... * Used to normalize depends hitpoints - sets their damage to damage of parent or parents.
* *
* Arguments: * Arguments:
* 0: Local Vehicle <OBJECT> * 0: Local Vehicle <OBJECT>
@ -16,16 +16,25 @@
*/ */
params ["_vehicle"]; params ["_vehicle"];
TRACE_2("params",_vehicle, typeOf _vehicle); TRACE_2("params",_vehicle,typeOf _vehicle);
// Can't execute all commands if the vehicle isn't local, exit if that's so // Can't execute all commands if the vehicle isn't local, exit if that's so
if !(local _vehicle) exitWith {ERROR_1("Vehicle Not Local %1", _vehicle);}; if !(local _vehicle) exitWith {ERROR_1("Vehicle Not Local %1", _vehicle);};
([_vehicle] call FUNC(getSelectionsToIgnore)) params ["", "_dependsIndexMap"]; ([_vehicle] call FUNC(getSelectionsToIgnore)) params ["", "_dependsIndexMap", "_complexDependsMap"];
// apply normalized damage to all depends hitpoints { // apply normalized damage to all depends hitpoints
{
private _damage = _vehicle getHitIndex _y; private _damage = _vehicle getHitIndex _y;
if (_y == -1) then { // handle complex depends parents
(_complexDependsMap get _x) params ["_parentHitPointScript", "_parentHitPointArray"];
{
private _hitPoint = _parentHitPointArray select _forEachIndex;
private _hitPointDamage = str (_vehicle getHitPointDamage _hitPoint);
_parentHitPointScript = [_parentHitPointScript, _hitPoint, _hitPointDamage] call CBA_fnc_replace;
} forEach _parentHitPointArray;
TRACE_1("complex script",_parentHitPointScript);
_damage = call compile _parentHitPointScript;
};
TRACE_2("setting depends hitpoint",_x,_damage); TRACE_2("setting depends hitpoint",_x,_damage);
_vehicle setHitIndex [_x, _damage]; _vehicle setHitIndex [_x, _damage];
} forEach _dependsIndexMap; } forEach _dependsIndexMap;