Улучшение ИИ

Так, палатка может быть последней в списке монстров. И вылет из-за того, что выходим за границы массива.

Цитата:

if (!CurMon) return EXEC_DEFAULT;

Вот здесь, если указатель на монстра равен 0, то выходим из цикла (0x421DE9), а не EXEC_DEFAULT.

Так, за границы массива мы не выйдем, там по 20 отрядов на каждую сторону, но мусор получить можем, если палатка действительно последний стек.

Так тоже вылет:

Код: Выделить всё

   if (!CurMon){
      c->return_address = 0x421DE9;
      return NO_EXEC_DEFAULT;
   }

После вылета нажмите "Не отправлять" и посмотрите крэш-лог. Очень бы помог адрес инструкции, на которой происходит вылет.

as239 писал(а):

Спасибо! То что нужно.

Код: Выделить всё

H3CreatureInformation *creature = &P_Creatures[Level*2+Grade+14*Town->type];

Level(0-6), Grade(0,1)

Suggestion to avoid extra arithmetic and risk of errors:

Код: Выделить всё

H3TownCreatureTypes& townCreatures = P_TownCreatureTypes[Town->type];
...
H3CreatureInformation* creature = &P_Creatures[townCreatures.base[0~6]];
H3CreatureInformation* creature = &P_Creatures[townCreatures.upgrade[0~6]];


Also has handy begin() and end() iterator that hase both Base() and Upgraded() directly...

AlexSpl писал(а):

После вылета нажмите "Не отправлять" и посмотрите крэш-лог. Очень бы помог адрес инструкции, на которой происходит вылет.

Всё ясно. Делим на 0 хитпоинтов, а это значит пропускаем в цикл несуществующего монстра. Нужно как-то понять, что мы имеем дело с последним монстром, и в этом случае не продолжать цикл, а выходить из него.

Короче, в c->edi попадает мусор после последнего стека, который рассматривается, как структура H3CombatMonster.

Предлагаю проверять не по указателю, а по индексу существа (сравнивать индекс текущего отряда с общим кол-вом отрядов героя):

Код: Выделить всё

_int32_ stacks_count[2];//+0x54BC  // 0 - attacker, 1 - defender

У _BattleStack_ есть поле:

Код: Выделить всё

_dword_ index_on_side;    // +248 F8 dd = номер стэка у стороны на поле боя

Вот его и сравнивать.

AlexSpl писал(а):

Всё ясно. Делим на 0 хитпоинтов, а это значит пропускаем в цикл несуществующего монстра. Нужно как-то понять, что мы имеем дело с последним монстром, и в этом случае не продолжать цикл, а выходить из него.

Короче, в c->edi попадает мусор после последнего стека, который рассматривается, как структура H3CombatMonster.

Да действительно похоже так и есть.
Сделал вот так, тупо считая, что палатка всегда последний стек:

Код: Выделить всё

   if (ItsWarMachine(CurMon->type))
   {
      c->return_address = 0x421DE9;
      return NO_EXEC_DEFAULT;

Вылета нет и поведение ИИ стало гораздо лучше.
Было:

 

Стало:

 

Почти как человек сыграл.

Цитата:

Сделал вот так, тупо считая, что палатка всегда последний стек:

Кстати, да. Где-то читал, что боевые машины в самом конце идут, а поэтому код верный (как только встречаем боевую машину, прерываем цикл, т.к. если и есть какие-то стеки после неё, то они тоже боевые машины).

Хотя... Как насчёт клонов и призванных элементалей? Так что проверку всё же нужно делать.