My spell: Demonic Wave

[Grundy]

im making a spell called Demonic Wave that targets an area and heals all allied demon units in the area and deals damage to all enemy non-demons in the area. i have like 4 or 5 different heroes that are supposed to be demons and i have a few units that some of them summon that are also demons. instead of using an "or" condition to check like 8 or 9 different unit types i want to just give demons a different classification under "Stats - Unit Classification" that only the demon units have. cant use Ancient because that classification is already giving to a different type of units, can't use mechanical because i have some mechanical units already, neutral wont work, summoned wont work (because all summoned units get that classification) cant use tauren because a different type has that (to be used with ancestral spirit, which only revives units with tauren classification), town hall wont work thats for a building classification and scroll of teleportation targets town halls i dont want demons to be targed for that scroll. tree wouldnt work for obvious reasons, undead wont work because im already using that classifcation for undead units, walkable wont work because that does other things, ward wouldnt work because that classification is already used for wards, and worker wouldnt work because then i'd get idle worker icons on the side of the screen. so the only classification i could use is "Giant" so i give my demon units the Giant classification.

but now i have a different problem.

i try to do an if/then/else to check if the units have the giant classification, i use the boolean "Unit - Unit Classification Check" to see if the unit has Giant classification, but the list of classifications does not have Giant in it. does anyone know of any jass function that checks for those types of classifications? it doesnt have "Tauren" in there either, but i KNOW they must have some kind of function for that bceause there are spells that only target taurens. probably coded in jass in the ability, but there has to be some function that checks if it has a certain classification and i cant find it in GUI. im hoping someone knows what it is in jass.

 

[Rad]

Dunno just use 8 conditions whats so bad about that

 

[Grundy]

Dunno just use 8 conditions whats so bad about that

for 1 it would be alot easier to use 1 condition
2 if i decided to make more demon units i would have to change not only this trigger but every trigger that effects demons.

if i could get it to work it would be more easier, efficient, and save time.

 

[Rad]

Um just use Or (multiple conditions) and copy paste to the other triggers, You could be done in the time to took you to reply so what the hell?

 

[Sargon]

Um just use Or (multiple conditions) and copy paste to the other triggers, You could be done in the time to took you to reply so what the hell?

He wants to make an efficient trigger. Efficiency and simplicity in triggers is good, because it lessens the chance of error. So why are you snapping at him for wanting to make good triggers?

 

[Rad]

Wtf sargon do you just monitor me all day? Its way more simple than trying to even figure out another option.

If you dont wanna do it the normal way, that works, then go ahead and spend 4 hours trying to figure it out

 

[Pyromancer]

also, it will save sum space. maby use worker classification, and put idle worker icon as none or change to idle demon, lol

 

[Grundy]

actually my way is a lot EASIER and more efficient and it will be much more easier to make new triggers based on demon units. and i just found out how to do it my way with 1 condition instead of 8 so i dont need help any more. took me less than 1 hour not 4 and it will make my life a lot easier now.

 

[Pyromancer]

how did you do it?

 

[Grundy]

He wants to make an efficient trigger. Efficiency and simplicity in triggers is good, because it lessens the chance of error. So why are you snapping at him for wanting to make good triggers?

whoa sargon on my side? thats a first.

Wtf sargon do you just monitor me all day? Its way more simple than trying to even figure out another option.

If you dont wanna do it the normal way, that works, then go ahead and spend 4 hours trying to figure it out

actually my way is a lot EASIER and more efficient and it will be much more easier to make new triggers based on demon units. and i just found out how to do it my way with 1 condition instead of 8 so i dont need help any more. took me less than 1 hour not 4 and it will make my life a lot easier now. and my way is the 'normal' way its just those stupid blizzard dudes forgot to add it in the GUI but they made a classification check for a reason. it makes it aLOT easier. doing a unit type compairison to check 8-9 different units is the noob way.

 

[Ruination]

I'm sorry you had to become defensive grundy. I'm curious though, and so people who search the forum for answers might find one in this thread, what custom script did you have to use to check for demons class?

 

[Grundy]

well im not being defensive, he just kept telling me over and over to do it the noob way so i had to give him reasons why i wont do it his noob way.

the way i did it was just use the Giant classification for all of my demon units, and you have to use jass or at least custom scripts to check if the unit has the giant classification. it is the same function that the "Unit - Unit Classification Check" boolean function does, except in GUI it doesnt give you the giant option. by looking on jass.sourceforge.net and looking up the 'unittype' variable i saw a huge list of different types (classifications) that can be used. i saw the one for giant is UNIT_TYPE_GIANT so you can use it in an if with a custom script for:

if ( IsUnitType( <Your Unit Here>, UNIT_TYPE_GIANT ) ) then
//your actions here
//else (optional)
//else actions here
endif

and using Giant classification for your own type of units will not interfere with anything because there are no units in the game that have Giant classification. i dont know why they even have it, but im glad they do. maybe they were going to give it to Mountain Giants but then decided not to? i dont know.

 

[Ruination]

I gotcha, thanks. Makes perfect sense

 

[MooMooMan]

use a trigger to make a different classification that should wurk

 

[Sargon]

use a trigger to make a different classification that should wurk

Umm...no it wouldn't. And he already solved the problem, about a third of the thread ago.