Rain Arrow 1.1 [GUI][MUI][Leakless]

[Moderator]

12th Dec 2015
IcemanBo: Too long as NeedsFix. Rejected.

Moderator: Pharaoh_ Date: 15:45, 19th May 2012 The spell is very simple, it isn't not coded within a loop, when it can clearly be done this way, has no configurables and uses wait. Arrays can be replaced with normal variables. Rejected for now

 

[Tank-Commander]

Triggers

• Rain Arrow
  • Events
    • Unit - A unit Starts the effect of an ability
  • Conditions
    • (Ability being cast) Equal to Rain of Arrow
  • Actions
    • Set RainArrowReal = 0.00
    • Set RainArrowPoint[1] = (Target point of ability being cast)
    • For each (Integer A) from 1 to 20, do (Actions)
      • Loop - Actions
        • Set RainArrowPoint[2] = (RainArrowPoint[1] offset by 125.00 towards RainArrowReal degrees)
        • Set RainArrowPoint[3] = (RainArrowPoint[1] offset by 250.00 towards RainArrowReal degrees)
        • Set RainArrowPoint[4] = (RainArrowPoint[1] offset by 375.00 towards RainArrowReal degrees)
        • Set RainArrowPoint[5] = (RainArrowPoint[1] offset by 500.00 towards RainArrowReal degrees)
        • Unit - Create 1 Small Arrow Dummy for (Owner of (Triggering unit)) at RainArrowPoint[2] facing Default building facing degrees
        • Animation - Change (Last created unit) flying height to 0.00 at 1000.00
        • Unit - Add a 1.00 second Generic expiration timer to (Last created unit)
        • Unit - Create 1 Small Arrow Dummy for (Owner of (Triggering unit)) at RainArrowPoint[3] facing Default building facing degrees
        • Animation - Change (Last created unit) flying height to 0.00 at 1000.00
        • Unit - Add a 1.00 second Generic expiration timer to (Last created unit)
        • Unit - Create 1 Small Arrow Dummy for (Owner of (Triggering unit)) at RainArrowPoint[4] facing Default building facing degrees
        • Animation - Change (Last created unit) flying height to 0.00 at 1000.00
        • Unit - Add a 1.00 second Generic expiration timer to (Last created unit)
        • Unit - Create 1 Small Arrow Dummy for (Owner of (Triggering unit)) at RainArrowPoint[5] facing Default building facing degrees
        • Animation - Change (Last created unit) flying height to 0.00 at 1000.00
        • Unit - Add a 1.00 second Generic expiration timer to (Last created unit)
        • Set RainArrowReal = (RainArrowReal + 24.00)
        • Custom script: call RemoveLocation (udg_RainArrowPoint[2])
        • Custom script: call RemoveLocation (udg_RainArrowPoint[3])
        • Custom script: call RemoveLocation (udg_RainArrowPoint[4])
        • Custom script: call RemoveLocation (udg_RainArrowPoint[5])
    • Custom script: set bj_wantDestroyGroup = true
    • Unit Group - Pick every unit in (Units within 500.00 of RainArrowPoint[1] matching (((Triggering unit) belongs to an enemy of (Owner of (Matching unit))) Equal to True)) and do (Actions)
      • Loop - Actions
        • Unit - Cause (Triggering unit) to damage (Picked unit), dealing (((1.00 x (Real((Level of Rain of Arrow for (Triggering unit))))) + 1.60) x (Real((Agility of (Triggering unit) (Include bonuses))))) damage of attack type Spells and damage type Normal
    • Custom script: call RemoveLocation (udg_RainArrowPoint[1])

Really far too simple to actually be approved, also needs a proper description, and should have these triggers provided in the description as well as have a proper tooltip and not cotain happy faces in it.

List of things:
- Too simple
- Needless SFX spam, it's basically warstomp without any other effects
- No configurables
- One bulk sum of damage which is dealt the instant the spell is cast
- Damages Magic immunes and picks dead units
- Damages structures (Could be intentional?)
- All Sfx could be created with proper looping as opposed to needing 5 locations (could just have 2)
- To make the circle you could use some easy math as opposed to using a real number: Very simply Real = Real + (360 / Number) Also your RainArrowAngle should be increased by 18 in this case, but you set it to 24 eitherway (360/20 = 18)
- Should be given a proper mana cost

That's it for now.

 

[.OmG.]

As I see in screen-shot spell is not bad, however you should improve all what T-C said.

 

[Almia]

Yes,Improve it like Tank-Commander said.ALso Put some Configurables(like damage or something)Just make a spell with (im sorry)Lots of Configurables(so the spell could be useful)

 

[Draaien]

Review
Rain Arrow 1.1
Rating:3.5/5
Comment:Spell is to simple,and you are using waits.I will give you 3.5/5 only because it looks cool :|

 

[Tank-Commander]

By "All damage is done in one bulk" I didn't mean to put a wait in there. . .

 

[Draaien]

look at the moderator`s review...

 

[BloodSoul]

• Set RainArrowinterge = (Player number of (Owner of (Triggering unit)))

What's the point?

 

[gorillabull]

U can do this with cluster rockets

 

[Almia]

My stomach hurts from laughing when he used a variable like this:
[trigger=] Set RainArrowinterge = (Player number of (Owner of (Triggering unit)))[/trigger]
Yeah your right dude gorilla bull