Spellpack Tauren Chieftain, Demon Hunter and Warden

--Thanatos--

New Member
Reaction score
33
Just made this from about a month ago, but forgive me I couldn't give you any ScreenShots. Just play the game and see it by yourself. This, is JESP (check it, please? I couldn't categorize it IMO).

Tauren Chieftain:
- Shockrush - Knockback ability, which if hit something, deals damage
- Blood for Blood - Deals damage back to nearby enemies when the buff is gone
- Raging Power - Deals damage back to attackers
- Demicalypse (Ultimate) - Slams the ground multiple times, creates random explosions everywhere. Channeling
i704791_Demicalypse.jpg


Demon Hunter:
- Chain Flame - Chaining Spell with Missile Art, and Speed above 522.
- Collapse of Power - Channeling Spell that heals yourself and damages nearby enemies
i704789_CollapseofPower.jpg

- Fake Up - Have a chance to move to the back of the attacker when attacked
- Illusions' Triumph (Ultimate) - Clones
i704790_IllusionsTriumph.jpg


Warden:
- Phantasmal Strike - As long as the buff remains, each time the target takes damage, it will take additional damage
- Magical Knives - Throw magical knives to the target position
i704792_MagicalKnives.jpg

- Vengeance Aura - Whenever an allied unit with the aura takes damage, deal damage back to the attacker equal to the (%) of the damage taken
- Blink Slices (Ultimate) - Strike the target position from different directions, 7 times.

Please post feedbacks. :p

Updates:
Removed da stupid Double Pause for Shockrush
Changed the activation of Raging Power
Better looks for Collapse of Power
Removed the real freaking weird (I still can't believe I wrote that) GetUnitAbilityLevelSwapped
 

Attachments

  • [SpellPack] Demon Hunter, Tauren Chieftain, and Warden.w3x
    158.9 KB · Views: 955

Hatebreeder

So many apples
Reaction score
381
Hey this is cool ^^ But, i dont like the demicalypse. It looks very weird. But thats my opinion. Everything else is very well made, good job ^^
 

--Thanatos--

New Member
Reaction score
33
Thanks! ;)

Sorry for no screenshots, it's freaking hard to shot one. I will continue and make some more heroes.

Yes, IMO the Demicalypse looks so weird to see those flying Trees. :banghead:
I must upgrade it again.
 

emjlr3

Change can be a good thing
Reaction score
395
i cannot believei you coded all that stuff how you did, strange stuff

(I am just looking through the .j)

Shockrush

call Shockrush_Wait (0.271828)

rofl, just use a polledwait, its really not a big deal, and just wait .2 or whatever, no need for oyu to have custom polled wait fucntions on all your spells

no need to pause timers twice

using variable names that make sense would make things infinitly easier to read

is all that checking of pathing really necessary?

rest works, no leaks ro mega uneeded stuff, and I assume all your formulas work as they should

Blood for Blood

fine, with same pet pieves as before

Raging Power

when you set up your two triggers, you loop twice, when you only need to once

everytime the units are attacked of spelled, it does some shit, that it only needs to actually do once, on the first attack or spell, after then the unit just need sto take damage

local integer a = GetUnitAbilityLevelSwapped (b,y) :(

Demicalypse

fine

and I am tired, I will leave someone else to look at the next hero ;=)
 

--Thanatos--

New Member
Reaction score
33
>local integer a = GetUnitAbilityLevelSwapped (b,y) :(
haha

For Raging Power, I've changed em. Almost 100% different now... (Haven't updated there yet, though, will update it ASAP)

>no need to pause timers twice
How in the hell I made that? :banghead: :banghead: :banghead: Updated

>call Shockrush_Wait (0.271828)
I though Blizzard's PolledWait leaks without set t = null ?

>using variable names that make sense would make things infinitly easier to read
BTW I couldn't even understand my own code this way. I don't know what in the freaking hell I
should change it to be. But will try ASAP.

Ty very much, though. :D

BTW where should I put those SpellPack Codes? (i.e. Wait, CheckPath (Vex), H2i,H2G,etc.) ?
 

emjlr3

Change can be a good thing
Reaction score
395
if you put all your common functions in the header in a place that you want ppl to paste over, that would be better IMO

as far as avariable names, stuff like "angle" or "ang" when fering to an angle, as opposed to some arbitrary letter, like "y", is a lot better
 

Sim

Forum Administrator
Staff member
Reaction score
534
> I though Blizzard's PolledWait leaks without set t = null ?

Recreating a whole function just for 1 missing nulling? Nah, remove that function. Variables need to be nulled, to a certain extent.

--------------------------------------

Nice spells, very original. [I love them.

Why do you add a "HV" prefix in the Handle Vars? Everyone simply use the handle vars. For better spell sharing, please remove it.

Just for the sake of it... Your code is really confusing. indenting is good, but there's always a limit.

--------------------------------------

--> Shockrush

Pretty funny actually to see the unit when it crosses a tree. It bugs out. Make it so it destroys trees, otherwise it looks bad :p

> Shockrush_H2U

Local Handle vars already take care of that. Just set a local unit variable to the returned handle value, directly.

Maybe it's just me, but it never damaged a unit in the way. Confirm?

Nice spell, a bit unoriginal though. Knockback has already been made a lot of time.

--> Blood for Blood

The description is... not descriptive. I read it twice but needed to try it out before I could clearly see what it did.

Remove the custom wait.

Nice spell, well done. Unfortunately, it has been done before by IKilledKEnny ;) A bit different though, because it only damaged one target. So... It's ok.

--> Raging Power

Nice ability.

--> Demicalypse

Wait...

You never destroy the group n

Impressive spell.

------------------------------

--> Chain Flame

Wait...

This spell shouldn't be able to come back on targets that got already hit once by the spell, just like chain lightning. At least I think.

Why a delay before the missile is thrown?

Good job.

--> Collapse of Power

Impressive.

Remove the wait, and that's about it.

The event e needs to be nulled.

--> Fake Up

There should be a way to turn that off. It is really annoying when you are, for example, trying to cast a spell and it procs.

--> Illusions' Triumph

Original :p

You should mention that the illusions don't last more than 3 or 4 seconds.

Otherwise, neat.
 

--Thanatos--

New Member
Reaction score
33
>Nice spells, very original. [I love them.
Ty!:D

>This spell shouldn't be able to come back on targets that got already hit once by the spell
It's toggle-able now :D

>There should be a way to turn that off. It is really annoying when you are, for example, trying to cast a spell and it procs
will change it ASAP. It's so annoying I'm testing the Collapse of Power and they attacked and I got "blinked" out and stop channeling.

Also for Shockrush, yes it never deal damage to units in the way.
 
G

gonecase

Guest
Is it possible for u to make Fake up avtive or toggle-able spell that would be awesome u know. Im useing that spell for a hero on my map. Crediting you!!! :) :)
 

--Thanatos--

New Member
Reaction score
33
Thanks for using it gonecase. :)

If you download the map again, it has the Toggle-able version of Fake Up. (based off Immolation, no mana)
 

Sim

Forum Administrator
Staff member
Reaction score
534
Chain Flame has no specific targets allowed. I casted it on my Warden and it damaged her :D

What's the Warden doing in there?

Blood for Blood's description is poor. Try to better explain what the spell does. For instance, you're telling us the spell deals the damage that the Tauren Chieftain did during the time he had the buff. Which is false.

Illusions' Triumph's illusions should last longer. They vanish like 2 seconds after they pop.

Otherwise... Great! Just fix that.
 

--Thanatos--

New Member
Reaction score
33
I Attached a wrong map. :D :D :D :D :D The Warden shouldn't be there.

>I used the Chain Flame and it hits the Warden :D
haha. roflpzomG! :D changed.

Typo for the Blood for Blood fixed. :)

I'm going to change the Illusions' Triumph into about 6 seconds. :D

BTW, the Warden is going to be finished soon. May it be on this map? If yes, please, change the Topic name. :) to "Tauren Chieftain, Demon Hunter, and Warden".

Edit: I mean "changed" when I last update it again. :p
 

Sim

Forum Administrator
Staff member
Reaction score
534
Sure.

Don't forget to add clear descriptions ;)
 

--Thanatos--

New Member
Reaction score
33
Ok... Now I got everything done. However, there's one thing I must ask...
I don't know how to post big screenies :D . Any help?
 

--Thanatos--

New Member
Reaction score
33
Okay. Finished! :D

Last update, may it be. (If there's no more bugg and leaks)
And finally Screenies, for some Abilities.

And, change the Topic please. :D
 
General chit-chat
Help Users
  • No one is chatting at the moment.

      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