Fatal Sword GUI v1.2

[Moderator]

12th Dec 2015
IcemanBo: For long time as NeedsFix. Rejected.

15:02, 5th Jan 2015
Maker: Avoid using Pause unit action in spells.

 

[GywGod133]

EDIT:
You should change this Set FS_Caster[FS_Temp] = (Casting unit) to Set FS_Caster[FS_Temp] = (Triggering unit).

This Set FS_CastRange[FS_Temp] = (Distance between (Position of FS_Caster[FS_Temp]) and (Position of FS_Target[FS_Temp])), you have 2 leaks

In your Trigger Loop, Set FS_Move[FS_Temp] = ((Position of FS_Caster[FS_Temp]) offset by FS_Speed towards (Facing of FS_Caster[FS_Temp]) degrees) you have leak

EDIT 2:
This Trigger - Turn on FS Loop <gen> put it in ITE (IF THEN ELSE)

You need to change this Trigger - Turn off FS Loop <gen> into Trigger - Turn off this trigger and also put it in a ITE

 

[fareezizfar]

UPDATED!!

Thanks for that GywGod133..

 

[deathismyfriend]

Use triggering unit. Never use casting unit. Casting unit is slower and less efficient.

This is currently not MUI and will only work when one unit is casting.

You have the indexing part which is correct. But you never loop through all the indexes in the loop trigger and you need to de-index when an instance of the spell is over.

If you do anything with level of ability it should always be stored. Otherwise you can have someone cast the ability then level it and now the spell that was cast by them now gets the damage that the new level of the ability does.

The range should also be configurable.

 

[GywGod133]

Said deathismyfriend

You need to learn Indexing

Check this https://www.hiveworkshop.com/forums...orials-279/visualize-dynamic-indexing-241896/

 

[Chaosy]

(100.00 x (Real((Level of FS_Ability for FS_Caster[FS_Temp]))))

you should use your variable and not 100

also, you turn on the loop even when it's not needed.

Not to mention that it isn't MUI

 

[deathismyfriend]

You can look at my tutorial things you'd should know when using trigger / gui. The chapter called how to index will show you how to properly index / de-index your spell.

Sorry I forgot to mention it in my above post.

 

[Meatmuffin]

Make it MUI. You don't use the same array integer when you move your variables from init to loop. And at the start of the trigger use this:

• For each (Integer Your_New_Integer_Variable) from 1 to FS_Temp, do (Actions)

There are still a lot of things for you to learn, please check some guides about MUI spells.

Edit: Btw, looks nothing like Zoro's Oni Giri. :/