• 🏆 Texturing Contest #33 is OPEN! Contestants must re-texture a SD unit model found in-game (Warcraft 3 Classic), recreating the unit into a peaceful NPC version. 🔗Click here to enter!
  • 🏆 Hive's 6th HD Modeling Contest: Mechanical is now open! Design and model a mechanical creature, mechanized animal, a futuristic robotic being, or anything else your imagination can tinker with! 📅 Submissions close on June 30, 2024. Don't miss this opportunity to let your creativity shine! Enter now and show us your mechanical masterpiece!🔗 Click here to enter!

Ultra Heal V 1.0

This bundle is marked as useful / simple. Simplicity is bliss, low effort and/or may contain minor bugs.
  • Like
Reactions: Hubcool
Heals a target unit and nearby units but heals the target unit more than the nearby units. For nearby units to heal they must be in a certain range of the priest. My first GUI spell ever so feedback is liked ^.^ but don't try and tear me apart too bad.

Keywords:
Priest, Heal, Ultra, Mega, Health
Contents

Just another Warcraft III map (Map)

Reviews
15:47, 8th Oct 2009 TriggerHappy187: This spell has been here for about a month now with no updates. It leaks as well (and uses the wrong event for casting).

Moderator

M

Moderator

15:47, 8th Oct 2009
TriggerHappy187:

This spell has been here for about a month now with no updates.

It leaks as well (and uses the wrong event for casting).
 
Level 5
Joined
Jul 2, 2005
Messages
60
Hello,
it leaks: Target point of ability being cast -> put in a variable and then removelocation and you need to create a unitgroup variable too ... destroygroup..
and it only works for the casters units, means that you cannot heal allies .... i think it would be better if you change that ! :)

Keep it going!
 
Level 22
Joined
Nov 14, 2008
Messages
3,256
Not bad idea, thought to make it myself into a map but now I don't need it! :)

Well your spell leak first of all but the most common bug is:

1.
When you use channel ability and not have a stop trigger you can abuse this.

Start cahnnel the ability and push the stop button all the time makes you not loosing mana.

2.
  • (Unit-type of (Casting unit)) Equal to Priest
is that line really needed? So I cant give heal ability to other units then priests? ^^

3.
  • Unit Group - Pick every unit in (Units within 600.00 of (Position of (Target unit of ability being cast))) and do (Actions) <------ mayor leak!
You leak in 2 ways here, first of all you leak at the position, second you leak at unit group.

Can be easaliy fixed by doin:

  • Set TempPoint = (Position of (Target unit of ability being cast))
  • Set TempGroup = (Units within 600.00 of TempPoint)
  • Unit Group - Pick every unit in TempGroup and do (Actions)

4. This is not my area of triggering but why you need sp id? really just create the sp and destroy it afterwards, it will take the same time as if you remove after the line you created it

That's all for now, hope it helped :)

~baasse[color=#00ff0]e[/color]
 
Level 17
Joined
Mar 17, 2009
Messages
1,349

Deuterium's Brief Review


1. Don't use:
  • Unit - A unit Begins channeling an ability
Instead use:
  • Unit - A unit Starts the effect of an ability

2. Remove:
  • (Unit-type of (Casting unit)) Equal to Priest

3. Leaks a group:
Unit Group - Pick every unit in (Units within 600.00 of (Position of (Target unit of ability being cast))) and do (Actions)


4. This here is sort of senseless as it doesn't serve any good purpose:
  • Set special_effect_ID = (special_effect_ID + 1)
  • Set special_effect[special_effect_ID] = (Last created special effect)

5. Although using the Wait here isn't much of a problem, if you know how to, avoid using it.


6. Neither MUI nor MPI.



To start off with, spell idea is very simple. However, work on fixing the just mentioned problems.

As a word of advice, use a special effect which you can destroy instantly and still get the effect. That way the spell would become instant (i.e. automatically MUI and wouldn't need the Wait anymore).
 
Top