Double Spinning Blade v1.1.5

[Rheiko]

sure

 

[FUJI]

Double Spinning Blade v1.1.5
This spell had been reviewed by jakeZinc.

The triggering does not have any issues have and it is clean.

• Set DSB_Damage_Group = (Units within DSB_AoE of TempPoint)
• Unit Group - Pick every unit in DSB_Damage_Group and do (Actions)

This two actions can be merged into 1 line and it will does not requiring unit group variables.

• Special Effect - Create a special effect attached to the DSB_Special_Effect_Location of TempUnit using DSB_Special_Effect
• Special Effect - Destroy (Last created special effect)

Can be merged into 1 line too but I'll not force you to do this seems it is GUI ^^.

• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
  • (Current order of DSB_Destroyer) Equal to (Order(harvest))
  • Then - Actions
  • Destructible - Kill TempDest
  • Else - Actions

Add a boolean condition: If Picked Destructible is dead equal to FALSE so that it will only pick alive trees not dead.

The documentation in configuration is nice, there's no problem in this ^^.

The configuration seems fine and it is documented.

• Unit - Cause DSB_Caster[DSB_Current_Index] to damage TempUnit, dealing DSB_Damage[DSB_Ability_Level[DSB_Current_Index]] damage of attack type Spells and damage type Normal

The attacktype and the damagetype must be configurable by users.

• -------- Area of Effect of the ability --------
• Set DSB_AoE = 150.00
• -------- How far your blade travels --------
• Set DSB_Distance = 1000.00
• -------- How far your blade spawns --------
• Set DSB_Start_Range = 100.00

Can be supported every levels.

Very Useful spell and having shadow dummy that returns a blade is seems cool.

I don't find anyting bad about the spell.

There's no importing instruction and the the categories of the trigger must be separated.

The special effects is lacking but it is enough in the concept of the spell ^^

Useful from AoS Maps! It is a blade that travels by distance and it's projectile come back when reached a desired distance that really useful.

If the blade comes back, it must follow the position of the caster every loop because it seems odd if the blade comes back in the last saved position of the caster that is easily removed. The dummy must not have a food cost. The dummy must have 0.00 collision size. You can store this into a variable

• (DSB_Distance x 0.03)

but it seems like it is unneeded to store that ^^.

• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
    • DSB_Destroy_Tree Equal to True
  • Then - Actions
    • Set TempPoint = (Center of (Playable map area))
    • Unit - Create 1 Peasant for Neutral Passive at TempPoint facing Default building facing degrees
    • Custom script: call RemoveLocation(udg_TempPoint)
    • Set DSB_Destroyer = (Last created unit)
    • Unit - Hide DSB_Destroyer
  • Else - Actions

You merge this into 3 lines. Before hiding a peasant unit, you must add an ability 'locust' and 'harvest'. Add a follow through time to the shadow dummy before returning the blade. The sound volume can be configurable.

The tooltip supported every level that seems good. Try adding a value of distance in the tooltip levels not only a damage ^^.

Rejected: 1-10
Unacceptable : 11-30
Lacking : 31-49
Useful : 50-69
Recommended : 70-90
Highly Recommended : 91-100
Director Cut : 101-105

1) Triggering - 17/20
2) Documentation - 10/10
3) Importing Instruction - 0/10
4) Special Effects - 12/15
5) Usefulness - 20/20
6) Tooltip - 9/10
7) Configuration - 6/10

Total Score - 69/105
jakeZinc Rating - 3.9/5
Status - Useful

You're near to Recommended status. You must add importing instruction so your spell will enter to the recommended status.

 

[Rheiko]

[There's no importing instruction and the the categories of the trigger must be separated.

there is, check my description

Add a boolean condition: If Picked Destructible is dead equal to FALSE so that it will only pick alive trees not dead.

i will

Can be supported every levels.

i'm too lazy to add it but i will

You merge this into 3 lines. Before hiding a peasant unit, you must add an ability 'locust' and 'harvest'. Add a follow through time to the shadow dummy before returning the blade. The sound volume can be configurable.

okay

The tooltip supported every level that seems good. Try adding a value of distance in the tooltip levels not only a damage ^^.

sure

_{Note: i'm not using mobile, just lazy to use white text at the moment}

 

[FUJI]

there is, check my description

Oh, I see. but you must put it inside in your map ^^.

 

[Rheiko]

not necessarily, because the first time they download the spell is here
for copying and pasting it'll difficult for them because they'll close and open the spell again
they could look at the description when they close the spell
but yes, i'll add them in the next version

 

[Daffa]

jakeZinc
Let me point out something :

You merge this into 3 lines. Before hiding a peasant unit, you must add an ability 'locust' and 'harvest'. Add a follow through time to the shadow dummy before returning the blade. The sound volume can be configurable.

Harvest? Peasant have it by default.

 

[FUJI]

^ Some of the users editing peasant with no harvest ability and some of them remove it. So it is safety ^^.

 

[T. D. W.]

Jake is right, and that's what i am going to do in the next version of KB3D

 

[Rheiko]

alright then, i'll update them when i have time
thanks everyone

 

[Zed]

This looks great, Tested it ingame.
4/5

 

[strongwill]

Oh, and i have to write all that in the trigger editor? Is there a simpler version of this?

 

[Rheiko]

Just copy 'em instead. Then configure them as the description explains.