[Cleanup] Fix SEResist array settings duplicate code (#3225)

# Notes
- These conditions did the same thing, combining them is simpler.
This commit is contained in:
Alex King 2023-04-03 16:53:48 -04:00 committed by GitHub
parent 5dd849ac75
commit 26fd52fb06
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -1505,12 +1505,14 @@ void Mob::ApplyAABonuses(const AA::Rank &rank, StatBonuses *newbon)
case SE_SpellEffectResistChance: {
for (int e = 0; e < MAX_RESISTABLE_EFFECTS * 2; e += 2) {
if (newbon->SEResist[e + 1] && (newbon->SEResist[e] == limit_value) &&
(newbon->SEResist[e + 1] < base_value)) {
newbon->SEResist[e] = limit_value; // Spell Effect ID
newbon->SEResist[e + 1] = base_value; // Resist Chance
break;
} else if (!newbon->SEResist[e + 1]) {
if (
!newbon->SEResist[e + 1] ||
(
newbon->SEResist[e + 1] &&
newbon->SEResist[e] == limit_value &&
newbon->SEResist[e + 1] < base_value
)
) {
newbon->SEResist[e] = limit_value; // Spell Effect ID
newbon->SEResist[e + 1] = base_value; // Resist Chance
break;
@ -3492,12 +3494,14 @@ void Mob::ApplySpellsBonuses(uint16 spell_id, uint8 casterlevel, StatBonuses *ne
{
for(int e = 0; e < MAX_RESISTABLE_EFFECTS*2; e+=2)
{
if(new_bonus->SEResist[e+1] && (new_bonus->SEResist[e] == limit_value) && (new_bonus->SEResist[e+1] < effect_value)){
new_bonus->SEResist[e] = limit_value; //Spell Effect ID
new_bonus->SEResist[e+1] = effect_value; //Resist Chance
break;
}
else if (!new_bonus->SEResist[e+1]){
if (
!new_bonus->SEResist[e + 1] ||
(
new_bonus->SEResist[e + 1] &&
new_bonus->SEResist[e] == limit_value &&
new_bonus->SEResist[e + 1] < effect_value
)
) {
new_bonus->SEResist[e] = limit_value; //Spell Effect ID
new_bonus->SEResist[e + 1] = effect_value; //Resist Chance
break;