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

function SpellBlizzardCallback takes nothing returns nothing
	set Timer = GetExpiredTimer()
	set TimerId = GetHandleId(Timer)
    
	set Caster = LoadUnitHandle(HT, TimerId, 'cstr')

	if GetUnitCurrentOrder(Caster) != 0xd0079 then // blizzard
		call PauseTimer(Timer)
		call DestroyTimer(Timer)
		call FlushChildHashtable(HT, TimerId)
       return
	endif 

	set CasterX = LoadReal(HT, TimerId, 'cstX')
	set CasterY = LoadReal(HT, TimerId, 'cstY')

	call GroupEnumUnitsInRange(Group, CasterX, CasterY, SpellBlizzardRange, null)
	call ForGroup(Group, function SpellBlizzardEnum) 
	call GroupClear(Group)    
endfunction 

ну дак как сделать это нормально ?

не понял сарказм, а этот код шутка?

ну или например ты ту же глобалку с таймером используешь где-то ещё, и этот код может затриггерится при уроне или смерти.

то можно таймер и кастера потерять после строки нанесения урона.

Плохая практика использовать комбинированый стиль именования.

вполне себе строчный ключ

попробуй напади!

Они падают скучно, одной пачкой. Сделал бы чтоб они вразнобой летели, как например здесь.

усложняем задачу?)

так а про итерацию, вынести к комментам в начале триггера?

group Group = CreateGroup() //Создаём группу, которую будем использовать для моментальной выборки юнитов

//if GetUnitCurrentOrder( Caster ) != 0xd0079 then - этот вариант работает быстрее, так как использует ид приказа без лишней возни со строками

я согласен теперь что не нужно его юзать.

Лишний раз срабатывает i = i + 1

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