Наработка персонажа Ассасин (ReNew)

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

У иконок все еще нету DISBTN и... рамок. Все иконки выглядят так, словно они пассивные. Скиллы перепутаны местами. У цепи есть полоски хп. Я не кодер, но этого уже достаточно.

Еще новая пассивка, какая-то неюзабельная. Старая была оригинальной и интересной, а это просто спам. Это всего-лишь мое мнение, а не наставление. Все-таки я не кодер. С этим разберется другой чувак.

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

Gnusik,твой спел был не муи,я сделал его муи
когда ты используешь полярные координаты то ты создаёшь точку и не удаляешь её,если использовать их очень часто то их станет очень много и будут лаги
PauseUnit

кат

Скопировать в буфер

struct sword_sp
unit caster
unit target
endstruct

function TimerStartEx takes timer whichTimer, real period, code handlerFunc, integer userData returns nothing
    call TimerStart(whichTimer, I2R(userData), false, null) // timer exploit, xgm 2007
    call PauseTimer(whichTimer)
    call TimerStart(whichTimer, period,false, handlerFunc ) // start timer
endfunction

function TimerGetUserData takes timer whichTimer returns integer
    return R2I(TimerGetRemaining(whichTimer )+0.5) // get user data from timer
endfunction

function Sword_Attack_Action takes nothing returns nothing
   local timer a=GetExpiredTimer()
   local sword_sp b=TimerGetUserData(a)
   local unit caster=b.caster
   local unit target=b.target
   local real facing=GetUnitFacing(target)*bj_DEGTORAD
   local real sin=Sin(facing)
   local real cos=Cos(facing)
   local real x1=GetUnitX(target)
   local real y1=GetUnitY(target)
   local real x2=GetUnitX(caster)
   local real y2=GetUnitY(caster)
   local real x3=x1+GetRandomReal(-80,80)
   local real y3=y1+GetRandomReal(-80,80)
    call SetUnitX(caster,x3)
    call SetUnitY(caster,y3)
    call SetUnitFacing(caster,bj_RADTODEG * Atan2(y1-y3,x1-x3))
    call UnitDamageTarget(caster,target,5,true,false,ATTACK_TYPE_NORMAL,DAMAGE_TYPE_NORMAL,WEAPON_TYPE_WHOKNOWS)
    call SetUnitAnimation(caster, "attack" )
    call SetUnitAnimation(target, "death" )
   set caster=null 
   set target=null
    call TimerStartEx(a,.06,function Sword_Attack_Action,b)
endfunction

function Trig_Sword_Attack_Conditions takes nothing returns boolean
    return GetSpellAbilityId() == 'A000'
endfunction

function Trig_Sword_Attack_Actions takes nothing returns nothing
   local unit caster=GetSpellAbilityUnit()
   local unit target=GetSpellTargetUnit()
   local real x1
   local real y1
   local real x2
   local real y2
   local real dx
   local real dy
   local real cos
   local real sin
   local real facing
   local real x3
   local real y3
   local timer a=CreateTimer()
   local sword_sp b=sword_sp.create()
    call PauseUnit(caster,true)
    call PauseUnit(target,true)
    call SetUnitInvulnerable(caster, true )
    call SetUnitPathing(caster, false )
    set b.caster=caster
    set b.target=target
    call TimerStartEx(a,.06,function Sword_Attack_Action,b)
    call TriggerSleepAction( 2.50 )
    call PauseTimer(a)
    call DestroyTimer(a)
    set b.caster=null
    set b.target=null
    call b.destroy()
    set facing=GetUnitFacing(target)*bj_DEGTORAD
    set sin=Sin(facing)
    set cos=Cos(facing)
    set x1=GetUnitX(target)
    set y1=GetUnitY(target)
    set x2=GetUnitX(caster)
    set y2=GetUnitY(caster)
    set x3=x1+cos*165
    set y3=y1+sin*165
    set dx=x1+cos*256
    set dy=y1+sin*256
    call SetUnitX(caster,x3)
    call SetUnitY(caster,y3)
    call SetUnitFacing(caster,bj_RADTODEG * Atan2(dy-y3,dx-x3))
    call TriggerSleepAction( 1.20 )
    call UnitDamageTarget(caster,target,200,true,false,ATTACK_TYPE_NORMAL,DAMAGE_TYPE_NORMAL,WEAPON_TYPE_WHOKNOWS)
    call SetUnitAnimation(target, "death" )
    call TriggerSleepAction( 0.35 )
    call SetUnitPathing(caster, true )
    call SetUnitInvulnerable(caster, false )
    call PauseUnit(caster,false)
    call PauseUnit(target,false)
   set caster=null
   set target=null
endfunction

//===========================================================================
function InitTrig_Sword_Attack takes nothing returns nothing
    local integer index=0
    set gg_trg_Sword_Attack = CreateTrigger(  )
        loop
            call TriggerRegisterPlayerUnitEvent(gg_trg_Sword_Attack, Player(index),EVENT_PLAYER_UNIT_SPELL_EFFECT, null)
            set index = index + 1
            exitwhen index == 15
        endloop
    call TriggerAddCondition( gg_trg_Sword_Attack, Condition( function Trig_Sword_Attack_Conditions ) )
    call TriggerAddAction( gg_trg_Sword_Attack, function Trig_Sword_Attack_Actions )
endfunction

Gnusik, дай дами юнитам из цепи пассивную способность москиты,тогда у них не будет хп бара

это твой 1 скил
создаётся новая точка,а старая не удаляется,таким образом у тебя получается лишняя точка которая тебе не нужна
в том что я кинул нету хэша
то есть твой 1 скил ужасно не оптимален(остальные не смотрел мне и его хватило),а я скинул тебе его исправленную версию
глобалки там где должны быть локалки и прочее

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

Gnusik, напиши когда утечки уберёшь

struct sword_sp
unit caster
unit target
endstruct
это только в джнгп работает
а чем тебе джнгп не нравится?
точки не юзай
юзай координаты

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

Скопировать в буфер

function Trig_Sword_Attack_Actions takes nothing returns nothing
    set udg_caster = GetSpellAbilityUnit()
    set udg_cell_cast = GetSpellTargetUnit()
    call PauseUnit( udg_caster, true )
    call PauseUnit( udg_cell_cast, true )
    call SetUnitTimeScalePercent( udg_caster, 160.00 )
    call SetUnitTimeScalePercent( udg_cell_cast, 160.00 )
    call EnableTrigger( gg_trg_Sword_Attack_2 )
    call TriggerSleepAction( 2.00 )
    call DisableTrigger( gg_trg_Sword_Attack_2 )
    call PauseUnit( udg_caster, false )
    call PauseUnit( udg_cell_cast, false )
    call SetUnitPathing( udg_caster, true )
    call SetUnitPathing( udg_cell_cast, true )
endfunction

Скопировать в буфер

    call SetUnitTimeScalePercent( udg_caster, 160.00 )
    call SetUnitTimeScalePercent( udg_cell_cast, 160.00 )

бж это
локалки до сих пор не вижу