Spellpack Undying

~GaLs~

† Ғσſ ŧħə ѕαĸε Φƒ ~Ğ䣚~ †
Reaction score
180
>>Yeah, nvm.
It is not that I copy anyone else's idea. I am just following what DotA(Ice Frog) does.
 

NetherHawk

New Member
Reaction score
26
nvm ahaha =/

i wanted to learn how to use structs from your map... but it seems u use ABC, which i dunno also... argh =/
 

~GaLs~

† Ғσſ ŧħə ѕαĸε Φƒ ~Ğ䣚~ †
Reaction score
180
**Bump** :p

>>i wanted to learn how to use structs from your map... but it seems u use ABC, which i dunno also... argh =/
They are just like handle vars.
In my situnation, my struct act like a large wrapper which wrap up every variable.

It is my routine to bump this now..
 

Cohadar

master of fugue
Reaction score
209
I think this should not be approved until you make raise dead as well
 

~GaLs~

† Ғσſ ŧħə ѕαĸε Φƒ ~Ğ䣚~ †
Reaction score
180
>>I think this should not be approved until you make raise dead as well
I didn't said I wanted to submit a DotA hero.
I just wanted to submit spell pack, why does it have matter to the Raise Dead?
 

Cohadar

master of fugue
Reaction score
209
Because it is natural for it to be there, and because you need to learn to finish what you started (education and stuff...)

EDIT:
JASS:
private function tAct takes nothing returns nothing
local timer t = GetExpiredTimer()
local Soul_Rip SR = GetTimerStructA(t)

    call ClearTimerStructA(t)
    call SR.destroy()
    
call DestroyTimer(t)
set t = null
endfunction


JASS:
private function tAct takes nothing returns nothing
local timer t = GetExpiredTimer()
local Soul_Rip SR = ClearTimerStructA(t) // <---- small trick

    call SR.destroy()
    
call DestroyTimer(t)
set t = null
endfunction


JASS:
    method onDestroy takes nothing returns nothing
    set .caster = null
    set .target = null
    call DestroyTrigger(.Infect)
    call DestroyTrigger(.Amplify)
    call DestroyGroup(.reg)
    set .Infect = null
    set .Amplify = null
    set .reg = null
    endmethod


JASS:
    method onDestroy takes nothing returns nothing
    //set .caster = null   // Don't null struct members they are not locals.
    //set .target = null
    call DestroyTrigger(.Infect)
    call DestroyTrigger(.Amplify)
    call DestroyGroup(.reg)
    //set .Infect = null
    //set .Amplify = null
    //set .reg = null
    endmethod


Disabled versions of some of your icons are not showing properly.
Do you know why is this?

EDIT:
You are also using constant functions for ability codes instead of constant globals.
functions in spells are just for stuff that have level as parameter, for example:
JASS:
function AOE takes integer level returns real
    return 100. + level*200.
endfunction
 

~GaLs~

† Ғσſ ŧħə ѕαĸε Φƒ ~Ğ䣚~ †
Reaction score
180
>>@Cohadar
I already finished what you said currently, but I won't update now. I am making Raise Dead.

>>You don't need to trigger Raise Dead, base it off Summon Lava Spawn and change units and art.
Hmm, right. I am currently base off that spell. But, how do you do about the Heal 50 hitpoints thing?
I don't see there is an option for it, or am I blinded?
 

Cohadar

master of fugue
Reaction score
209
I stopped playing dota before that hero was added,
but if you describe what that rise dead does,
I am sure I could give a couple hints...
 

Somatic

You can change this now in User CP.
Reaction score
84
Just detect when the unit type of summoned unit died, create a unit to use fan of knive, with negative damage (for the healing) and cast it targeting own player unit.
 

Cohadar

master of fugue
Reaction score
209
Ok here is the PUI part:
JASS:
struct ZombieData
    unit hero // owner of zombie
    unit zombie
    integer counter // for attack six times thing
    private static ZombieData array PUI
    
    static method create takes unit hero, unit zombie returns ZombieData
        local ZombieData ret = ZombieData.allocate()
        set ret.hero = hero
        set ret.zombie = zombie
        set ret.counter = 0
        set ZombieData.PUI[GetUnitIndex(zombie)] = ret // zombie -> ZombieData link
        return ret
    endmethod
    
    static method Get takes unit zombie returns ZombieData
        return ZombieData.PUI[GetUnitIndex(zombie)]
    endmethod
    
    method onDestroy takes nothing returns nothing
        set ZombieData.PUI[GetUnitIndex(.zombie)] = 0 // break the link
    endmethod
endstruct


You register on attack trigger and if attacking unit is zombie you increase it's counter. if counter gets to 6 you raise a new zombie.

How do you keep track of number of zombies?
Well I will leave that as a homework ;)

EDIT: fixed some typos in code.
 

~GaLs~

† Ғσſ ŧħə ѕαĸε Φƒ ~Ğ䣚~ †
Reaction score
180
>>You register on attack trigger and if attacking unit is zombie you increase it's counter. if counter gets to 6 you raise a new zombie.
Sorry bro.
In DotA, it is registered in EVENT_UNIT_DAMAGED. The "S" bug won't work with this spell. So... I think that in DotA, this spell was base on Lava Spawn. (IceFrog loves to use original spell of Blizzard, right? ;))

The problem was the healing 50 hp thingy...
 

~GaLs~

† Ғσſ ŧħə ѕαĸε Φƒ ~Ğ䣚~ †
Reaction score
180
>>Irrelevant, you steel need ZombieData.
-.-lll
There is a missle effect that moving towards the Hero when a zombie die.
The +50 hp effect will only happen when the missle reach the Hero.


And, in the other hand... I need 3 trigger for 1 spell again..?
1. Your PUI. (PUI is an insult in chinese you know?)
2. Your ZombieData (You said everything need 1 trigger right?)
3. Main Trigger.

>>Simple as that.
You think so?
 

Cohadar

master of fugue
Reaction score
209
ZombieData is a struct not a trigger ffs, just put it in spells trigger.
Heal effect is a dummy caster death-coil or something like that.
So when zombie dies use dummy caster to cast 50 heal on hero.

I said every scope should have it's own trigger.


EDIT:
I just looked at lava spawn spell.
You were right, that is how IceFrog did it, the only problem is that if you use lava spawn ability spell will not be MUI (it will be only multi-player instancable)

So you basically need to do this:

Forget about ZombieData

Base Raise dead on summon lava spawns

JASS:
globals
    unit array Dirges  // connects player <-> dirge
endglobals


When Dirge casts a summon spell you do:
JASS:
function RaiseDeadActions takes nothing returns nothing
    set Dirges[GetPlayerId(GetOwningPlayer(GetTriggerUnit()))] = GetTriggerUnit()
endfunction


And when spawn dies:
JASS:
function ZombieDiesActions takes nothing returns nothing
    local unit zombie = GetTriggerUnit
    local unit hero = Dirges[GetPlayerId(GetOwningPlayer(zombie))]
    local real x = GetUnitX(zombie)
    local real y = GetUnitY(zombie)

    // Cast some heal magic on hero from the position of dying unit
endfunction
 

emjlr3

Change can be a good thing
Reaction score
396
not bad, a few things however

Soul Rip:

  • should use a timer stack, CSSafety is common for people
  • could use a global group, and remove the need of creating and destroying/nulling each time
  • no need to make comparisons, just add a not in front of it
  • the group can grab the target itself....
  • do not need to null lightning varaible on destroying the struct
  • damage/unit should be level dependant
  • rest is good
Hearstopper

  • could use a global group, and remove the need of creating and destroying/nulling each time
  • rest is good

Plague:

  • infect range and duration should be level dependant
  • timer stack needed
  • could again use global groups
  • dont need to null structs variables
  • rest looks good

anyhow, needs some updates IMO

good job though
 

duyen

New Member
Reaction score
214
Ah, forgot about the healing while dead, but just create a dummy, give it an expiration timer and order it to cast death coil on Undying.
 

~GaLs~

† Ғσſ ŧħə ѕαĸε Φƒ ~Ğ䣚~ †
Reaction score
180
I have gone to some sort of camp, sorry for late reply.. I just came back today.

>>@Cohadar
Hmm, I see.

>>should use a timer stack, CSSafety is common for people
I used it before, but ABC gives alot debug messages to me.
After I change from NewTimer to CreateTimer, this fixed the debug warning.

>>could use a global group, and remove the need of creating and destroying/nulling each time
I see.

>>no need to make comparisons, just add a not in front of it
Hmm? not?
Using not in where..?

>>the group can grab the target itself....
Argh, I have forgoten in DotA, it has no including the caster itself.

>>do not need to null lightning varaible on destroying the struct
I see.

>>damage/unit should be level dependant
I'll see what I can do about it.
Actually, what is dependant means? I see boolean option in ObjectEditor says something like Independance or something alike.

>> * infect range and duration should be level dependant
* timer stack needed
* could again use global groups
* dont need to null structs variables
* rest looks good

oK!

>>Ah, forgot about the healing while dead, but just create a dummy, give it an expiration timer and order it to cast death coil on Undying.
The problem is... You don't know who is the caster of the zombie when the zombie dies.
 
General chit-chat
Help Users
  • No one is chatting at the moment.
  • jonas jonas:
    That sounds like fun!
    +1
  • The Helper The Helper:
    it was a blast!
  • The Helper The Helper:
    I am going to post the Youtube of the investigation in the forums when it is ready
    +1
  • jonas jonas:
    cool!
  • vypur85 vypur85:
    Sounds cool TH.
  • tom_mai78101 tom_mai78101:
    I was on a Legend of Zelda marathon...
  • tom_mai78101 tom_mai78101:
    Am still doing it now
    +1
  • jonas jonas:
    which one(s) are you playing?
  • jonas jonas:
    I played a little bit of the switch title two weeks ago and found it quite boring
  • The Helper The Helper:
    just got back from San Antonio this weekend had the best Buffalo Chicken Cheesesteak sandwhich in Universal City, TX - place was called Yous Guys freaking awesome! Hope everyone had a fantastic weekend!
    +1
  • The Helper The Helper:
    Happy Tuesday!
  • The Helper The Helper:
    We have been getting crazy numbers reported by the forum of people online the bots are going crazy on us I think it is AI training bots going at it at least that is what it looks like to me.
  • The Helper The Helper:
    Most legit traffic is tracked on multiple Analytics and we have Cloud Flare setup to block a ton of stuff but still there is large amount of bots that seem to escape detection and show up in the user list of the forum. I have been watching this bullshit for a year and still cannot figure it out it is drving me crazy lol.
    +1
  • Ghan Ghan:
    Beep boop
    +1
  • The Helper The Helper:
    hears robot sounds while 250 bots are on the forum lol
  • The Helper The Helper:
    Happy Saturday!
    +1
  • The Helper The Helper:
    and then it was Thursday...
    +2
  • tom_mai78101 tom_mai78101:
    And then Monday
    +1
  • The Helper The Helper:
    I got the day off today!
    +1
  • tom_mai78101 tom_mai78101:
    How...? (T-T)
  • The Helper The Helper:
    I took the day off. I work for myself so I can do that.
    +1
  • Varine Varine:
    Well I'm already over summer
  • jonas jonas:
    varine! good to see you
  • jonas jonas:
    what's going on, what's got you going
  • The Helper The Helper:
    good to see you varine hope you are well my friend

    The Helper Discord

    Members online

    No members online now.

    Affiliates

    Hive Workshop NUON Dome World Editor Tutorials

    Network Sponsors

    Apex Steel Pipe - Buys and sells Steel Pipe.
    Top