[Bots] Move BotGetSpellsByType to cache (#4655)

This commit is contained in:
nytmyr 2025-02-07 13:05:36 -06:00 committed by GitHub
parent e0d95b4302
commit ed7023f336
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
4 changed files with 29 additions and 28 deletions

View File

@ -12631,12 +12631,20 @@ void Bot::CleanBotBlockedBuffs()
} }
} }
std::vector<BotSpells_wIndex> Bot::BotGetSpellsByType(uint16 spell_type) { const std::vector<BotSpells_wIndex>& Bot::BotGetSpellsByType(uint16 spell_type) const {
if (AIBot_spells_by_type[spell_type].empty()) { auto it = AIBot_spells_by_type.find(spell_type);
spell_type = GetParentSpellType(spell_type);
if (it == AIBot_spells_by_type.end() || it->second.empty()) {
it = AIBot_spells_by_type.find(GetParentSpellType(spell_type));
if (it == AIBot_spells_by_type.end()) {
static const std::vector<BotSpells_wIndex> empty;
return empty;
}
} }
return AIBot_spells_by_type[spell_type]; return it->second;
} }
void Bot::AssignBotSpellsToTypes(std::vector<BotSpells>& AIBot_spells, std::unordered_map<uint16, std::vector<BotSpells_wIndex>>& AIBot_spells_by_type) { void Bot::AssignBotSpellsToTypes(std::vector<BotSpells>& AIBot_spells, std::unordered_map<uint16, std::vector<BotSpells_wIndex>>& AIBot_spells_by_type) {

View File

@ -305,7 +305,7 @@ public:
uint16 BotGetSpells(int spellslot) { return AIBot_spells[spellslot].spellid; } uint16 BotGetSpells(int spellslot) { return AIBot_spells[spellslot].spellid; }
uint32 BotGetSpellType(int spellslot) { return AIBot_spells[spellslot].type; } uint32 BotGetSpellType(int spellslot) { return AIBot_spells[spellslot].type; }
uint16 BotGetSpellPriority(int spellslot) { return AIBot_spells[spellslot].priority; } uint16 BotGetSpellPriority(int spellslot) { return AIBot_spells[spellslot].priority; }
std::vector<BotSpells_wIndex> BotGetSpellsByType(uint16 spell_type); const std::vector<BotSpells_wIndex>& BotGetSpellsByType(uint16 spell_type) const;
float GetProcChances(float ProcBonus, uint16 hand) override; float GetProcChances(float ProcBonus, uint16 hand) override;
int GetHandToHandDamage(void) override; int GetHandToHandDamage(void) override;
bool TryFinishingBlow(Mob *defender, int64 &damage) override; bool TryFinishingBlow(Mob *defender, int64 &damage) override;
@ -669,7 +669,7 @@ public:
void MapSpellTypeLevels(); void MapSpellTypeLevels();
const std::map<int32_t, std::map<int32_t, BotSpellTypesByClass>>& GetCommandedSpellTypesMinLevels() { return commanded_spells_min_level; } const std::map<int32_t, std::map<int32_t, BotSpellTypesByClass>>& GetCommandedSpellTypesMinLevels() { return commanded_spells_min_level; }
std::list<BotSpellTypeOrder> GetSpellTypesPrioritized(uint8 priority_type); std::list<BotSpellTypeOrder> GetSpellTypesPrioritized(uint8 priority_type);
uint16 GetParentSpellType(uint16 spell_type); static uint16 GetParentSpellType(uint16 spell_type);
bool IsValidSpellTypeBySpellID(uint16 spell_type, uint16 spell_id); bool IsValidSpellTypeBySpellID(uint16 spell_type, uint16 spell_id);
inline uint16 GetCastedSpellType() const { return _castedSpellType; } inline uint16 GetCastedSpellType() const { return _castedSpellType; }
void SetCastedSpellType(uint16 spell_type); void SetCastedSpellType(uint16 spell_type);

View File

@ -134,14 +134,7 @@ void bot_command_discipline(Client* c, const Seperator* sep)
} }
if (spell_id == UINT16_MAX) { // Aggressive/Defensive type if (spell_id == UINT16_MAX) { // Aggressive/Defensive type
std::vector<BotSpells_wIndex> bot_spell_list; const std::vector<BotSpells_wIndex>& bot_spell_list = bot_iter->BotGetSpellsByType(aggressive ? BotSpellTypes::DiscAggressive : BotSpellTypes::DiscDefensive);
if (aggressive) {
bot_spell_list = bot_iter->BotGetSpellsByType(BotSpellTypes::DiscAggressive);
}
else if (defensive) {
bot_spell_list = bot_iter->BotGetSpellsByType(BotSpellTypes::DiscDefensive);
}
for (int i = bot_spell_list.size() - 1; i >= 0; i--) { for (int i = bot_spell_list.size() - 1; i >= 0; i--) {
if (!IsValidSpell(bot_spell_list[i].spellid)) { if (!IsValidSpell(bot_spell_list[i].spellid)) {

View File

@ -912,7 +912,7 @@ std::list<BotSpell> Bot::GetBotSpellsForSpellEffect(Bot* caster, uint16 spell_ty
} }
if (caster->AI_HasSpells()) { if (caster->AI_HasSpells()) {
std::vector<BotSpells_wIndex> bot_spell_list = caster->BotGetSpellsByType(spell_type); const std::vector<BotSpells_wIndex>& bot_spell_list = caster->BotGetSpellsByType(spell_type);
for (int i = bot_spell_list.size() - 1; i >= 0; i--) { for (int i = bot_spell_list.size() - 1; i >= 0; i--) {
if (!IsValidSpellAndLoS(bot_spell_list[i].spellid, caster->HasLoS())) { if (!IsValidSpellAndLoS(bot_spell_list[i].spellid, caster->HasLoS())) {
@ -921,7 +921,7 @@ std::list<BotSpell> Bot::GetBotSpellsForSpellEffect(Bot* caster, uint16 spell_ty
if ( if (
caster->CheckSpellRecastTimer(bot_spell_list[i].spellid) && caster->CheckSpellRecastTimer(bot_spell_list[i].spellid) &&
(bot_spell_list[i].type == spell_type || bot_spell_list[i].type == caster->GetParentSpellType(spell_type)) && (bot_spell_list[i].type == spell_type || bot_spell_list[i].type == GetParentSpellType(spell_type)) &&
caster->IsValidSpellTypeBySpellID(spell_type, bot_spell_list[i].spellid) && caster->IsValidSpellTypeBySpellID(spell_type, bot_spell_list[i].spellid) &&
(IsEffectInSpell(bot_spell_list[i].spellid, spell_effect) || GetSpellTriggerSpellID(bot_spell_list[i].spellid, spell_effect)) (IsEffectInSpell(bot_spell_list[i].spellid, spell_effect) || GetSpellTriggerSpellID(bot_spell_list[i].spellid, spell_effect))
) { ) {
@ -950,7 +950,7 @@ std::list<BotSpell> Bot::GetBotSpellsForSpellEffectAndTargetType(Bot* caster, ui
} }
if (caster->AI_HasSpells()) { if (caster->AI_HasSpells()) {
std::vector<BotSpells_wIndex> bot_spell_list = caster->BotGetSpellsByType(spell_type); const std::vector<BotSpells_wIndex>& bot_spell_list = caster->BotGetSpellsByType(spell_type);
for (int i = bot_spell_list.size() - 1; i >= 0; i--) { for (int i = bot_spell_list.size() - 1; i >= 0; i--) {
if (!IsValidSpellAndLoS(bot_spell_list[i].spellid, caster->HasLoS())) { if (!IsValidSpellAndLoS(bot_spell_list[i].spellid, caster->HasLoS())) {
@ -959,7 +959,7 @@ std::list<BotSpell> Bot::GetBotSpellsForSpellEffectAndTargetType(Bot* caster, ui
if ( if (
caster->CheckSpellRecastTimer(bot_spell_list[i].spellid) && caster->CheckSpellRecastTimer(bot_spell_list[i].spellid) &&
(bot_spell_list[i].type == spell_type || bot_spell_list[i].type == caster->GetParentSpellType(spell_type)) && (bot_spell_list[i].type == spell_type || bot_spell_list[i].type == GetParentSpellType(spell_type)) &&
caster->IsValidSpellTypeBySpellID(spell_type, bot_spell_list[i].spellid) && caster->IsValidSpellTypeBySpellID(spell_type, bot_spell_list[i].spellid) &&
( (
IsEffectInSpell(bot_spell_list[i].spellid, spell_effect) || IsEffectInSpell(bot_spell_list[i].spellid, spell_effect) ||
@ -991,7 +991,7 @@ std::list<BotSpell> Bot::GetBotSpellsBySpellType(Bot* caster, uint16 spell_type)
} }
if (caster->AI_HasSpells()) { if (caster->AI_HasSpells()) {
std::vector<BotSpells_wIndex> bot_spell_list = caster->BotGetSpellsByType(spell_type); const std::vector<BotSpells_wIndex>& bot_spell_list = caster->BotGetSpellsByType(spell_type);
for (int i = bot_spell_list.size() - 1; i >= 0; i--) { for (int i = bot_spell_list.size() - 1; i >= 0; i--) {
if (!IsValidSpellAndLoS(bot_spell_list[i].spellid, caster->HasLoS())) { if (!IsValidSpellAndLoS(bot_spell_list[i].spellid, caster->HasLoS())) {
@ -1000,7 +1000,7 @@ std::list<BotSpell> Bot::GetBotSpellsBySpellType(Bot* caster, uint16 spell_type)
if ( if (
caster->CheckSpellRecastTimer(bot_spell_list[i].spellid) && caster->CheckSpellRecastTimer(bot_spell_list[i].spellid) &&
(bot_spell_list[i].type == spell_type || bot_spell_list[i].type == caster->GetParentSpellType(spell_type)) && (bot_spell_list[i].type == spell_type || bot_spell_list[i].type == GetParentSpellType(spell_type)) &&
caster->IsValidSpellTypeBySpellID(spell_type, bot_spell_list[i].spellid) caster->IsValidSpellTypeBySpellID(spell_type, bot_spell_list[i].spellid)
) { ) {
BotSpell bot_spell; BotSpell bot_spell;
@ -1020,7 +1020,7 @@ std::vector<BotSpell_wPriority> Bot::GetPrioritizedBotSpellsBySpellType(Bot* cas
std::vector<BotSpell_wPriority> result; std::vector<BotSpell_wPriority> result;
if (caster && caster->AI_HasSpells()) { if (caster && caster->AI_HasSpells()) {
std::vector<BotSpells_wIndex> bot_spell_list = caster->BotGetSpellsByType(spell_type); const std::vector<BotSpells_wIndex>& bot_spell_list = caster->BotGetSpellsByType(spell_type);
for (int i = bot_spell_list.size() - 1; i >= 0; i--) { for (int i = bot_spell_list.size() - 1; i >= 0; i--) {
if (!IsValidSpellAndLoS(bot_spell_list[i].spellid, caster->HasLoS())) { if (!IsValidSpellAndLoS(bot_spell_list[i].spellid, caster->HasLoS())) {
@ -1035,7 +1035,7 @@ std::vector<BotSpell_wPriority> Bot::GetPrioritizedBotSpellsBySpellType(Bot* cas
if ( if (
caster->CheckSpellRecastTimer(bot_spell_list[i].spellid) && caster->CheckSpellRecastTimer(bot_spell_list[i].spellid) &&
(bot_spell_list[i].type == spell_type || bot_spell_list[i].type == caster->GetParentSpellType(spell_type)) && (bot_spell_list[i].type == spell_type || bot_spell_list[i].type == GetParentSpellType(spell_type)) &&
caster->IsValidSpellTypeBySpellID(spell_type, bot_spell_list[i].spellid) caster->IsValidSpellTypeBySpellID(spell_type, bot_spell_list[i].spellid)
) { ) {
if ( if (
@ -1109,7 +1109,7 @@ BotSpell Bot::GetFirstBotSpellBySpellType(Bot* caster, uint16 spell_type) {
result.ManaCost = 0; result.ManaCost = 0;
if (caster && caster->AI_HasSpells()) { if (caster && caster->AI_HasSpells()) {
std::vector<BotSpells_wIndex> bot_spell_list = caster->BotGetSpellsByType(spell_type); const std::vector<BotSpells_wIndex>& bot_spell_list = caster->BotGetSpellsByType(spell_type);
for (int i = bot_spell_list.size() - 1; i >= 0; i--) { for (int i = bot_spell_list.size() - 1; i >= 0; i--) {
if (!IsValidSpellAndLoS(bot_spell_list[i].spellid, caster->HasLoS())) { if (!IsValidSpellAndLoS(bot_spell_list[i].spellid, caster->HasLoS())) {
@ -1118,7 +1118,7 @@ BotSpell Bot::GetFirstBotSpellBySpellType(Bot* caster, uint16 spell_type) {
if ( if (
caster->CheckSpellRecastTimer(bot_spell_list[i].spellid) && caster->CheckSpellRecastTimer(bot_spell_list[i].spellid) &&
(bot_spell_list[i].type == spell_type || bot_spell_list[i].type == caster->GetParentSpellType(spell_type)) && (bot_spell_list[i].type == spell_type || bot_spell_list[i].type == GetParentSpellType(spell_type)) &&
caster->IsValidSpellTypeBySpellID(spell_type, bot_spell_list[i].spellid) caster->IsValidSpellTypeBySpellID(spell_type, bot_spell_list[i].spellid)
) { ) {
result.SpellId = bot_spell_list[i].spellid; result.SpellId = bot_spell_list[i].spellid;
@ -1217,14 +1217,14 @@ BotSpell Bot::GetBestBotSpellForPercentageHeal(Bot* caster, Mob* tar, uint16 spe
result.ManaCost = 0; result.ManaCost = 0;
if (caster && caster->AI_HasSpells()) { if (caster && caster->AI_HasSpells()) {
std::vector<BotSpells_wIndex> bot_spell_list = caster->BotGetSpellsByType(spell_type); const std::vector<BotSpells_wIndex>& bot_spell_list = caster->BotGetSpellsByType(spell_type);
for (int i = bot_spell_list.size() - 1; i >= 0; i--) { for (int i = bot_spell_list.size() - 1; i >= 0; i--) {
if (!IsValidSpell(bot_spell_list[i].spellid)) { if (!IsValidSpell(bot_spell_list[i].spellid)) {
continue; continue;
} }
if ( if (
(bot_spell_list[i].type == spell_type || bot_spell_list[i].type == caster->GetParentSpellType(spell_type)) && (bot_spell_list[i].type == spell_type || bot_spell_list[i].type == GetParentSpellType(spell_type)) &&
caster->IsValidSpellTypeBySpellID(spell_type, bot_spell_list[i].spellid) && caster->IsValidSpellTypeBySpellID(spell_type, bot_spell_list[i].spellid) &&
IsCompleteHealSpell(bot_spell_list[i].spellid) && IsCompleteHealSpell(bot_spell_list[i].spellid) &&
caster->CastChecks(bot_spell_list[i].spellid, tar, spell_type) caster->CastChecks(bot_spell_list[i].spellid, tar, spell_type)
@ -1925,7 +1925,7 @@ BotSpell Bot::GetDebuffBotSpell(Bot* caster, Mob *tar, uint16 spell_type) {
return result; return result;
if (caster->AI_HasSpells()) { if (caster->AI_HasSpells()) {
std::vector<BotSpells_wIndex> bot_spell_list = caster->BotGetSpellsByType(spell_type); const std::vector<BotSpells_wIndex>& bot_spell_list = caster->BotGetSpellsByType(spell_type);
for (int i = bot_spell_list.size() - 1; i >= 0; i--) { for (int i = bot_spell_list.size() - 1; i >= 0; i--) {
if (!IsValidSpellAndLoS(bot_spell_list[i].spellid, caster->HasLoS())) { if (!IsValidSpellAndLoS(bot_spell_list[i].spellid, caster->HasLoS())) {
@ -1971,7 +1971,7 @@ BotSpell Bot::GetBestBotSpellForResistDebuff(Bot* caster, Mob *tar, uint16 spell
bool needs_disease_resist_debuff = (tar->GetDR() + level_mod) > 100; bool needs_disease_resist_debuff = (tar->GetDR() + level_mod) > 100;
if (caster->AI_HasSpells()) { if (caster->AI_HasSpells()) {
std::vector<BotSpells_wIndex> bot_spell_list = caster->BotGetSpellsByType(spell_type); const std::vector<BotSpells_wIndex>& bot_spell_list = caster->BotGetSpellsByType(spell_type);
for (int i = bot_spell_list.size() - 1; i >= 0; i--) { for (int i = bot_spell_list.size() - 1; i >= 0; i--) {
if (!IsValidSpellAndLoS(bot_spell_list[i].spellid, caster->HasLoS())) { if (!IsValidSpellAndLoS(bot_spell_list[i].spellid, caster->HasLoS())) {