Добавлено (26-12-2010, 20:37)
---------------------------------------------
Вот, с х и у. Что еще оптимизировать?

function Trig_cast_Timer takes nothing returns nothing
local timer ta = GetExpiredTimer()
local integer i = GetHandleId(ta)
local unit t = LoadUnitHandle(udg_Hash, i, 1)
local unit u = LoadUnitHandle(udg_Hash, i, 0)
local real x = LoadReal(udg_Hash, i, 2) используется 1 раз в коде - незачем подргружать в переменную
local real y = LoadReal(udg_Hash, i, 3) аналогично с предыдущим действием
call DestroyEffect(AddSpecialEffectTarget("Abilities\\Weapons\\Bolt\\BoltImpact.mdl", t, "chest"))
call UnitDamageTarget(u, t, 300., true, false, ATTACK_TYPE_NORMAL, DAMAGE_TYPE_NORMAL, WEAPON_TYPE_WHOKNOWS)
call SetUnitPosition(t,x,y)
call PauseTimer(ta) ненужное действие
call DestroyTimer(ta)
call FlushChildHashtable(udg_Hash, i)
set t=null
set u=null
А таймер и не обнулил - утечка
endfunction

function Trig_cast_Actions takes nothing returns nothing
local timer ta здесь должно быть создание таймера, смотри мой вариант.
local integer i устанавливай значение переменных при инициализации
if GetSpellAbilityId() == 'A000' then вообще это должно быть условием триггера а не кода. Ибо поток и память выделяются. собсна - бред
set ta = CreateTimer() ненужное действие
set i = GetHandleId(ta) ненужное действие
call SaveUnitHandle(udg_Hash, i, 0, GetTriggerUnit())
call SaveUnitHandle(udg_Hash, i, 1, GetSpellTargetUnit()) этот юнит должен быть впеременной
call SaveReal(udg_Hash, i, 2, GetWidgetX(GetSpellTargetUnit()))
call SaveReal(udg_Hash, i, 3, GetWidgetY(GetSpellTargetUnit()))
call TimerStart(ta, 5., true, function Trig_cast_Timer) true здесь точно не нужен
set ta = null при твоей конструкции - конечно можно, однако по нормальному при присвоении переменной в ините - обнуление переменной тоже должно быть в ините
endif
endfunction