Sonic Missile v1.0

[Moderator]

16:11, 20th Sep 2014
TriggerHappy:

The spell works and the script looks fine.

Though, for the interval I think 0.03 looks a lot better than 0.05.

 

[GywGod133]

Set SM_OwnerofUnit = (Owner of SM_Caster[SM_Index])
change it into this
Set SM_OwnerofUnit = Triggering Player

other is good coding

 

[mogulkhan1Axe]

so that doesnt matter cuz its not array right? and what is the difference?

 

[GywGod133]

so that doesnt matter cuz its not array right?

yep... triggering player is enough

 

[mogulkhan1Axe]

okay thanks for feedback i updated it

 

[GywGod133]

Maybe you can do the one variable for temp points
just use array

for example
Set temppoint[1] = (Position of SM_Caster[SM_Index])
call RemoveLocation( udg_tempoint[1])

instead using many variables

but its only a suggestion
for me its a good way to decrease the numbers of variables

 

[mogulkhan1Axe]

when i make that variable temppoint[blabla] should it be default array size 3 or 1 ?

 

[GywGod133]

when i make that variable temppoint[blabla] should it be default array size 3 or 1 ?

You can do create a many temppoints[1, 2, 3, 4, 5, etc (Maximum 8176)]

 

[mogulkhan1Axe]

What i meant

image

Because i only use 3 points

 

[GywGod133]

What i meant

image

Because i only use 3 points

oh OKAY

use 1 array

 

[mogulkhan1Axe]

okay

 

[Vinz]

If you make it array put an s. TempPoints

There are maps that use TempPoint as a non array. It would also make more sense.

Also btw move the RemoveLocations in the loop outside the inner ITE since you do it twice either way.

 

[mogulkhan1Axe]

If you make it array put an s. TempPoints

There are maps that use TempPoint as a non array. It would also make more sense.

Damn i was just about to update thanks for the heads up

 

[mogulkhan1Axe]

You can change the speed by changing the variable ''SM_DistanceTraveled'' in the Variable trigger

 

[Vinz]

I was slow to update the post. :<

Anyway, decent spell. 4/5

 

[mogulkhan1Axe]

I was slow to update the post. :<

Anyway, decent spell. 4/5

Thanks

 

[deathismyfriend]

You can do create a many temppoints[1, 2, 3, 4, 5, etc (Maximum 8176)]

Max array size is 8191 not 8176.

@mogulkhan1Axe
You should take a look at the variable editor section in my tutorial http://www.hiveworkshop.com/forums/...now-when-using-triggers-gui-233242/#Section 1. (this link will bring you directly to that section. It has info on why variables should be 1 and what need to be higher and the reason for that.

 

[mogulkhan1Axe]

Max array size is 8191 not 8176.

@mogulkhan1Axe
You should take a look at the variable editor section in my tutorial http://www.hiveworkshop.com/forums/...now-when-using-triggers-gui-233242/#Section 1. (this link will bring you directly to that section. It has info on why variables should be 1 and what need to be higher and the reason for that.

Thanks for the reply as always !

image

look at this how it should be i mean AKA.GywGod133 already said that only array size 1 is needed

 

[deathismyfriend]

The one on the left. When you can you should read my tutorial.

 

[Vinz]

Those array specific amounts are just for a few var types.

 

[mogulkhan1Axe]

Those array specific amounts are just for a few var types.

ahhh okay

 

[Kazeon]

• SM_DistanceTraveled should be SM_AoE or SM_HitRadius
  
• Unit facing smoothing factor seems better for homing missile: Increase/decrease missile's facing per tick. Not instantly order the missile to face it's target.
  
• 
  
  • Set SM_Level[SM_Index] = (Level of SM_Ability for SM_Caster[SM_Index])
  Remove that variable, you are only using it for once.
  
  • Set SM_Damage[SM_Index] = SM_InitialDamage[SM_Level[SM_Index]]
• Optional:
  
  • (SM_Target[tempInt] is alive) Equal to True
  It seems weird if the missile gets removed instantly after the target has died. At least, let them arrive at target's location first. Check wc3 native abilities for example.

 

[mogulkhan1Axe]

• SM_DistanceTraveled should be SM_AoE or SM_HitRadius
  
• Unit facing smoothing factor seems better for homing missile: Increase/decrease missile's facing per tick. Not instantly order the missile to face it's target.
  
• 
  
  • Set SM_Level[SM_Index] = (Level of SM_Ability for SM_Caster[SM_Index])
  Remove that variable, you are only using it for once.
  
  • Set SM_Damage[SM_Index] = SM_InitialDamage[SM_Level[SM_Index]]
• Optional:
  
  • (SM_Target[tempInt] is alive) Equal to True
  It seems weird if the missile gets removed instantly after the target has died. At least, let them arrive at target's location first. Check wc3 native abilities for example.

edit:yea i didnt rly know how to name the variable
edit1:i dont know how to do that
edit2:i intend to do the slow someday or something similar so i left the level(arrayed)
edit3:so what

 

[Vinz]

Good idea actually, make the facing start from +/-90 degrees then proceed smoothly.