Life Break v. 1.1 {Steel0RaZoR)

[Moderator]

13:21, 1st Jun 2011
Maker: The spell is too simple to be approved. Keep on practising

 

[Adiktuz]

please post triggers on the descriptions please...

 

[Jay the Editor]

Your trigger

[trigger=Life Break]]Life Break
Events
Unit - A unit Begins casting an ability
Conditions
Life Break Equal to (Ability being cast)
Actions
Set LifeBreak[1] = (Target unit of ability being cast)
Set LifeBreak[2] = (Casting unit)
If (All Conditions are True) then do (Then Actions) else do (Else Actions)
If - Conditions
(Level of Life Break for LifeBreak[2]) Equal to 1
Then - Actions
If (All Conditions are True) then do (Then Actions) else do (Else Actions)
If - Conditions
(Life of LifeBreak[2]) Greater than or equal to 50.00
Then - Actions
Unit - Set life of LifeBreak[1] to ((Life of LifeBreak[1]) x 0.50)
Unit - Set life of LifeBreak[2] to ((Life of LifeBreak[2]) x 0.35)
Unit - Create 1 Life Break Dummy for (Owner of LifeBreak[2]) at (Position of LifeBreak[2]) facing Default building facing degrees
Set LifeBreak[3] = (Last created unit)
Unit - Add a 5.00 second Generic expiration timer to LifeBreak[3]
Unit - Add Life Break Slow to LifeBreak[3]
Unit - Order LifeBreak[3] to Human Sorceress - Slow LifeBreak[1]
Special Effect - Create a special effect attached to the origin of LifeBreak[1] using Abilities\Spells\Demon\DarkPortal\DarkPortalTarget.mdl
Special Effect - Destroy (Last created special effect)
Unit - Explode LifeBreak[3]
Else - Actions
Else - Actions
If (All Conditions are True) then do (Then Actions) else do (Else Actions)
If - Conditions
(Level of Life Break for (Casting unit)) Equal to 2
Then - Actions
If (All Conditions are True) then do (Then Actions) else do (Else Actions)
If - Conditions
(Life of (Casting unit)) Greater than or equal to 50.00
Then - Actions
Unit - Set life of LifeBreak[1] to ((Life of LifeBreak[1]) x 0.50)
Unit - Set life of LifeBreak[2] to ((Life of LifeBreak[2]) x 0.30)
Unit - Create 1 Life Break Dummy for (Owner of LifeBreak[2]) at (Position of LifeBreak[2]) facing Default building facing degrees
Set LifeBreak[3] = (Last created unit)
Unit - Add a 5.00 second Generic expiration timer to LifeBreak[3]
Unit - Add Life Break Slow to LifeBreak[3]
Unit - Order LifeBreak[3] to Human Sorceress - Slow LifeBreak[1]
Special Effect - Create a special effect attached to the origin of LifeBreak[1] using Abilities\Spells\Demon\DarkPortal\DarkPortalTarget.mdl
Special Effect - Destroy (Last created special effect)
Unit - Explode LifeBreak[3]
Else - Actions
Else - Actions
If (All Conditions are True) then do (Then Actions) else do (Else Actions)
If - Conditions
(Level of Life Break Slow for (Casting unit)) Equal to 3
Then - Actions
If (All Conditions are True) then do (Then Actions) else do (Else Actions)
If - Conditions
(Life of (Casting unit)) Greater than or equal to 50.00
Then - Actions
Unit - Set life of LifeBreak[1] to ((Life of LifeBreak[1]) x 0.50)
Unit - Set life of LifeBreak[2] to ((Life of LifeBreak[2]) x 0.25)
Unit - Create 1 Life Break Dummy for (Owner of LifeBreak[2]) at (Position of LifeBreak[2]) facing Default building facing degrees
Set LifeBreak[3] = (Last created unit)
Unit - Add a 5.00 second Generic expiration timer to LifeBreak[3]
Unit - Add Life Break Slow to LifeBreak[3]
Unit - Order LifeBreak[3] to Human Sorceress - Slow LifeBreak[1]
Special Effect - Create a special effect attached to the origin of LifeBreak[1] using Abilities\Spells\Demon\DarkPortal\DarkPortalTarget.mdl
Special Effect - Destroy (Last created special effect)
Unit - Explode LifeBreak[3]
Else - Actions
Else - Actions
[/trigger]

Next time please post your trigger instead.

• Unit - A unit Begins casting an ability

I will prefer to use

• Unit - A unit Starts the effect of an ability

Btw, you don't need to repeat the If/Then/Else condition just to differentiate the level of Life Break. You can make it like this instead

Simpler Trigger

[trigger=Life Break]Life Break
Events
Unit - A unit Starts the effect of an ability
Conditions
Life Break Equal to (Ability being cast)
Actions
Set LifeBreak[1] = (Target unit of ability being cast)
Set LifeBreak[2] = (Casting unit)
If (All Conditions are True) then do (Then Actions) else do (Else Actions)
If - Conditions
(Life of LifeBreak[2]) Greater than or equal to 50.00
Then - Actions
Unit - Set life of LifeBreak[1] to ((Life of LifeBreak[1]) x 0.50)
Unit - Set life of LifeBreak[2] to ((Life of LifeBreak[2]) x (0.35 - (0.05 x (Real((Level of Life Break for LifeBreak[2]))))))
Unit - Create 1 Life Break Dummy for (Owner of LifeBreak[2]) at (Position of LifeBreak[2]) facing Default building facing degrees
Set LifeBreak[3] = (Last created unit)
Unit - Add a 5.00 second Generic expiration timer to LifeBreak[3]
Unit - Add Life Break Slow to LifeBreak[3]
Unit - Order LifeBreak[3] to Human Sorceress - Slow LifeBreak[1]
Special Effect - Create a special effect attached to the origin of LifeBreak[1] using Abilities\Spells\Demon\DarkPortal\DarkPortalTarget.mdl
Special Effect - Destroy (Last created special effect)
Unit - Explode LifeBreak[3]
Custom script: set udg_LifeBreak[3] = null
Else - Actions
Custom script: set udg_LifeBreak[2] = null
Custom script: set udg_LifeBreak[1] = null
[/trigger]

At least, nice try =D

 

[Adiktuz]

nice edit Jay, but I would say, just use three different unit variables rather than using an array where you only use three indexes... two different variables are faster and takes less space... and you leak a position... no need to null the globals too...
and maybe reduce the life timer to just about 1 second...


at the spell, so simple...

 

[Jay the Editor]

Yeap. I will prefer to use 3 different variables to differentiate it better.
Owh.. I overlook the position =P
Btw, what d'ya means by null the global? ><

 

[Adiktuz]

you set the unit variables to null, but that isnt really needed since they're globals...

anyway, the spell is too simple so I vote for rejection...

• Life Break
  • Events
    • Unit - A unit Starts the effect of an ability
  • Conditions
    • Life Break Equal to (Ability being cast)
  • Actions
    • Set TempUnit1 = (Target unit of ability being cast)
    • Set TempUnit2 = (Triggering unit)
    • Set TempLoc = Position of TempUnit2
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Life of TempUnit2) Greater than or equal to 50.00
      • Then - Actions
        • Unit - Set life of TempUnit1 to (Life of TempUnit1) x 0.50)
        • Unit - Set life of TempUnit2 to ((Life of TempUnit2) x (0.35 - (0.05 x (Real((Level of Life Break for TempUnit1))))))
        • Unit - Create 1 Life Break Dummy for (Owner of TempUnit1) at (TempLoc) facing Default building facing degrees
        • Set TempUnit2 = (Last created unit)
        • Unit - Add a 1.00 second Generic expiration timer to TempUnit2
        • Unit - Add Life Break Slow to TempUnit2
        • Unit - Order TempUnit2] to Human Sorceress - Slow TempUnit1
        • Special Effect - Create a special effect attached to the origin of TempUnit1 using Abilities\Spells\Demon\DarkPortal\DarkPortalTarget.mdl
        • Special Effect - Destroy (Last created special effect)
        • Else - Actions
      • Custom script: call RemoveLocation(udg_TempLoc)

 

[Jay the Editor]

Adiktuz, Ohh i see.. Thankx
Btw, i vote for rejection too.. there's still a lot of spells in DotA, why not choose a harder one?

 

[Adiktuz]

you can also remove the If-then by moving the condition into the general conditions tab of the trigger...

making the spells is fine but uploading it here is a different issue... if you want feedback, you could post on the trigger and scipts section...

 

[Magtheridon96]

This spell can only be MUI if you use "A unit Starts the effect of an ability"
Trust me.
Nice idea btw
...

The truth: Your spell is too simple to be uploaded here
Try to make something cool
If you want to improve the spell, you could add a lot of configurables (ex: damage, effects, hp payment, knockback boolean, optional knockback trigger, ...)

You should make this spell worthwhile.

Henceforth, i shall give you a 1/5 and vote for rejection I'm sorry...

I gave you some ideas to help, so i hope you can fix this spell, because it would be really awesome with a little work

Good luck

PM me if you need any help with triggering (i don't write others' triggers, but i do help when it comes to leaks, MUI-bility ;p, efficiency, functionality.)

thanks for understanding (-.- unless you hit your computer and broke the screen in anger ;P)

 

[baassee]

This spell can only be MUI if you use "A unit Starts the effect of an ability"
Trust me.

And where did you get this information from? As it's incorrect. It's MUI already.

EDIT:

This spell should be splitted into 2 triggers. One with the current event as a stop trigger. Just checking the hp of the caster and order it to stop if it doesn't apply to it and therefore not wasting cooldown and mana.

The second trigger should be like it is now with Jay's edit (with Adiktuz fixes) and with the event A Unit Starts the Effect of an Ability.

 

[Steel0RaZoR]

I know its too simple, and I'm still learning on how to do spell triggers.

 

[Jay the Editor]

take ur time to learn try to dig up ideas from different maps and combine the ideas.. that will be epic! ^-^