Deafening Strike v2.1

Moderator · Jun 3, 2009

07:00, 3rd Jun 2009
hvo-busterkomo: Leak free and MUI. It seems indexing systems in GUI have become quite popular. The documentation was extensive, allowing anyone to easily modify it. The only suggestion I have is to combine the "Deafening Strike Init" and "Actions to Apply" into a single trigger (it doesn't really matter). I guess people may find it easier to read the way you have it set up. One last thing; it's 'deafening', not 'defening'

The_Reborn_Devil:
The triggering looks good, but you don't need to null globals.
Approved.

Palaslayer · Jun 3, 2009

nice.

Roflcoptor · Jun 3, 2009

Thanks, the only thing I don't like is the animation while approaching the enemy. Anyone know how to change this?

Mage_Goo · Jun 3, 2009

Found Bug
If you target destructible with this spell, (like barrel) you will flying to upper right of map and never come back.

Will review the spell if got time.

Palaslayer · Jun 3, 2009

then make the spell unable to target destructibles.

Roflcoptor · Jun 3, 2009

Vincent · Jun 4, 2009

nice spell! ill try it.

Edit: i tried the spell and here are my comments
-i think its a bit strange that the enemy flies up to meet the blade masters "Blade" face first only to be knocked back down, unless the blade master is a telekinetic of course.

it would be best if its melee or if an illusion knocks them from behind.

My rating is 3.2/5 may be useful for some people.

Deuterium · Jun 6, 2009

Since the spell seems flawless and I'm really tired and sleepy, I won't be doing that huge review I always use.

First, (some people would nag about this being a BJ as they don't know the core of this) don't use bj_wantDestroyGroup unless you know how it functions. It's not wrong to use it if you don't know how it functions, but why not use call DestroyGroup() manually then?

Anyways, in case you don't know how it functions, it interacts with a part of a BJ function when picking units which contains an if/then/else to automatically call DestroyGroup() if the bj_wantDestroyGroup is set to true.

It's one additional action and the decrease of efficiency due to it would be very minimal, so it's not that much of a big deal.

Yet I still prefer removing it manually.

You've nulled globals that don't leak and I usually know that is unnecessary. However, that's a controversial matter so I'd let you do what you want considering that.

Now the one big problem, the animations and effects.
I looks really weird that the enemy unit is flying before being hit.

I don't know about your mathematical abilities, but if they're high enough you can use some advanced math make the triggering unit's motion arc-ish so that he takes the unit up and then slams:

Scratch your head a bit and try to find out how to do it.

Combine linear movement, with an arc-ish motion, then linear, then slam to the ground!

Other than that, everything seemed nice and functioning as supposed to!

Amazing job for a second spell!

All-in-all:

Spell is well done and functions as it should. Idea is nice and can be useful. Triggering is MUI, leakless and as efficient as it could be in GUI. Units' elevation could be improved and adjusted, but that does not affect that badly.

Palaslayer · Jun 7, 2009

Deuterium said:
First, (some people would nag about this being a BJ as they don't know the core of this) don't use bj_wantDestroyGroup unless you know how it functions. It's not wrong to use it if you don't know how it functions, but why not use call DestroyGroup() manually then?

Anyways, in case you don't know how it functions, it interacts with a part of a BJ function when picking units which contains an if/then/else to automatically call DestroyGroup() if the bj_wantDestroyGroup is set to true.

about the bj thingy, in gui it is just shorter and as a result more effective. and it sucks to set a variable first, use it and then manually remove it..

Deuterium · Jun 7, 2009

Palaslayer said:
about the bj thingy, in gui it is just shorter and as a result more effective. and it sucks to set a variable first, use it and then manually remove it..

Well after second thought, it's almost the same thing.
Using bj_wantDestroyGroup you set a boolean, manually you set a Unit Group Variable.
So in both cases you are setting a variable.

The difference is that later you have to script call DestroyGroup() when and however using if bj_wantDestroyGrouo it runs the call DestroyGroup() automatically.

So it's the same thing in the end...

Palaslayer · Jun 7, 2009

yes, but the work is not as much as without bj...

Roflcoptor · Jun 7, 2009

Thanks I'll work on the movement since it's kinda unrealistic

EDIT: I updated the spell. It's now near to the movement Deuterium suggested except that the caster just blinks.

Palaslayer · Jun 30, 2009

approved!

Roflcoptor · Jun 30, 2009

Well @ Deut: some globals have to be nulled, not because they leak but just because they are set like this (example):

Set X[A]=(X[A]+1)

Now if X[A] has reached for example the value 10, and the trigger runs the second time with the same index it would start with 11 and end with 20 and so on......

Well btw updated^^

I'll perhaps workout a second version of this spell working with setUnitX and setUnitY, but I'm currently working on hero contest

Deuterium · Jun 30, 2009

Roflcoptor said:
I'll perhaps workout a second version of this spell working with setUnitX and setUnitY, but I'm currently working on hero contest

It'll become very crowded if in GUI i guess, but would be good

Roflcoptor · Jun 30, 2009

Yup defenately, but I am far from being able to do this spell in jass since there are so many things i don't understand nad to which i can't find answers in any tutorial :/

Deuterium · Jul 1, 2009

Yeah Jass tutorials are lacking... they only mention the only basic.

You need to learn through reading what people do in their spells...
questions are welcome

I might be able to help on some stuff.

calex3 · Jul 8, 2009

downloading again to see.....
Hmm nice....i thought that this could be easier...but don't really know....and nice effects...overall 4.5 cause of its simpleness i think.....so 4.5= 5

i will rate!

Kingz · Jul 9, 2009

Just to mention:
bj_wantDestroyGroup() is less efficent than manualy calling DestroyGroup().
All bj functions are bad cause they are functions which call another function.
Unfourtanetly almost all GUI functions are bj.
Point is, if you want efficency use DestroyGroup() over bj_wantDestroyGroup().
Though it is only a minor efficency improvement, it is an improvement.

Deuterium · Jul 9, 2009

Well, Kingz I don't think in GUI it's gonna make any difference

I mean here's what's gonna happen in the Jass background:

If you use the bj as true

JASS:

set bj_wantDestroyGroup = true //this happens before the pick units function

    //here's the function
    local boolean wantDestroy = bj_wantDestroyGroup
    set bj_wantDestroyGroup = false
    call ForGroup(whichGroup, callback)
    if (wantDestroy) then // since it's true it'll call the destroy
        call DestroyGroup(whichGroup)
    endif

If done manually

JASS:

    local boolean wantDestroy = bj_wantDestroyGroup
    set bj_wantDestroyGroup = false
    call ForGroup(whichGroup, callback)
    if (wantDestroy) then // since it's false it'll skip this
        call DestroyGroup(whichGroup)
    endif
    call DestroyGroup(whichGroup) //manually destroying

So in GUI, same thing

correct me if I'm wrong

Roflcoptor · Jul 9, 2009

More comments on the spell plox

I don't really think, that it's much of a big deal anyway, since it only happens once per cast and it only is (if Kingz is right) such a small issue that it won't effect performance anyway unless you have like 100 casters casting simultanously

Deafening Strike v2.1

Defeaning Strike (Map)

Moderator

Moderator

Similar threads