[JASS] need help

[Cr34p]

I wrote this little spell... and it don't want to work

what did i wrong, because i can't see anything wrong

struct holystruct
  unit target
  real life
  integer counter
  integer countertwo = 0
    method onDestroy takes nothing returns nothing
        set this.target = null
        set this.countertwo = 0
    endmethod
endstruct

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

function holytimer takes nothing returns nothing
  local timer t = GetExpiredTimer()
  local holystruct Dat = GetHandleInt(t,"struct")
  local effect sfx
  local real life
    set sfx = AddSpecialEffectTarget("Abilities\\Spells\\Human\\HolyBolt\\HolyBoltSpecialArt.mdl",Dat.target,"origin")
    set life = GetUnitState(Dat.target,UNIT_STATE_LIFE)
    call SetUnitState(Dat.target,UNIT_STATE_LIFE,life+Dat.life)
    call BJDebugMsg("Heal")    
    call DestroyEffect(sfx)
    set Dat.countertwo = Dat.countertwo + 1
    if (Dat.countertwo >= Dat.counter) then
        call DestroyTimer(t)
        call holystruct.destroy(Dat)
    endif
    set t = null    
endfunction

function hlActions takes nothing returns nothing
  local holystruct Dat = holystruct.create()
  local timer t = CreateTimer()
  local unit u = GetTriggerUnit()
  local unit tar = GetSpellTargetUnit()
  local integer counter = (4+(GetUnitAbilityLevelSwapped('A000',u)*1))  
    set Dat.target = tar
    set Dat.counter = counter
    set Dat.life = (GetHeroInt(u,true))    
    call TimerStart(t,1.00,true,function holytimer)
    call SetHandleInt(t,"struct",Dat)
    set u = null
    set tar = null
endfunction

function InitTrig_HolyLight takes nothing returns nothing
  local trigger HL = CreateTrigger()
    call TriggerRegisterAnyUnitEventBJ(HL,EVENT_PLAYER_UNIT_SPELL_EFFECT)
    call TriggerAddCondition(HL,Condition(function hlCondition))
    call TriggerAddAction( HL, function hlActions )
  set HL = null  
endfunction

 

[spiwn]

Lol much.
Works perfectly(I think) here.
Though I can suggest using:
GetUnitAbilityLevel(u,'A000')

Instead of:
GetUnitAbilityLevelSwapped('A000',u)

And you didn't null t in hlActions
P.s. Though you can improve the code a bit by replacing stuff like:

local effect sfx
  local real life
    set sfx = AddSpecialEffectTarget("Abilities\\Spells\\Human\\HolyBolt\\HolyBoltSpecialArt.mdl",Dat.target,"origin")
set life = GetUnitState(Dat.target,UNIT_STATE_LIFE)

with

local effect sfx = AddSpecialEffectTarget("Abilities\\Spells\\Human\\HolyBolt\\HolyBoltSpecialArt.mdl",Dat.target,"origin")
  local real life = GetUnitState(Dat.target,UNIT_STATE_LIFE)

 

[Cr34p]

yes i see, but don't think that this is the prob^^
but anyway thanks for tip xD

 

[spiwn]

But, there is no problem...
If you think there is, please tell us what it is... cause my paranormal guessing skills are all used up for now.

 

[Cr34p]

hm, so it should be a spell which heals the target for int of caster each 1 second, but it doesnt work , and idk why :<

 

[spiwn]

FFS.
What is not working...
I.e. Unit is not healed; No effect appears.
And for the last time already: It is working...

 

[Cr34p]

^^so now i can say why it didn't worked... so the trigger was named "Holy Light" but the init function was InitTrig_HolyLight.... so i had to rename the trigger into "HolyLight" ... ffs...
stupid stupid stupid

 

[spiwn]

Next, say what the problem is(along with the desired result). I.e. Syntax error.