WarCraft 3

Баг через 6000 сек

Добавлен pAxsIs

Тестирую свою карту (которая еще, конечно, не закончена), и вот иногда, чтобы проверить на утечность и другие траблы времени, оставляю ее на произвол и ухожу по делам. Но дважды, что я оставлял на ночь. Она переставала работать нормально ровно через 6000 сек (по таймеру времени карты). Второй раз я лично сам это наблюдал. То есть вначале юниты начинали бегать на месте, через 10 сек они телепортировались, а еще через 2-3 минуты экран в игре зависал, и даже не откликается. В диспетчере задач игра работает и отвечает, ресурсы не жрет чрезмерно. Утечек за все 6000 сек нет, но что не так, я не понимаю. Триггеров и таймеров или чего-то другого именно на 6000 сек - нет.

Есть идеи в чем проблема?

N1ghtSiren

Принятый ответ

Возможно какой-либо переодический таймер на +- 0.7 сек, заполняющий массивы.
Массив в 8192 закончился, и при подборе свободных ячеек каждый раз уходит в бесконечный луп, который выдаёт лаг в 3 сек и обрывается на оп лимите.
Единственная идея, ибо сам сталкивался с этим

Скачать - 2953 КБ 81

ОЖИДАНИЕ РЕКЛАМЫ...

Показан только небольшой набор комментариев вокруг указанного. Перейти к актуальным.

pAxsIs 5 лет назад

8gabriel8:

pAxsIs, ты говорил, что у тебя триггеры создаются, но может есть на них какой лимит. Не пробовал проверять их?

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

А вообще почему решил плодить триггеры, может саму систему переделать, чтобы использовать переменные по кругу вместо создания триггеров?

Вот пример, как ты представляешь сделать без временного триггера отлов момента удара юнита без создания временного триггера. В момент замаха юнита, на цель создается временный триггер с событием EVENT_UNIT_DAMAGED. Который отлавливает удар, чтобы нанести доп урон.
Альтернативный вариант:
Можно, конечно, написать уже заранее триггер без события и добавлять событие во время замаха. Тогда количество триггеров не изменится, но будут плодиться события, которые нельзя удалить.

Смотреть код

function StunnedSpellDmg_Actions takes unit attacker, unit target, boolean isStunned returns nothing
    if ( !isStunned ) then
        call UnitDamageTarget( attacker, target, ( ( I2R(GetUnitPointValueByType(GetUnitTypeId(attacker))) ) + 1 ), true, false, ATTACK_TYPE_CHAOS, DAMAGE_TYPE_POISON, WEAPON_TYPE_WHOKNOWS )
    else
        call UnitDamageTarget( attacker, target, ( ( 4.00 * I2R(GetUnitPointValueByType(GetUnitTypeId(attacker))) ) + 1 ), true, false, ATTACK_TYPE_CHAOS, DAMAGE_TYPE_POISON, WEAPON_TYPE_WHOKNOWS )
    endif
endfunction

function StunnedSpellDmg_Conditions takes nothing returns boolean
    local trigger trg      = GetTriggeringTrigger()
    local integer trgH     = GetHandleId( trg )
    local unit    attacker = LoadUnitHandle( udg_kiwHash, trgH, StringHash("attacker") )
    local unit    target   = LoadUnitHandle( udg_kiwHash, trgH, StringHash("target") )
    local boolean isBuffed = LoadBoolean( udg_kiwHash, trgH, StringHash("isStunned") )
    
    if GetTriggerEventId() == EVENT_UNIT_DAMAGED then
        if GetEventDamageSource() == attacker then
            call DisableTrigger( trg )
            call StunnedSpellDmg_Actions( attacker, target, isBuffed )
            
            call FlushChildHashtable( udg_kiwHash, trgH )
            call DestroyTriggerEx( trg )
        endif
    else
        call FlushChildHashtable( udg_kiwHash, trgH )
        call DestroyTriggerEx( trg )
    endif
    
    set trg      = null
    set attacker = null
    set target   = null
    
    return false
endfunction

function StunnedSpellDmg_EventReg takes nothing returns nothing
    local trigger trg      = CreateTrigger()
    local unit    target   = GetTriggerUnit()
    local unit    attacker = GetAttacker()
    local integer trgH     = GetHandleId( trg )
    local boolean isBuffed = false
    
    if ( GetUnitAbilityLevel( target, 'BPSE') > 0 ) then
        set isBuffed = true
    endif
    
    
    call TriggerRegisterUnitEvent( trg, target, EVENT_UNIT_DAMAGED )
    call TriggerRegisterTimerEvent( trg, 1, false )
    call TriggerAddCondition( trg, Condition( function StunnedSpellDmg_Conditions ) )
    call SaveUnitHandle( udg_kiwHash, trgH, StringHash("attacker"), attacker )
    call SaveUnitHandle( udg_kiwHash, trgH, StringHash("target"), target )
    call SaveBoolean( udg_kiwHash, trgH, StringHash("isStunned"), isBuffed )
    
    set target   = null
    set attacker = null
    set trg      = null
endfunction

function Trig_Stunned_Conditions takes nothing returns boolean
    return ( ( GetUnitTypeId(GetAttacker()) == udg_arrBTypes_Warriors[1] ) or \\
    ( GetUnitTypeId(GetAttacker()) == udg_arrBTypes_Warriors[2] ) or \\
    ( GetUnitTypeId(GetAttacker()) == udg_arrBTypes_Warriors[3] ) or \\
    ( GetUnitTypeId(GetAttacker()) == udg_arrBTypes_Warriors[4] ) or \\
    ( GetUnitTypeId(GetAttacker()) == udg_arrBTypes_Warriors[5] ) )
endfunction

function Trig_Stunned_Actions takes nothing returns nothing
    call StunnedSpellDmg_EventReg()
endfunction

//===========================================================================
function InitTrig_Stunned takes nothing returns nothing
    set gg_trg_Stunned = CreateTrigger(  )
    call TriggerRegisterAnyUnitEventBJ( gg_trg_Stunned, EVENT_PLAYER_UNIT_ATTACKED )
    call TriggerAddCondition( gg_trg_Stunned, Condition( function Trig_Stunned_Conditions ) )
    call TriggerAddAction( gg_trg_Stunned, function Trig_Stunned_Actions )
endfunction

NazarPunk:

А там насколько помню были баги в TriggerAddAction, из-за чего я досих пор в условиях действия пишу.

Поподробнее пожалуйста. Что за баги?

Clamp 5 лет назад

pAxsIs:

как ты представляешь сделать без временного триггера отлов момента удара юнита без создания временного триггера

https://xgm.guru/p/wc3/catch-dmg

Msey 5 лет назад

pAxsIs:

Можно, конечно, написать уже заранее триггер без события и добавлять событие во время замаха. Тогда количество триггеров не изменится, но будут плодиться события, которые нельзя удалить.

Самым попсовым вариантом было и является попадание юнита на карту и добавление ему события "получает урон"

pAxsIs 5 лет назад
Отредактирован pAxsIs

Msey:

Самым попсовым вариантом было и является попадание юнита на карту и добавление ему события "получает урон"

Пробовал этот вариант, но это в этом случае будет постоянный рост событий в триггере, как я уже писал. Неужели динамические триггеры могут быть проблемой? И вызывать подобный баг.

Clamp:

https://xgm.guru/p/wc3/catch-dmg

Разве у Msey не такие же трудозатраты по количеству событий в итоге? Ведь из комментариев под твоей наработкой:
ScorpioT1000:

это просто штука которая регает "юнит получил урон" на всех юнитов на карте.

nazarpunk 5 лет назад

pAxsIs:

Поподробнее пожалуйста. Что за баги?

В подробностях уже не помню, а гугл нашёл только xgm.guru/p/wc3/165628?postid=313095

для отлова урона можно использовать system-physical-damage-detection , там можно модифицировать урон.

8gabriel8 5 лет назад

pAxsIs, чтобы обнулить события, есть действие, которое приводит триггер в первоначальное состояние. Можно заносить юнитов в виде событий и в группу, триггер на смерть будет удалять из группы, потом, например, каждые 5 минут обновлять триггер до первоначального состояния и сразу добавлять туда события на членов группы.

Запамятовал упомянуть, что игра зависла, когда попытался выйти из неё после 6000-секундного бага.

pAxsIs 5 лет назад

Всех услышал, пойду тестить каждый кейс, отпишусь как найду в чем (хотя бы примерно) причина.

8gabriel8:

pAxsIs, чтобы обнулить события, есть действие, которое приводит триггер в первоначальное состояние. Можно заносить юнитов в виде событий и в группу, триггер на смерть будет удалять из группы, потом, например, каждые 5 минут обновлять триггер до первоначального состояния и сразу добавлять туда события на членов группы.

Запамятовал упомянуть, что игра зависла, когда попытался выйти из неё после 6000-секундного бага.

Спасибо за идею, попробую.

nazarpunk 5 лет назад

pAxsIs:

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

JASS автоматически не делает весь код хорошим, а популярность карты не залог оптимального решения.

Порылся я в вашем коде и это ужасно. Копирование триггера, чтобы сделать теже действия, но с другими параметрами повеселило.
Лучше переделать всё пока не позно, а то потом в этой каше ничего не разберёте.

pAxsIs 5 лет назад
Отредактирован pAxsIs

NazarPunk:

JASS автоматически не делает весь код хорошим, а популярность карты не залог оптимального решения.

Речь была о другом, не о популярности или о JASS, а о том, что там есть подобные конструкции почти везде и это не вызывает проблем, что следовательно не должно быть проблемой у меня.

Порылся я в вашем коде и это ужасно. Копирование триггера, чтобы сделать теже действия, но с другими параметрами повеселило.
Лучше переделать всё пока не позно, а то потом в этой каше ничего не разберёте.

Изначально писал все на триггерах, можно увидеть везде аналоги, еще до сих пор не переписал все на JASS, как раз недавно писал о том сабж, что перехожу с ГУИ на JASS. Конструкции, где используется копирование триггера сведены к минимуму.

Сейчас просмотрел беглым взглядом, не совмещены только Триггеры: SpawnLine* в один триггер и CompSpellBack* в один триггер, больше копированных конструкций нет. А про эти 2 момента знаю, но как уже писал еще не закончил переход с ГУИ на JASS. Если видите что-то еще "ужасное" - поделитесь, я приму во внимание.

Возможность передачи массива в функцию очень бы улучшило код, ведь для многих функций полиморфизм бы не помешал. (Знаю про vJass и cJass, но хочу работать только с нативными инструментами)

nazarpunk 5 лет назад
Отредактирован nazarpunk

OrderBackToLineB, OrderBackToLineR
ForceLineBlue1, ForceLineBlue2, ForceLineBlue3
ForceLineRed1, ForceLineRed2, ForceLineRed3
Line1, Line2, Line3

Триггеры Types * где в каждом событие Map Initalization что-бы задать несколько переменных.

Притом, что я карту даже сохранить не смог ибо JGNP ругается
local are only supported at the top function

pAxsIs:

но хочу работать только с нативными инструментами

если нативные инструменты замедляют разработку, то почему бы не использовать нормальные, где такую кашу можно написать проще

раскрыть

if ( ( GetUnitUserData(pickedUnit) / 10 ) == 1 ) then
            local location orderLoc = Location(GetRectCenterX(udg_arrRRegions_WayP1[modulus]), GetRectCenterY(udg_arrRRegions_WayP1[modulus]))
            call IssuePointOrderLoc( pickedUnit, "attack", orderLoc )
            call RemoveLocation( orderLoc )
        
        elseif ( ( GetUnitUserData(pickedUnit) / 10 ) == 2 ) then
            local location orderLoc = Location(GetRectCenterX(udg_arrRRegions_WayP2[modulus]), GetRectCenterY(udg_arrRRegions_WayP2[modulus]))
            call IssuePointOrderLoc( pickedUnit, "attack", orderLoc )
            call RemoveLocation( orderLoc )
            
        elseif ( ( GetUnitUserData(pickedUnit) / 10 ) == 3 ) then
            local location orderLoc = Location(GetRectCenterX(udg_arrRRegions_WayP3[modulus]), GetRectCenterY(udg_arrRRegions_WayP3[modulus]))
            call IssuePointOrderLoc( pickedUnit, "attack", orderLoc )
            call RemoveLocation( orderLoc )

// Гдето в при инициализации
real SpawnRectR[][];
real SpawnRectB[][];

SpawnRectB[0][0] = udg_rct_SpawnRectB_0_0;
SpawnRectB[0][1] = udg_rct_SpawnRectB_0_1;
SpawnRectR[1][0] = udg_rct_SpawnRectR_1_0;
SpawnRectR[1][1] = udg_rct_SpawnRectR_1_1;

// и.т.д
function SpawnRectGet(boolean isB, integer r1, integer r2){
 	if (isB) {
		return SpawnRectB[r1][r2];
	}
	return SpawnRectR[r1][r2];
}

function SpawnOrder(unit u, boolean isB, integer r1, integer r2) {
	rect rct = SpawnRectGet(isB, r1, r2);

	IssuePointOrder( pickedUnit, "attack", GetRectCenterX(rct), GetRectCenterY(rct) );
	rct = null;
}

// где-то в событиях

integer data = GetUnitUserData(pickedUnit) ;
integer modulus = data - (data / 10) * 10
integer data10 = data / 10;

if (data10 >= 1 || data10 <=3){
	SpawnOrder(pickedUnit, data10, modulus);
}

pAxsIs 5 лет назад

NazarPunk:

OrderBackToLineB, OrderBackToLineR
ForceLineBlue1, ForceLineBlue2, ForceLineBlue3
ForceLineRed1, ForceLineRed2, ForceLineRed3
Line1, Line2, Line3

Знал, что возможно об этом упомянешь,

но соль в том, что для

"OrderBackToLine*" желательно передавать структуру, чтобы он нормально все обрабатывал, что тут невозможно сделать.
"ForceLine*" есть 2 варианта развития, один это сделать поиск необходимого региона по которому он будет выбирать как реагировать на то или иное событие, либо сделать 6 триггеров, каждый из которых отдельно вызывается. Конечно, если бы я понимал, что у меня возможно будет какое-то изменение в будущем в плане количество спаунов, то лучше было бы сделать один триггер обрабатывающий все, тут согласен.
"Line*" однозначно нужно разделить на 3 части, поскольку нужно передавать массив, либо писать дикий костыль на определение массивов, что я не люблю делать. Если писать, то сразу основательно. Там логика может отличаться. Поэтому нужно сохранить 3 линии для удобства изменения условий для каждой линии.

По поводу двумерных массивов полностью согласен, что так можно решить проблему. Но я пока писал только на ГУИ, где таковых нет (за исключением хэш таблиц, которыми я начал пользоваться только недавно, когда начал использовать JASS). В глобальных переменных объявить двумерный массив нельзя. А если пытаться объявить глобальные в своем коде, то пишет ошибку, что глобальные должны быть объявлены в самом начале. Поэтому я пока обходился одномерными массивами, что вызвало подобное. Так что тут я с тобой полностью солидарен.
В идеале вообще создавать объект со своим набором полей и методов. Но я пока вначале пути JASS, так что пользуюсь только им. А вообще пишу на С++

Спасибо за рекомендации

Показан только небольшой набор комментариев вокруг указанного. Перейти к актуальным.

Чтобы оставить комментарий, пожалуйста, войдите на сайт.

Баг через 6000 сек

N1ghtSiren

Принятый ответ

Вход

Регистрация на сайте

Переместить ресурс 移动资源

Создание задания

Выбор проекта для репоста

Управление интересами