Spell Summon Treant

BRUTAL

BRUTAL

I'm working
Reaction score
118
Summon Treant

Requires GTrigger, Key Timers 2; all included in the map.

Spell Description: Summon the Treant of Evisceration from nature's inertly evil energy.

Screen Shots:
st1rv.jpg
st2ha.jpg

st3v.jpg
st4w.jpg

Code:
JASS:
// Summon Treant by BRUTAL.
// Requires GT, and KT2.
// Credits are welcome but not necessary.
// June 30th, 2010.

scope SummonTreant initializer init

globals
    private constant integer ID                 ='A000'  // ID of the spell.
    
    private constant integer DUMMY_BEAM         = 'h005'    // ID of the beam unit.
    private constant boolean BEAM_ON            = true      // Whether you want the beam effect on.
    private constant integer BEAM_RED           = 0         // Red value of the beam unit.
    private constant integer BEAM_GREEN         = 255       // Green value of the beam unit.
    private constant integer BEAM_BLUE          = 0         // Blue value of the beam unit.
    private constant integer BEAM_ALPHA         = 255       // Alpha value of the beam unit
    private constant real    BEAM_SCALE         = 1.        // Scaling size of the beam unit.
        
    private constant integer DUMMY_BIRD         = 'h001'    // ID of the bird unit.
    private constant boolean BIRD_ON            = true      // Whether you want the bird effect on.
    private constant integer BIRD_RED           = 255       // Red value of the bird unit.
    private constant integer BIRD_GREEN         = 255       // Green value of the bird unit.
    private constant integer BIRD_BLUE          = 255       // Blue value of the bird unit.
    private constant integer BIRD_ALPHA         = 255       // Alpha value of the bird unit.
    private constant real    BIRD_SCALE         = 1.        // Scaling size of the bird unit. 
    private constant real    BIRD_TLIFE         = 6.        // Timed life of the birds.
    private constant real    BIRD_OFFSET        = 200.      // Offset distance of the birds.
    private constant real    BIRD_HEIGHT        = 4.        // The interval for increasing flying height.
        
    private constant integer DUMMY_EXPLO        = 'h004'    // ID of the explosion unit.
    private constant boolean EXPLO_ON           = true      // Whether you want the explosion effect on.
    private constant integer EXPLO_RED          = 0         // Red value of the explosion unit.
    private constant integer EXPLO_GREEN        = 255       // Green value of the explosion unit.
    private constant integer EXPLO_BLUE         = 75        // Blue value of the explosion unit.
    private constant integer EXPLO_ALPHA        = 155       // Alpha value of the explosion unit.
    private constant real    EXPLO_SCALE        = 2.        // Scaling size of the explosion unit.
		
    private constant integer DUMMY_GREEN        = 'h006'    // ID of the green ball unit.
    private constant boolean GREEN_ON           = true      // Whether you want the green ball effect on.
    private constant integer GREEN_RED          = 255       // Red value of the green ball unit.
    private constant integer GREEN_GREEN        = 255       // Green value of the green ball unit.
    private constant integer GREEN_BLUE         = 255       // Blue value of the green ball unit.
    private constant integer GREEN_ALPHA        = 255       // Aplha value of the green ball unit.
    private constant real    GREEN_SCALE        = .75       // Scaling size of the green ball unit.
    private constant real    GREEN_MSPEED       = 100.      // Movement speed of the green ball unit.
		
    private constant integer DUMMY_LEAF         = 'h007'    // ID of the leaf unit.
    private constant boolean LEAF_ON            = true      // Whether you want the leaf effect on.
    private constant real    LEAF_SCALE         = 1.        // Scaling size of the leaf unit.
        
    private constant integer DUMMY_SPAWN        = 'h000'    // ID of the spawning effect unit.
    private constant boolean SPAWN_ON           = true      // Whether you want the spawning effect on.
    private constant integer SPAWN_RED          = 255       // Red value of the spawning effect unit.
    private constant integer SPAWN_GREEN        = 255       // Green value of the spawning effect unit.
    private constant integer SPAWN_BLUE         = 255       // Blue value of the spawning effect unit.
    private constant integer SPAWN_ALPHA        = 255       // Alpha value of the spawning effect unit.
    private constant real    SPAWN_SCALE        = 1.        // Scalaing size of the spawning effect unit.
    private constant integer SPAWN_NUM          = 5         // Amount of spawning effect units created on summon.
    private constant real    SPAWN_OFFSET       = 200.      // Offset of the spawning effect unit.
		
    private constant integer DUMMY_SPIN         = 'h003'    // ID of the spin dummy unit.
    private constant boolean SPIN_ON            = true      // Whether you want the spinning effect on.
    private constant integer SPIN_RED           = 255       // Red value of the spin dummy unit.
    private constant integer SPIN_GREEN         = 255       // Green value of the spin dummy unit.
    private constant integer SPIN_BLUE          = 255       // Blue value of the spin dummy unit.
    private constant real    SPIN_SCALE         = .85       // Scaling value of the spin dummy unit.
    private constant real    SPIN_OFFSET        = 250.      // Starting offset of the inward spinning.
    private constant real    SPIN_TLIFE         = 1.        // Timed life of the spin dummy unit.
        
    private constant integer SPAWNED_UNIT       = 'e000'    // ID of the spawned unit.
    private constant integer SPAWNED_UNIT_RED   = 255       // Red value of the spawned unit.
    private constant integer SPAWNED_UNIT_GREEN = 255       // Green value of the spawned unit.
    private constant integer SPAWNED_UNIT_BLUE  = 255       // Blue value of the spawned unit.
    private constant integer SPAWNED_UNIT_ALPHA = 255       // Alpha value of the spawned unit.
    private constant real    SPAWNED_UNIT_SCALE = 1.        // Scaling value of the spawned unit.
    private constant real    SPAWNED_UNIT_TLIFE = 60.       // Timed life of the spawned unit.
		
    private constant boolean RIPPLE_ON          = true      // Whether you want the ripple effect on.
    private constant real RIPPLE_TIME        = 3.        // Time length of the ripple.
    private constant real RIPPLE_S_RADIUS       = 50.       // Starting radius of the ripple.
    private constant real RIPPLE_E_RADIUS       = 300.      // Ending radius of the ripple.
    private constant real RIPPLE_DEPTH          = 64.       // Depth of the ripple.
    private constant real RIPPLE_SEQ            = 1.        // Time between each ripple.
    private constant real RIPPLE_SEQ_SPACE      = 50.       // Space between each ripple.
    
    private constant string SPAWN_SOUND         = "Abilities\\Spells\\Human\\MarkOfChaos\\MarkOfChaos.wav"
    // Sound played during spawn effects.
    private constant boolean SPAWN_SOUND_ON     = true      // Enable or disable the sound during the spawning effects.
    private constant integer SPAWN_SOUND_VOL    = 100       // Volume of the sound during the spawning effects. 
    // Recommended that this is lower then SUM_SOUND_VOL so that both sounds can be heard.
    
    private constant string SUM_SOUND           = "Abilities\\Spells\\Human\\Banish\\BanishLoop1.wav"
    // Background sound played during the summon.
    private constant boolean SUM_SOUND_ON       = true      // Enable or disable the sound during summon.
    private constant integer SUM_SOUND_VOL    = 127         // Volume of the sound during summon. 
    
    private constant string  EXPLO_SOUND        = "Units\\NightElf\\Ent\\ForceOfNatureBirth1.wav"
    // Sound played during the explosion effect.
    private constant boolean EXPLO_SOUND_ON     = true      // Enable or disable the sound during the explosion effect.
    private constant integer EXPLO_SOUND_VOL    = 127       // Volume of the sound during the explosion effect. 
    
    private constant string  SFX_BLINK          = "Abilities\\Spells\\NightElf\\Blink\\BlinkTarget.mdl"
    // Path of the blink sfx that occurs on cast to signify where birds will spawn from.

    private constant real    PERIOD             = .02       // Period of the KT2 timer.
    // Configurable time options, measured in seconds.

    private constant real    MAX_TIME           = 7.        // Max time of the spell in seconds.
    // Extra time past SUM_TIME is to account for the birds still visible on screen, to fly more
    
    private constant real    SUM_TIME           = 5.        // Amount of time that it takes to summon the unit.
    private constant real    SUM_SOUND_TIME_ST  = 0         // Amount of time into the summen when SUM_SOUND begins playing.
    private constant real    SUM_SOUND_TIME_END = 4.6       // Amount of time into the summon when SUM_SOUND is no longer played.
	
    private constant real    BIRD_START         = .6        // Time into the summon when birds spawn.
    private constant real    BIRD_END           = 2.5       // Time into the summon when birds stop spawning.
    
    private constant real    GREEN_START        = 0.        // Time into the summon when green ball effect spawns.
    private constant real    GREEN_END          = 3.        // Time into the summon when the green ball effect stop spawning.
    
    private constant real    LEAF_START         = 0.        // Time into the summon when leaves fly up.
    private constant real    LEAF_END           = SUM_TIME  // Time into the summon when leaves stop flying up.
   
    private constant real    SPIN_START         = 1         // Time into the summon when the dummys spin inward.
    private constant real    SPIN_END           = SUM_TIME  // Time into the summon when the spinning is done.

    private          integer filter
endglobals

private function Sound takes unit u returns nothing
    local sound s=CreateSound(SPAWN_SOUND,false,false,false,10,10,"")
    call StartSound(s)
    call AttachSoundToUnit(s,u)
    call SetSoundVolume(s,SPAWN_SOUND_VOL)
    call KillSoundWhenDone(s)
    set s=null
endfunction

private struct Data
    unit caster
    unit spawnEffect
    unit array bird[100]
    player p
    real x
    real y
    real angle
    integer array birdAlpha[100]
    group birds=CreateGroup()
    integer birdIndex=0
    real birdTimer=0
    real greenFloatTimer=0
    real leafTimer=0
    real spawnGrowth=1
    real spawnTimer=0
    real spawnAlpha=0
    real spinAngle=0
    real spinDistance=SPIN_OFFSET
    real spinTimer=0
    sound array sumSound[10]
    real sumSoundTimer=0
    real array sumSoundVol[10]
    integer ticks=R2I(MAX_TIME/PERIOD)
    integer maxTicks=.ticks
    
    static method Create takes unit u, real tx, real ty returns Data
        local Data d=Data.allocate()
        local integer i=1
        local unit dummy
        set d.caster=u
        set d.p=GetOwningPlayer(u)
        set d.x=tx
        set d.y=ty
        set d.angle=bj_DEGTORAD*(GetUnitFacing(u)+90)
        static if BEAM_ON then
            set dummy=CreateUnit(d.p,DUMMY_BEAM,tx,ty,0)
            call SetUnitVertexColor(dummy,BEAM_RED,BEAM_GREEN,BEAM_BLUE,BEAM_ALPHA)
            call SetUnitScale(dummy,BEAM_SCALE,0,0)
            call UnitApplyTimedLife(dummy,'BTLF',SUM_TIME)
        endif
        static if SPAWN_ON then
            set d.spawnEffect=CreateUnit(d.p,DUMMY_SPAWN,tx,ty,0)
            call SetUnitVertexColor(d.spawnEffect,SPAWN_RED,SPAWN_GREEN,SPAWN_BLUE,SPAWN_ALPHA)
            call SetUnitScale(d.spawnEffect,SPAWN_SCALE,0,0)
            static if SPAWN_SOUND_ON then
                call Sound(d.spawnEffect)
            endif
        endif
        static if SUM_SOUND_ON then
            loop
            exitwhen i>10
                set d.sumSound<i>=CreateSound(SUM_SOUND,false,false,false,10,10,&quot;&quot;)
                set d.sumSoundVol<i>=SUM_SOUND_VOL
                call StartSound(d.sumSound<i>)
                call AttachSoundToUnit(d.sumSound<i>,d.spawnEffect)
                call SetSoundVolume(d.sumSound<i>,SUM_SOUND_VOL)
                call KillSoundWhenDone(d.sumSound<i>)
                set i=i+1
            endloop
        endif
        call DestroyEffect(AddSpecialEffect(SFX_BLINK,tx+BIRD_OFFSET*Cos(d.angle),ty+BIRD_OFFSET*Sin(d.angle)))
        set dummy=null
        return d
    endmethod
    
    static method BirdSpin takes nothing returns nothing
        local Data d=Data(filter)
        local unit u=GetEnumUnit()
        local real x=GetUnitX(u)
        local real y=GetUnitY(u)
        local real angle=Atan2(y-d.y,x-d.x)
        local real x2=d.x+BIRD_OFFSET*Cos(angle+.087)
        local real y2=d.y+BIRD_OFFSET*Sin(angle+.087)
        call SetUnitX(u,x2)
        call SetUnitY(u,y2)
        set x=d.x+BIRD_OFFSET*Cos(angle+.105)
        set y=d.y+BIRD_OFFSET*Sin(angle+.105)
        call SetUnitFacing(u,Atan2(y-y2,x-x2)/.0175)
        call SetUnitFlyHeight(u,GetUnitFlyHeight(u)+BIRD_HEIGHT,0)
        set u=null
    endmethod
    
    method Summon takes nothing returns nothing
        local sound s
        local integer i=0
        local real angle
        local real x
        local real y
        local real x2
        local real y2
        local unit d
        set filter=this
        if .ticks==.maxTicks-SUM_TIME/PERIOD then
            static if RIPPLE_ON then
                call TerrainDeformRipple(.x,.y,RIPPLE_E_RADIUS,RIPPLE_DEPTH,R2I(RIPPLE_TIME)*1000,1,2.*RIPPLE_E_RADIUS/RIPPLE_SEQ_SPACE,2.*RIPPLE_TIME/RIPPLE_SEQ,RIPPLE_S_RADIUS/RIPPLE_E_RADIUS,false)
            endif
            static if EXPLO_ON then
                set d=CreateUnit(.p,DUMMY_EXPLO,.x,.y,0)
                call SetUnitVertexColor(d,EXPLO_RED,EXPLO_GREEN,EXPLO_BLUE,EXPLO_ALPHA)
                call SetUnitScale(d,EXPLO_SCALE,0,0)
            endif
            static if EXPLO_SOUND_ON then
                set s=CreateSound(EXPLO_SOUND,false,false,false,10,10,&quot;&quot;)
                call StartSound(s)
                call AttachSoundToUnit(s,d)
                call SetSoundVolume(s,EXPLO_SOUND_VOL)
                call KillSoundWhenDone(s)
            endif
            static if SPAWN_ON then
                loop
                exitwhen i&gt;SPAWN_NUM-1
                    set x=.x+SPAWN_OFFSET*Cos(i*1.26)
                    set y=.y+SPAWN_OFFSET*Sin(i*1.26)
                    set d=CreateUnit(.p,DUMMY_SPAWN,x,y,0)
                    call SetUnitVertexColor(d,SPAWN_RED,SPAWN_GREEN,SPAWN_BLUE,SPAWN_ALPHA)
                    call SetUnitScale(d,SPAWN_SCALE,0,0)
                    call UnitApplyTimedLife(d,&#039;BTLF&#039;,.75)
                    static if SPAWN_SOUND_ON then
                        call Sound(d)
                    endif
                    set i=i+1
                endloop
            endif
            call RemoveUnit(.spawnEffect)
            set d=CreateUnit(.p,SPAWNED_UNIT,.x,.y,.angle/.0175-90)
            call SetUnitVertexColor(d,SPAWNED_UNIT_RED,SPAWNED_UNIT_GREEN,SPAWNED_UNIT_BLUE,SPAWNED_UNIT_ALPHA)
            call SetUnitScale(d,SPAWNED_UNIT_SCALE,0,0)
            call UnitApplyTimedLife(d,&#039;BTLF&#039;,SPAWNED_UNIT_TLIFE)
            call SetUnitAnimation(d,&quot;birth&quot; )
            call QueueUnitAnimation(d,&quot;stand&quot;)
        endif
        static if SUM_SOUND_ON then
            if .sumSoundTimer==1.5 and .ticks&lt;=.maxTicks-SUM_SOUND_TIME_ST/PERIOD and .ticks&gt;.maxTicks-SUM_SOUND_TIME_END/PERIOD then
                set i=0
                loop
                exitwhen i&gt;10
                    set .sumSound<i>=CreateSound(SUM_SOUND,false,false,false,10,10,&quot;&quot;)
                    call StartSound(.sumSound<i>)
                    call AttachSoundToUnit(.sumSound<i>,.spawnEffect)
                    call SetSoundVolume(.sumSound<i>,SUM_SOUND_VOL)
                    call KillSoundWhenDone(.sumSound<i>)
                    set i=i+1
                endloop
                set .sumSoundTimer=0
            else
                set .sumSoundTimer=.sumSoundTimer+PERIOD
            endif
            if .ticks&lt;.maxTicks-220 then
                set i=0
                loop
                exitwhen i&gt;4
                    set .sumSoundVol<i>=.sumSoundVol<i>-1
                    call SetSoundVolume(.sumSound<i>,R2I(.sumSoundVol<i>))
                    set i=i+1
                endloop
            endif
        endif
        static if GREEN_ON then 
            if .ticks&lt;.maxTicks-GREEN_START/PERIOD and .ticks&gt;=.maxTicks-GREEN_END/PERIOD and .greenFloatTimer&gt;=25*PERIOD then
                set i=0
                loop
                exitwhen i&gt;4
                    set x2=GetRandomInt(250,400)
                    set angle=GetRandomReal(0,6.283)
                    set x=.x+x2*Cos(angle)
                    set y=.y+x2*Sin(angle)
                    set d=CreateUnit(.p,DUMMY_GREEN,x,y,0)
                    call SetUnitVertexColor(d,GREEN_RED,GREEN_GREEN,GREEN_BLUE,GREEN_ALPHA)
                    call SetUnitScale(d,GREEN_SCALE,0,0)
                    call SetUnitMoveSpeed(d,GREEN_MSPEED)
                    call UnitApplyTimedLife(d,&#039;BTLF&#039;,(.ticks-100)*PERIOD)
                    call IssuePointOrder(d,&quot;move&quot;,.x,.y)
                    set i=i+1
                endloop
                set .greenFloatTimer=0
            else
                set .greenFloatTimer=.greenFloatTimer+PERIOD
            endif
        endif
        static if SPAWN_ON then
            if .ticks&gt;=.maxTicks-SUM_TIME/PERIOD and .spawnTimer&gt;=50*PERIOD then
                call Sound(.spawnEffect)
                set .spawnGrowth=.spawnGrowth+.25
                call SetUnitScale(.spawnEffect,.spawnGrowth,.spawnGrowth,.spawnGrowth)
                call SetUnitAnimation(.spawnEffect,&quot;birth&quot;)
                set .spawnTimer=0
            else
                set .spawnTimer=.spawnTimer+PERIOD
            endif
        endif
        static if SPIN_ON then 
            if .ticks&gt;.maxTicks-(SPIN_END/PERIOD-40) and .ticks&lt;.maxTicks-SPIN_START/PERIOD then
                if .spinTimer==3*PERIOD then
                    set i=0
                    loop
                    exitwhen i&gt;2
                        set x=.x+.spinDistance*Cos(.angle+.spinAngle+2.1*i)
                        set y=.y+.spinDistance*Sin(.angle+.spinAngle+2.1*i)
                        set x2=.x+.spinDistance*Cos(.angle+.spinAngle+(2.1*i)+.0175)
                        set y2=.y+.spinDistance*Sin(.angle+.spinAngle+(2.1*i)+.0175)
                        set angle=Atan2(y2-y,x2-x)
                        set d=CreateUnit(.p,DUMMY_SPIN,x,y,angle/.0175)
                        call SetUnitVertexColor(d,SPIN_RED,SPIN_GREEN,SPIN_BLUE,0)
                        call SetUnitScale(d,SPIN_SCALE,0,0)
                        call UnitApplyTimedLife(d,&#039;BTLF&#039;,SPIN_TLIFE)
                        call SetUnitTimeScale(d,0)
                        call SetUnitVertexColor(d,255,255,255,R2I(.spawnAlpha))
                        set i=i+1
                    endloop
                    set .spinTimer=0
                else
                    set .spinTimer=.spinTimer+PERIOD
                endif
                if .spawnAlpha&lt;=255 then
                    set .spawnAlpha=.spawnAlpha+.3825
                endif
                set .spinAngle=.spinAngle+.035
                set .spinDistance=.spinDistance-1.5
            endif
        endif
        static if BIRD_ON then
            set i=0
            loop
            exitwhen i==.birdIndex
                set .birdAlpha<i>=.birdAlpha<i>-1
                call SetUnitVertexColor(.bird<i>,BIRD_RED,BIRD_GREEN,BIRD_BLUE,.birdAlpha<i>)
                set i=i+1
            endloop
            if  .ticks&lt;.maxTicks-BIRD_START/PERIOD and .ticks&gt;.maxTicks-BIRD_END/PERIOD then
                if .birdTimer==3*PERIOD then
                    set x=.x+BIRD_OFFSET*Cos(.angle)
                    set y=.y+BIRD_OFFSET*Sin(.angle)
                    set x2=.x+BIRD_OFFSET*Cos(.angle+.0175)
                    set y2=.y+BIRD_OFFSET*Sin(.angle+.0175)
                    set angle=Atan2(y2-y,x2-x)
                    set d=CreateUnit(.p,DUMMY_BIRD,x,y,angle/.0175)
                    set .bird[.birdIndex]=d
                    set .birdAlpha[.birdIndex]=BIRD_ALPHA
                    set .birdIndex=.birdIndex+1
                    call SetUnitVertexColor(d,BIRD_RED,BIRD_GREEN,BIRD_BLUE,BIRD_ALPHA)
                    call SetUnitScale(d,BIRD_SCALE,0,0)
                    call UnitApplyTimedLife(d,&#039;BTLF&#039;,BIRD_TLIFE)
                    call GroupAddUnit(.birds,d)
                    call UnitAddAbility(d,&#039;Amrf&#039;)
                    set .birdTimer=0
                else
                    set .birdTimer=.birdTimer+PERIOD
                endif
            endif
            call ForGroup(.birds,function Data.BirdSpin)
        endif
        static if LEAF_ON then 
            if .leafTimer==15*PERIOD and .ticks&lt;.maxTicks-LEAF_START/PERIOD and .ticks&gt;.maxTicks-LEAF_END/PERIOD then
                set x2=GetRandomReal(0,150)
                set angle=GetRandomReal(0,6.283)
                set x=.x+x2*Cos(angle)
                set y=.y+x2*Sin(angle)
                set d=CreateUnit(.p,DUMMY_LEAF,x,y,GetRandomReal(0,150))
                call SetUnitScale(d,LEAF_SCALE,0,0)
                call SetUnitAnimationByIndex(d,3)
                call UnitApplyTimedLife(d,&#039;BTLF&#039;,2.5)
                set .leafTimer=0
            else
                set .leafTimer=.leafTimer+PERIOD
            endif
        endif
        set d=null
        set d=null
    endmethod
endstruct 

private function PeriodicFunc takes nothing returns boolean
    local Data d=KT_GetData()
    set d.ticks=d.ticks-1
    if d.ticks&lt;=0 then  
        call d.destroy()
        return true
    endif
    call d.Summon()
    return false 
endfunction 

private function Remove takes nothing returns nothing
    local integer i=GetUnitTypeId(GetTriggerUnit())
    if i==DUMMY_BEAM or i==DUMMY_BIRD or i==DUMMY_EXPLO or i==DUMMY_GREEN or i==DUMMY_LEAF or i==DUMMY_SPAWN or i==DUMMY_SPIN then
        call RemoveUnit(GetTriggerUnit())
    endif
endfunction

private function Actions takes nothing returns nothing
    call KT_Add(function PeriodicFunc,Data.Create(GetTriggerUnit(),GetSpellTargetX(),GetSpellTargetY()),PERIOD)
endfunction

private function init takes nothing returns nothing
    local trigger t=CreateTrigger()
    local sound s
    call TriggerRegisterAnyUnitEventBJ(t,EVENT_PLAYER_UNIT_DEATH)
    call TriggerAddAction(t,function Remove)
    call TriggerAddAction(GT_RegisterStartsEffectEvent(CreateTrigger(),ID),function Actions)
    set s=CreateSound(SUM_SOUND,false,false,false,10,10,&quot;&quot;)
    call StartSound(s)
    call SetSoundVolume(s,0)
    call KillSoundWhenDone(s)
    set s=CreateSound(SPAWN_SOUND,false,false,false,10,10,&quot;&quot;)
    call StartSound(s)
    call SetSoundVolume(s,0)
    call KillSoundWhenDone(s)
    set s=CreateSound(EXPLO_SOUND,false,false,false,10,10,&quot;&quot;)
    call StartSound(s)
    call SetSoundVolume(s,0)
    call KillSoundWhenDone(s)
    set s=null
endfunction

endscope</i></i></i></i></i></i></i></i></i></i></i></i></i></i></i></i></i></i></i>


I tried to make it as customizable as possible. If there are other things that you think should be customizable, let me know.
Feedback is welcomed, if you see any bugs or mistakes let me know.
 

Attachments

  • Summon Treant.w3x
    73.1 KB · Views: 587
Ok.

1) I've got a major problem with the globals section.. it's a giant mess. Inconsistent indenting. No spacing to categorise it (even just a bit). It's just a giant messy block right now.

2) Use [ljass]static if[/ljass] for constant booleans. Like SPAWN_ON and BEAM_ON.

I don't have much motivation to look it thorougly, since to me it's really messy. It's me. You can't really fix that. I'm picky. I prefer stuff like [ljass]call KillUnit( unit )[/ljass] to [ljass]call KillUnit(unit)[/ljass] (note the spacing between the brackets).

Anyways the spell looks good :) Didn't it come like 2nd or 1st or something for the Flash SFX contest?
 
Alright.
I figured the global block would get that, but it didn't really seem too messy for me. I'll space out certain chunks and hopefully make it more neat.
What is a static if? I've never heard of that (since I've been gone).
I prefer the no space between brackets because its not hard to read without spaces, but thats just me.
Thanks, yes it tied for first.
 
A static if compiles only if it matches the boolean.... if it doesn't. It's not compiled. So it doesn't check for the if = efficiency I think?

like:

JASS:
globals
    constant boolean KILL_ENEMY = true
endglobals

static if KILL_ENEMY then
    call KillUnit( unit )
endif


[ljass]static if not KILL_ENEMY then[/ljass]
[ljass] call ShowUnit( unit, false )[/ljass]
[ljass]endif[/ljass]
 
looks neat, but I would prefer to use this for summoning some boss unit, not a treant :p
 
BRUTAL said:
Alright.
I figured the global block would get that, but it didn't really seem too messy for me. I'll space out certain chunks and hopefully make it more neat.
What is a static if? I've never heard of that (since I've been gone).
I prefer the no space between brackets because its not hard to read without spaces, but thats just me.
Thanks, yes it tied for first.
Click to expand...

no, it's not just you. i dislike lots of useless whitespace myself.

as for the spell...it looks pretty good. although i don't like the way it sounds...seems too mechanical. like a factory sound or something.
 
JASS:
globals
    private constant integer ID                 = &#039;A000&#039;    // ID of the spell.
    
    private constant integer DUMMY_BEAM         = &#039;h005&#039;    // ID of the beam unit.
    private constant boolean BEAM_ON            = true      // Whether you want the beam effect on.
    private constant integer BEAM_RED           = 0         // Red value of the beam unit.
    private constant integer BEAM_GREEN         = 255       // Green value of the beam unit.
    private constant integer BEAM_BLUE          = 0         // Blue value of the beam unit.
    private constant integer BEAM_ALPHA         = 255       // Alpha value of the beam unit
    private constant real    BEAM_SCALE         = 1.        // Scaling size of the beam unit.
        
		
    private constant integer DUMMY_BIRD         = &#039;h001&#039;    // ID of the bird unit.
    private constant boolean BIRD_ON            = true      // Whether you want the bird effect on.
    private constant integer BIRD_RED           = 255       // Red value of the bird unit.
    private constant integer BIRD_GREEN         = 255       // Green value of the bird unit.
    private constant integer BIRD_BLUE          = 255       // Blue value of the bird unit.
    private constant integer BIRD_ALPHA         = 255       // Alpha value of the bird unit.
    private constant real    BIRD_SCALE         = 1.        // Scaling size of the bird unit. 
    private constant real    BIRD_TLIFE         = 6.        // Timed life of the birds.
    private constant real    BIRD_OFFSET        = 200.      // Offset distance of the birds.
    private constant real    BIRD_HEIGHT        = 4.        // The interval for increasing flying height.
        
		
    private constant integer DUMMY_EXPLO        = &#039;h004&#039;    // ID of the explosion unit.
    private constant boolean EXPLO_ON           = true      // Whether you want the explosion effect on.
    private constant integer EXPLO_RED          = 0         // Red value of the explosion unit.
    private constant integer EXPLO_GREEN        = 255       // Green value of the explosion unit.
    private constant integer EXPLO_BLUE         = 75        // Blue value of the explosion unit.
    private constant integer EXPLO_ALPHA        = 155       // Alpha value of the explosion unit.
    private constant real    EXPLO_SCALE        = 2.        // Scaling size of the explosion unit.
        
		
    private constant integer DUMMY_GREEN        = &#039;h006&#039;    // ID of the green ball unit.
    private constant boolean GREEN_ON           = true      // Whether you want the green ball effect on.
    private constant integer GREEN_RED          = 255       // Red value of the green ball unit.
    private constant integer GREEN_GREEN        = 255       // Green value of the green ball unit.
    private constant integer GREEN_BLUE         = 255       // Blue value of the green ball unit.
    private constant integer GREEN_ALPHA        = 255       // Aplha value of the green ball unit.
    private constant real    GREEN_SCALE        = .75       // Scaling size of the green ball unit.
    private constant real    GREEN_MSPEED       = 100.      // Movement speed of the green ball unit.
        
		
    private constant integer DUMMY_LEAF         = &#039;h007&#039;    // ID of the leaf unit.
    private constant boolean LEAF_ON            = true      // Whether you want the leaf effect on.
    private constant real    LEAF_SCALE         = 1.        // Scaling size of the leaf unit.
        
		
    private constant integer DUMMY_SPAWN        = &#039;h000&#039;    // ID of the spawning effect unit.
    private constant boolean SPAWN_ON           = true      // Whether you want the spawning effect on.
    private constant integer SPAWN_RED          = 255       // Red value of the spawning effect unit.
    private constant integer SPAWN_GREEN        = 255       // Green value of the spawning effect unit.
    private constant integer SPAWN_BLUE         = 255       // Blue value of the spawning effect unit.
    private constant integer SPAWN_ALPHA        = 255       // Alpha value of the spawning effect unit.
    private constant integer SPAWN_NUM          = 5         // Amount of spawning effect units created on summon.
    private constant real    SPAWN_OFFSET       = 200.      // Offset of the spawning effect unit.
    private constant real    SPAWN_SCALE        = 1.        // Scalaing size of the spawning effect unit.
        
		
    private constant integer DUMMY_SPIN         = &#039;h003&#039;    // ID of the spin dummy unit.
    private constant boolean SPIN_ON            = true      // Whether you want the spinning effect on.
    private constant real    SPIN_OFFSET        = 250.      // Starting offset of the inward spinning.
    private constant integer SPIN_RED           = 225       // Red value of the spin dummy unit.
    private constant integer SPIN_GREEN         = 255       // Green value of the spin dummy unit.
    private constant integer SPIN_BLUE          = 255       // Blue value of the spin dummy unit.
    private constant real    SPIN_SCALE         = .85       // Scaling value of the spin dummy unit.
    private constant real    SPIN_TLIFE         = 1.        // Timed life of the spin dummy unit.
        
		
    private constant integer SPAWNED_UNIT       = &#039;e000&#039;    // ID of the spawned unit.
    private constant integer SPAWNED_UNIT_RED   = 255       // Red value of the spawned unit.
    private constant integer SPAWNED_UNIT_GREEN = 255       // Green value of the spawned unit.
    private constant integer SPAWNED_UNIT_BLUE  = 255       // Blue value of the spawned unit.
    private constant integer SPAWNED_UNIT_ALPHA = 255       // Alpha value of the spawned unit.
    private constant real    SPAWNED_UNIT_SCALE = 1.        // Scaling value of the spawned unit.
    private constant real    SPAWNED_UNIT_TLIFE = 60.       // Timed life of the spawned unit.
        
		
    private constant boolean RIPPLE_ON          = true      // Whether you want the ripple effect on.
    private constant string  SOUND              = &quot;Units\\NightElf\\Ent\\ForceOfNatureBirth1.wav&quot;
    // Path of the sound that is played when the spawn effect birth animation is looped.
	
    private constant string  SFX_BLINK          = &quot;Abilities\\Spells\\NightElf\\Blink\\BlinkTarget.mdl&quot;
    // Path of the blink sfx that occurs on cast to signify where birds will spawn from.
	
    private constant real    PERIOD             = .02       // Period of the KT2 timer.
    // Configurable time options, measured in seconds.
    
    private constant real    MAX_TIME           = 7.        // Max time of the spell in seconds.
    // Extra time past SUM_TIME is to account for the birds still visible on screen, to fly more
	
	
    private constant real    SUM_TIME           = 5.        // Amount of time that it takes to summon the unit.
    
	
    private constant real    BIRD_START         = .6        // Time into the summon when birds spawn.
    private constant real    BIRD_END           = 2.5       // Time into the summon when birds stop spawning.
    
	
    private constant real    GREEN_START        = 0.        // Time into the summon when green ball effect spawns.
    private constant real    GREEN_END          = 3.        // Time into the summon when the green ball effect stop spawning.
    
	
    private constant real    LEAF_START         = 0.        // Time into the summon when leaves fly up.
    private constant real    LEAF_END           = SUM_TIME  // Time into the summon when leaves stop flying up.
   
   
    private constant real    SPIN_START         = 1         // Time into the summon when the dummys spin inward.
    private constant real    SPIN_END           = SUM_TIME  // Time into the summon when the spinning is done.


    private          integer filter
endglobals

:rolleyes:
 
Scope names, non-constant globals, and (all) function names should be named in UpperCamelCase. Simple naming conventions make things easier to read, and also allow people to easily identify what type a variable, function, etc is. This isn't quite as much of an issue here, as everything is private besides the scope name.
_____________

There also seems to be some confusion about static ifs:
JASS:
globals
    constant boolean BOOL1 = false
    constant boolean BOOL2 = true
endglobals

function ExampleFunc takes nothing returns nothing
    static if BOOL1 then
        call KillUnit(GetTriggerUnit())
    endif
    static if BOOL2 then
        call GetRandomReal(2, 4)
    endif
endfunction

Compiles to:
JASS:
globals
    constant boolean BOOL1 = false
    constant boolean BOOL2 = true
endglobals

function ExampleFunc takes nothing returns nothing
//    static if BOOL1 then
//        call KillUnit(GetTriggerUnit())
//    endif
//    static if BOOL2 then
        call GetRandomReal(2, 4)
//    endif
endfunction

More efficient? Oh yes. :p
_____________

Function 'Sound' doesn't need the string parameter, as all that's being passed to it is a constant, it seems. Just use the constant directly.
_____________

JASS:
.
    if d.ticks&lt;=0 then  
        call d.destroy()
        return true
    else
        call d.summon()
        return false
    endif

Statements like these could be changed to:
JASS:
.
    if d.ticks&lt;=0 then  
        call d.destroy()
        return true
    endif
    call d.summon()
    return false

_____________

JASS:
        if LEAF_ON and .leafTimer==15*PERIOD and .ticks&lt;.maxTicks-LEAF_START/PERIOD and .ticks&gt;.maxTicks-LEAF_END/PERIOD then
            set x2=GetRandomReal(0,150)
            &lt;...Other Stuff...&gt;
            set .leafTimer=0
        else
            set .leafTimer=.leafTimer+PERIOD
        endif

If [ljass]LEAF_ON[/ljass] is false, [ljass].leafTimer[/ljass] is being pointlessly incremented. Other similar issues are also present in the code.
_____________

You could use radians directly instead of using all those conversions from degrees.
_____________

In method 'create', [ljass]real x[/ljass] and [ljass]real y[/ljass] are only used once. You should just inline them. [ljass]real angle[/ljass] could then also be removed, and the struct member could be used instead.
_____________

I'm sure I've missed quite a few things in this post, but it'll do for now.
 
@Laiev:
That's the style of coding I am liking :D

Bitch Romek Bitch. I nag you for some time now and you don't check my resource :( But this guy gets like ZOMG. Lolz. Meh.

I think this is overkill for a treant..
Click to expand...

It was for a fancy sfx contest :p
 
@BlackRose

same here :thup:

i just forget to add space to the rawcode ability D: (already fixed)
 
I dont really like that looking globals :p.

JASS:
globals
    private constant integer SOMETHING   = &#039;A000&#039;
    private constant real ANOTHER_THING = bj_PI
endglobals


i think this could look nicer like this:

JASS:
globals
    //The Rawcode for the Spell
    private constant integer SOMETHIN     = &#039;A000&#039;
    //Declaring bj_PI for some strange reason. -.-
    private constant real ANOTHER_THING = bj_PI
endglobals


EDIT:

In your actions, you can store GetTriggerUnit(), GetSpellTargetPoint(Loc?) and, then use them. I think what you use now causes leaks.
 
Crazy_Dead said:
JASS:
globals
    //The Rawcode for the Spell
    private constant integer SOMETHIN     = &#039;A000&#039;
    //Declaring bj_PI for some strange reason. -.-
    private constant real ANOTHER_THING = bj_PI
endglobals
Click to expand...

If it was something only one person worked on and they found it natural. That's ok :)

But this is a public resource.... to me, I can't stand reading something like that (what you suggested). The no space after the [ljass]//[/ljass] irritates me. But that's enough of that...

In your actions, you can store GetTriggerUnit(), GetSpellTargetPoint(Loc?) and, then use them. I think what you use now causes leaks.
Click to expand...

[ljass]real[/ljass] don't leak. [ljass]unit[/ljass] won't leak as it's not being declared as local, it's being used straight off.*

*Need citation :(
 
Dinowc said:
looks neat, but I would prefer to use this for summoning some boss unit, not a treant :p
Click to expand...
Thanks. Well who says a treant can't be a boss, just because its small? lol.

SanKakU said:
no, it's not just you. i dislike lots of useless whitespace myself.

as for the spell...it looks pretty good. although i don't like the way it sounds...seems too mechanical. like a factory sound or something.
Click to expand...
Good, I'm only going to make the globals spaced out because of the large amount of variables.
Thanks anyways, but I agree with the sound. I changed the sounds to give it a better sense of belonging, I'll find even more suitable ones soon enough.

Crazy_Dead said:
I think this is overkill for a treant..
Click to expand...
Why? And its not just a treant, look at its skills; that is a treant I'd want fighting for me. I even changed the name of it so its a little more than a treant.

At Laiev; thanks, I used that global block as a template.

Romek said:
Function 'Sound' doesn't need the string parameter, as all that's being passed to it is a constant, it seems. Just use the constant directly.
Click to expand...
Oh my mistake, I must have accidentally added that parameter in randomly one night. I changed the sounds so the function is slightly different now.

Romek said:
JASS:
.
    if d.ticks&lt;=0 then  
        call d.destroy()
        return true
    else
        call d.summon()
        return false
    endif

Statements like these could be changed to:
JASS:
.
    if d.ticks&lt;=0 then  
        call d.destroy()
        return true
    endif
    call d.summon()
    return false
Click to expand...
Alright I changed that.

Romek said:
JASS:
        if LEAF_ON and .leafTimer==15*PERIOD and .ticks&lt;.maxTicks-LEAF_START/PERIOD and .ticks&gt;.maxTicks-LEAF_END/PERIOD then
            set x2=GetRandomReal(0,150)
            &lt;...Other Stuff...&gt;
            set .leafTimer=0
        else
            set .leafTimer=.leafTimer+PERIOD
        endif

If [ljass]LEAF_ON[/ljass] is false, [ljass].leafTimer[/ljass] is being pointlessly incremented. Other similar issues are also present in the code.
Click to expand...
Wow, thanks for pointing that out. I just remembered how inlining conditions was always good. I fixed all those issues.

Romek said:
You could use radians directly instead of using all those conversions from degrees.
_____________

In method 'create', [ljass]real x[/ljass] and [ljass]real y[/ljass] are only used once. You should just inline them. [ljass]real angle[/ljass] could then also be removed, and the struct member could be used instead.
Click to expand...
Done.

I also made the ripple effects configurable, as well as the new sounds with some properties.
 
You must log in or register to reply here.
General chit-chat
Help Users
  • No one is chatting at the moment.
  • The Helper The Helper:
    Check out the Halloween Cocktails - https://www.thehelper.net/threads/cocktail-halloween-cocktails.196616/
  • Varine Varine:
    youtube.com

    EPIC: The Musical

    Full Playlist of EPIC: The Musical The Troy Saga (5 songs) The Cyclops Saga (4 songs) The Ocean Saga (4 songs) The Circe Saga (4 songs) The Underworld Saga (...
    youtube.com youtube.com
    +1
  • Varine Varine:
    I've been really into this
  • Varine Varine:
    It's a musical retelling of The Odyssey
  • Varine Varine:
    Also if you want mixed drinks without alcohol, there's a handful of non-alcoholic 'spirits'. Ritual Zero and Free Spirit are the two brands I've tried and they are alright. They don't have a TON of options, but they have some gin and whiskey alternatives that are fun to play with.
  • Varine Varine:
    I got a couple bottles to make some mixy drinks around holidays when I mostly want to drink, they aren't exact replacements but they are surprisingly close.
  • Varine Varine:
    I ended up with some hop water things that I really like instead for when I want to feel like I'm participating
  • Varine Varine:
    I just got moved to unsupervised probation, so I can get away with drinking a bit now. I technically am not supposed to, but I don't think I get checked anymore. I really want to smoke weed but it scares me
  • The Helper The Helper:
    Fruity Vodka Cocktail Recipes https://www.thehelper.net/threads/cocktail-fruity-vodka-cocktails.196623/
  • The Helper The Helper:
    Happy Wednesday! I am not feeling it today, my teeth are bothering me, or the fragments that were left behind are coming up and it is extra painful today. Don't ever let them tell you that getting full dental implants is easy. They have to pull all your teeth to do those and apparently they are not very good at getting all the teeth out all the time.
     +1
  • The Helper The Helper:
    That is true with Dentures too. I am taking the day off and working from home. The big difference is though, I am not drinking anymore so I have a nice tea and coffee regimen set up :)
  • Varine Varine:
    I wish i had a 3D scanner
  • Varine Varine:
    That would make what I'm doing SO much easier
  • Varine Varine:
    I guess I could just take a picture of it and then use that as a reference. I have a circuit board with some weird holes I can't get measured right
  • Varine Varine:
    Oh I got a Klipper mod for my big printer! It comes with a bunch of parts and I don't totally understand what it does but apparently it's way better than Marlin, which is I think what it currently runs
     +1
  • The Helper The Helper:
    Nice!
  • The Helper The Helper:
    Check out the Purple Rain cocktail - https://www.thehelper.net/threads/cocktail-purple-rain.196665/
  • Varine Varine:
    You're people in Florida safe? The only couple I know are safe but I think they evacuated.
  • The Helper The Helper:
    They are without power but safe. Flood damage and stuff alot of tornados hit east Florida. No fatalities or injuries to any of my people that I know of
  • The Helper The Helper:
    They have been through all the Florida hurricanes they are like me I have been through every texas hurricane for the last 50 years.
  • The Helper The Helper:
    Never evacuated.
  • The Helper The Helper:
    Check out this bad ass Chicken Noodle soup recipe - https://www.thehelper.net/threads/crack-chicken-noodle-soup.196687/
  • The Helper The Helper:
    Happy Saturday!
     +1
  • The Helper The Helper:
    New Cajun pasta recipe with steak - https://www.thehelper.net/threads/pasta-creamy-cajun-steak-alfredo.196727/
  • The Helper The Helper:
    Check out Poor Man's Husband Casserole - https://www.thehelper.net/threads/casserole-poor-mans-husband-casserole.196740/

      The Helper Discord

      Members online

      No members online now.
      Online now:
      325 (members: 0, guests: 325)

      Share this page

      Affiliates

      Hive Workshop NUON Dome World Editor Tutorials

      Network Sponsors

      Apex Steel Pipe - Buys and sells Steel Pipe.
      Top