Fire laser shield v0.0.1

[Moderator]

21:14, 4th Jan 2010
The_Reborn_Devil:

This spell is not MUI nor MPI and it's too simple. Rejected.

 

[X-OMG-X]

Could you post the triggers here? Give the spell a better description. Also you spelled laser wrong.

 

[Thanathos]

Triggers

• dead
  • Events
    • Unit - A unit Dies (Bad)
  • Conditions
    • (Triggering unit) Equal to Caster
  • Actions
    • Set time = 0.00
    • Countdown Timer - Pause timer
    • For each (Integer A) from 1 to 6, do (Actions)
      • Loop - Actions
        • Lightning - Destroy Lighting[(Integer A)]

• cast
  • Events
    • Unit - A unit Starts the effect of an ability (Bad)
  • Conditions
    • (Ability being cast) Equal to Laiser shield
  • Actions
    • Set Caster = (Casting unit)
    • Set TempAngle[(Player number of (Owner of (Triggering unit)))] = 0.00
    • Set Real = 0.00
    • Countdown Timer - Start timer as a Repeating timer that will expire in 0.03 seconds
    • For each (Integer A) from 1 to 6, do (Actions)
      • Loop - Actions
        • Set Real = (Real + 60.00)
        • Set TempPoint1 = (Position of Caster)
        • Set TempPoint2 = (TempPoint1 offset by 200.00 towards Real degrees)
        • Set TempAngle[(Integer A)] = (Angle from TempPoint1 to TempPoint2)
        • Lightning - Create a Finger of Death lightning effect from source TempPoint1 to target TempPoint2
        • Set Lighting[(Integer A)] = (Last created lightning effect)
        • Lightning - Change color of Lighting[(Integer A)] to (1.00 0.00 0.00) with 1.00 alpha
        • Custom script: call RemoveLocation(udg_TempPoint1)
        • Custom script: call RemoveLocation(udg_TempPoint2)

• effect
  • Events
    • Time - timer expires
  • Conditions
  • Actions
    • For each (Integer A) from 1 to 6, do (Actions)
      • Loop - Actions
        • Set TempAngle[(Integer A)] = (TempAngle[(Integer A)] + Speed)
        • Set TempPoint1 = (Position of Caster)
        • Set TempPoint2 = (TempPoint1 offset by 200.00 towards TempAngle[(Integer A)] degrees)
        • Lightning - Move Lighting[(Integer A)] to source TempPoint1 and target TempPoint2
        • Unit - Create 1 Fire for (Owner of Caster) at TempPoint2 facing TempAngle[(Integer A)] degrees
        • Unit - Add a 0.90 second Generic expiration timer to (Last created unit)
        • Unit - Cause (Last created unit) to damage circular area after 0.00 seconds of radius 100.00 at TempPoint2, dealing Damage damage of attack type Spells and damage type Normal
        • Unit - Create 1 smoke for (Owner of Caster) at TempPoint2 facing TempAngle[(Integer A)] degrees
        • Unit - Add a 0.90 second Generic expiration timer to (Last created unit)
        • Custom script: call RemoveLocation(udg_TempPoint1)
        • Custom script: call RemoveLocation(udg_TempPoint2)

• timer
  • Events
    • Time - timer expires
  • Conditions
  • Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • time Less than 25.00
      • Then - Actions
        • Set time = (time + 0.03)
      • Else - Actions
        • Set time = 0.00
        • Countdown Timer - Pause timer
        • For each (Integer A) from 1 to 6, do (Actions)
          • Loop - Actions
            • Lightning - Destroy Lighting[(Integer A)]

• unleak
  • Events
    • Unit - A unit Dies (Bad)
  • Conditions
    • Or - Any (Conditions) are true
      • Conditions
        • (Unit-type of (Triggering unit)) Equal to Fire
        • (Unit-type of (Triggering unit)) Equal to smoke
  • Actions
    • Unit - Remove (Triggering unit) from the game

• learn
  • Events
    • Unit - A unit Learns a skill (Bad)
  • Conditions
    • (Learned Hero Skill) Equal to Laiser shield
  • Actions
    • Set Damage = (Damage + 3.00)
    • Set Speed = (Speed + 1.00)

Well this spell is not MUI and you used a way to much triggers to create it. You can easily merge the effect and timer trigger just by putting the actions of both into one single trigger. The unleak and dead trigger can also easily be merged by moving the actions of one trigger to the other and creating corresponding if statements. I also couldn´t find any documentation on how to import or adjust this spell. You also shouldn´t use Integer-A for your loops since that can screw up sometimes, just create your own integer for that. Using the circular damage function of GUI is a very bad idea since this will damage any unit (yep any unit including the caster, allies and/or magic immune units). But at least you know how to clean leaks. However this also reminds me on a spell i did a while ago...

At this stage i don´t think this is going to be approved.

~TNT

 

[Lambdadelta]

Wow. Nice effects. The spiral effect it leaves for a second when moving is done well.
The only thing you need to work on is the code. It's not MUI as stated already. It's using damage area function, which hurts allies as well. And lots more.

 

[VoidLordXtreme]

wohoo eyecandy!!!
btw rly nice coding

 

[Maker]

Laiser?

 

[YourNameHere]

btw rly nice coding

Are you fucking joking? The coding wasn't good at all, it is not MUI in any way and uses so much triggers which aren't needed at all.

 

[VoidLordXtreme]

Are you fucking joking? The coding wasn't good at all, it is not MUI in any way and uses so much triggers which aren't needed at all.

now as you say it
seems i just rushed over the coding

 

[Datnes]

Can i still improve it and let fix the Rejection rate?