Another Spell Problem.

[lionheadxO]

So in the map I am making, there are no heroes just regular units with the same skill. And I am trying to make dispersion, like in DotA. And so I would guess i need a defensive passive skill like evasion, but the problem is my other units have evasion. And so I'm wondering if there is any other skill we could use. I'm guessing I would use drunken brawler instead. But I have no idea how I'm going to get the dispersion to work.

So this is what I'm trying to do.

15% chance when hit to disperse all damage in an 600 AoE around you.

 

[ertaboy356b]

You need to use Jass on that skill because GUI isn't fast enough... Do you know how to Jass??

 

[lionheadxO]

damn, no I don't. =/

 

[ertaboy356b]

Made this myself.. Don't expect it to be perfect..

function Dispersed takes string EyeCandy, integer Chance returns nothing
local real x = GetWidgetX(GetTriggerUnit())
local real y = GetWidgetY(GetTriggerUnit())
local integer a = 0
local group b = CreateGroup()
local real Damage = GetEventDamage()
local unit DamageTaker
local effect FX
if GetRandomInt(1,100) <= Chance then
call GroupEnumUnitsInRange(b,x,y,600,null)
call SetUnitState(GetTriggerUnit(),UNIT_STATE_LIFE,GetUnitState(GetTriggerUnit(),UNIT_STATE_LIFE) + Damage)
set FX = AddSpecialEffectTarget(EyeCandy,GetTriggerUnit(),"chest")
loop
if GetEnumUnit() != GetTriggerUnit() and IsUnitAlly(GetEnumUnit(),GetOwningPlayer(GetTriggerUnit())) then
set DamageTaker = FirstOfGroup(b)
exitwhen DamageTaker == null  
call UnitDamageTarget(GetTriggerUnit(),DamageTaker,Damage,true,false,ATTACK_TYPE_CHAOS,DAMAGE_TYPE_NORMAL,WEAPON_TYPE_AXE_MEDIUM_CHOP)
call GroupRemoveUnit(b,DamageTaker)
endif
endloop
call DestroyGroup(b)
set b = null
set DamageTaker = null
call TriggerSleepAction(1.00)
call DestroyEffect(FX)
set FX = null
endif
endfunction

Import this on the predefined script (The very top in the triggers)..

Then make a trigger.. Add this as an action:

Custom Script: call Dispersed(<The Model Path of the Effect>, <The Chance>)
The Model of the Effect is the model path.. example is model/model/model.mdl something like that... Always remember that when doing Jass, paths have 2 "\" instead of one.... So the path looks like this "model\\model\\model.mdl"...
The Chance is the chance of happening.. you may want to add a variable into this.. Then insert it as this "udg_variablename"...

When done, it should looks like this

Events - Unit takes damage
Condition - Level of dispersed to triggering unit is greater than 0
Actions:
    Set Chance = 30
    Custom Script: call Dispersed("model\\model\\model.mdl",udg_Chance)

----------------------------------------------------------------------------------------
Edit: Updated the code.... a condition was added so that it wouldn't deal damage to friendly units...
Some text are altered..

 

[Pyrogasm]

Your slashes are going the wrong way. They should be like this: "\\". If you don't add the double slashes, the game will crash upon saving; just a warning.


On another note, ertaboy356b's code will not work; you need to pass the triggering unit as an argument like so:

function Dispersed takes unit u, string EyeCandy, integer Chance returns nothing
    local real x = GetWidgetX(u)
    local real y = GetWidgetY(u)
    local integer a = 0
    local group b = CreateGroup()
    local real Damage = GetEventDamage()
    local unit DamageTaker
    local effect FX

    if GetRandomInt(1,100) <= Chance then
        call GroupEnumUnitsInRange(b,x,y,600,null)
        call SetUnitState(GetTriggerUnit(),UNIT_STATE_LIFE,GetWidgetLife(u) + Damage)
        set FX = AddSpecialEffectTarget(EyeCandy,u,"chest")
        loop
            if GetEnumUnit() != u then
                set DamageTaker = FirstOfGroup(b)
                exitwhen DamageTaker == null  
                call UnitDamageTarget(u,DamageTaker,Damage,true,false,ATTACK_TYPE_CHAOS,DAMAGE_TYPE_NORMAL,WEAPON_TYPE_AXE_MEDIUM_CHOP)
                call GroupRemoveUnit(b,DamageTaker)
            endif
        endloop
        call DestroyGroup(b)

        set u = null
        set b = null
        set DamageTaker = null
        call TriggerSleepAction(1.00)
        call DestroyEffect(FX)
        set FX = null
    endif
endfunction

And you would now call it like this:

Events
    Unit - Unit takes damage
Conditions
    (Level of Dispersed for (Triggering unit)) greater than 0
Actions
    Set Chance = 30
    Custom Script: call Dispersed(GetTriggerUnit(),"model\\model\\model.mdl",udg_Chance)

 

[ertaboy356b]

Ok ^_^... But I used triggering unit upon testing.. that's because the event is a unit takes damage.... it worked for me.. oh yeah.. I might fixed it...

 

[SerraAvenger]

I am pretty sure you do not have to null all your locals - shouldn't they be destroyed automaticallyat the end of the foo?

 

[ertaboy356b]

I'm just playing safe.... People are doing it...

 

[AceHart]

Locals are destroyed when the function ends, which is actually the point of using them.
However, you still need to tell the game that you also don't need whatever they pointed to anymore...

 

[SerraAvenger]

Locals are destroyed when the function ends, which is actually the point of using them.
However, you still need to tell the game that you also don't need whatever they pointed to anymore...

...wich I do in a every language by destroying them, not by loosing all pointers to it...
With a garbage collector I could null all other variables that point to it, when I destroy the last pointer to it would be destroyed...

And if the vars are destroyed anyway, why null them then?


omg another offtopic discussion started by me

 

[Evote]

Sorry i carn't see the problem

 

[worldofDeath]

Sorry i carn't see the problem

If you dont see the problem and cant help dont post that you cant see the problem, your trying to help not tell some one you have no clue what there talking about...