Unit groups and Dead units

[Devalut]

Hello, tried to make an AOE spell that deals damage divided by the number of units in the group.

My problem is that the corpses are being counted with this group despite being conditioned out of it.

• Set EL_Damage_Group = (Units within EL_Damage_Area_of_Effect of EL_Targeted_Location matching ((((Matching unit) is A structure) Equal to False) and ((((Matching unit) is Magic Immune) Equal to False) and ((((Matching unit) is alive) Equal to True) and (((Matching unit) belongs to
• Unit Group - Pick every unit in EL_Damage_Group and do (Actions)
  • Loop - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Number of units in EL_Damage_Group) Less than 2
      • Then - Actions
        • Unit - Cause EL_Caster to damage (Picked unit), dealing (EL_Total_Damage x 1.50) damage of attack type Spells and damage type Magic
      • Else - Actions
        • Unit - Cause EL_Caster to damage (Picked unit), dealing (EL_Total_Damage / (Real((Number of units in EL_Damage_Group)))) damage of attack type Spells and damage type Magic
    • Special Effect - Create a special effect attached to the origin of (Picked unit) using Abilities\Weapons\Bolt\BoltImpact.mdl
    • Special Effect - Destroy (Last created special effect)

I've also tried units in range matching condition with no luck either.
- (Matching Unit) is alive equal to true.
- (Matching Unit) is dead equal to false.

 

[TriggerHappy]

Should be something like (and (Matching Unit)'s Current Life is Greater than or Equal to 0.405)

You may not be able to go as accurate as 0.405 in GUI, so you may need a variable.

Set RealVariable = 0 Custom script: set udg_RealVariable = 0.405 Set EL_Damage_Group = (Units within EL_Damage_Area_of_Effect of EL_Targeted_Location and (Matching Unit)'s Current Life is Greater than or Equal to RealVariable)

 

[Devalut]

Nope doesn't work

Targeted a single unit next to a corpse and dealt 100 damage

it was split down the middle.

Thanks for your quick response.

 

[TriggerHappy]

Can you show me your code then?

 

[PurplePoot]

GetUnitLife>0 works just as well since this isn't a damage detection system.

 

[LordDz]

Add "(Matching Unit) is alive equal to true." to your tempgroup.

 

[Devalut]

GetUnitLife>0 works just as well since this isn't a damage detection system.

Tried it, same result, thanks though.

Can you show me your code then?

It's simply the same as the first post's code but with the implementation of your suggestion.

Add "(Matching Unit) is alive equal to true." to your tempgroup.

oops sorry I'll add that instead of the last line at the top

Gunna try and add a buff to the units struck and then pick them accordingly, Thank you everyone who posted.

 

[PurplePoot]

It's impossible to tell what exactly is going on in your boolexpr since it cuts off the side of the screen. Can you post that line outside of

• tags?

 

[Devalut]

Set EL_Damage_Group = (Units within EL_Damage_Area_of_Effect of EL_Targeted_Location matching ((((Matching unit) is A structure) Equal to False) and ((((Matching unit) is Magic Immune) Equal to False) and ((((Matching unit) is alive) Equal to True) and (((Matching unit) is dead) Equal to False

I've tried it with both Alive and Dead booleans by themselves by the way

 

[LordDz]

Erm you don't need Both (Matching unit) is alive) Equal to True) and (((Matching unit) is dead)

A unit either Alive OR dead

 

[Devalut]

Erm you don't need Both (Matching unit) is alive) Equal to True) and (((Matching unit) is dead)

A unit either Alive OR dead

odd I thought I did post "I've tried it with both Alive and Dead booleans by themselves by the way"

 

[PurplePoot]

Interesting. Boolexprs have some bugs, but I was under the impression GUI was immune to them. Apparently it isn't, unless I'm missing something.

The easiest solution is to just fake that part of the filter in the ForGroup, however ugly it may be. Freehand:

• Unit Group - Pick Every Unit in (Units within blah of blah matching ((((Matching Unit) is a Structure) Equal to False) and (((Matching Unit) is Magic Immune) Equal to False))) and do Actions
  • Loop - Actions
    • If (All conditions are true) then do (then actions) else do (else actions)
      • If - Conditions
        • (((Picked Unit) is Alive) Equal to True)
      • Then - Actions
        • -------- do stuff as normal --------
      • Else - Actions

 

[LordDz]

They do work. I'm 100% sure of that.

 

[PurplePoot]

They do work. I'm 100% sure of that.

Boolexprs mostly work, they just have some awful edge-case bugs. I don't know for sure if that's what's happening here and nor do I remember whether those bugs even apply to GUI, but it's a distinct possibility given the code looks fine unless the OP is leaving something out.

 

[Devalut]

Directly copy pasted

Only one trigger for entire spell

 

[Troll-Brain]

IIRC, the IsUnitAlive gui stuff is some kind of GetWidgetLife > 0, and still IIRC a dead unit can have some life, let's say more than 0.405 (could even be caused by some passive ability)

He should use IsUnitType(...DEAD), i suppose you can do it also in GUI.

 

[PurplePoot]

IIRC, the IsUnitAlive gui stuff is some kind of GetWidgetLife > 0, and still IIRC a dead unit can have some life, let's say more than 0.405 (could even be caused by some passive ability)

He should use IsUnitType(...DEAD), i suppose you can do it also in GUI.

They can have life, but it's really really really rare and requires the mapmaker to have increased their life after they were dead.

 

[Zwiebelchen]

Can you convert the trigger to JASS and post it? This way we can find out if there's an issue with the GUI conditions.


But yes, there are abilities that increase the life above zero, even for dead units (tranquility is one of those abilities), breaking the unit alive condition.


Another issue:
You shouldn't use "Number of units in" within the group loop. It will basicly enumerate all units inside the group and count them on each group loop iteration, which might not only cause bugs, but also make your spell terribly CPU intense.
Instead, get the number of units before picking all units in the group and save it to a variable.

 

[IcemanBo]

Do you work with bj_wantDestroyGroup before?

If yes, try to add following on top of code, to ensure your group wont get destoyed when you count units:

Custom script: set bj_wantDestroyGroup = false

....
Your code

And like said by Zwiebelchen above you should count them with a custom integer to avoid calling this uncool CountUnits function so often. (and will bug if bj_wantDestroyGroup = true)

And if not, don't count units each time, would be enough todo it once before your loop and store it into a variable.

 

[Troll-Brain]

This custom script is not a good advise, it will screw up an "every unit of type", because how the GUI filters were done.
It will only enum the units of player(0), and then the group will be destroyed (and there is possibly other issues, such as a group leak, can't remember)
You can find them yourself following the logic.

So just use DestroyGroup.

 

[IcemanBo]

Yes, I agree that using DestroyGroup is better. But if he set bj_wantDestroyGroup = true before, that may be the reason for his bug. So he can easily test it with setting it to false before.

 

[Troll-Brain]

That doesn't make sense, since all units are enumed, even dead ones he said.
The problem here should be with dead unit which have life, like it has been said, the fix is to use IsUnitType(...DEAD).

 

[IcemanBo]

Ok, I see. Me stupid <.< .. can't read. :s

 

[Devalut]

Can you convert the trigger to JASS and post it? This way we can find out if there's an issue with the GUI conditions.

Another issue:
You shouldn't use "Number of units in" within the group loop. It will basicly enumerate all units inside the group and count them on each group loop iteration, which might not only cause bugs, but also make your spell terribly CPU intense.
Instead, get the number of units before picking all units in the group and save it to a variable.

JASS converted

function Trig_Bolt_Spell_Conditions takes nothing returns boolean
    if ( not ( GetSpellAbilityId() == 'A03H' ) ) then
        return false
    endif
    return true
endfunction

function Trig_Bolt_Spell_Func013002003001 takes nothing returns boolean
    return ( IsUnitType(GetFilterUnit(), UNIT_TYPE_STRUCTURE) == false )
endfunction

function Trig_Bolt_Spell_Func013002003002001 takes nothing returns boolean
    return ( IsUnitType(GetFilterUnit(), UNIT_TYPE_MAGIC_IMMUNE) == false )
endfunction

function Trig_Bolt_Spell_Func013002003002002 takes nothing returns boolean
    return ( IsUnitDeadBJ(GetFilterUnit()) == false )
endfunction

function Trig_Bolt_Spell_Func013002003002 takes nothing returns boolean
    return GetBooleanAnd( Trig_Bolt_Spell_Func013002003002001(), Trig_Bolt_Spell_Func013002003002002() )
endfunction

function Trig_Bolt_Spell_Func013002003 takes nothing returns boolean
    return GetBooleanAnd( Trig_Bolt_Spell_Func013002003001(), Trig_Bolt_Spell_Func013002003002() )
endfunction

function Trig_Bolt_Spell_Func018Func001C takes nothing returns boolean
    if ( not ( CountUnitsInGroup(udg_EL_Damage_Group) < 2 ) ) then
        return false
    endif
    return true
endfunction

function Trig_Bolt_Spell_Func018A takes nothing returns nothing
    if ( Trig_Bolt_Spell_Func018Func001C() ) then
        call UnitDamageTargetBJ( udg_EL_Caster, GetEnumUnit(), ( udg_EL_Total_Damage * 1.50 ), ATTACK_TYPE_NORMAL, DAMAGE_TYPE_MAGIC )
    else
        call UnitDamageTargetBJ( udg_EL_Caster, GetEnumUnit(), ( udg_EL_Total_Damage / I2R(CountUnitsInGroup(udg_EL_Damage_Group)) ), ATTACK_TYPE_NORMAL, DAMAGE_TYPE_MAGIC )
    endif
    call AddSpecialEffectTargetUnitBJ( "origin", GetEnumUnit(), "Abilities\\Weapons\\Bolt\\BoltImpact.mdl" )
    call DestroyEffectBJ( GetLastCreatedEffectBJ() )
endfunction

function Trig_Bolt_Spell_Func020Func004002003001 takes nothing returns boolean
    return ( IsUnitAliveBJ(GetFilterUnit()) == true )
endfunction

function Trig_Bolt_Spell_Func020Func004002003002 takes nothing returns boolean
    return ( IsPlayerEnemy(GetOwningPlayer(GetFilterUnit()), udg_EL_Owner) == true )
endfunction

function Trig_Bolt_Spell_Func020Func004002003 takes nothing returns boolean
    return GetBooleanAnd( Trig_Bolt_Spell_Func020Func004002003001(), Trig_Bolt_Spell_Func020Func004002003002() )
endfunction

function Trig_Bolt_Spell_Func020Func005Func001001 takes nothing returns boolean
    return ( IsUnitDeadBJ(GetEnumUnit()) == true )
endfunction

function Trig_Bolt_Spell_Func020Func005A takes nothing returns nothing
    if ( Trig_Bolt_Spell_Func020Func005Func001001() ) then
        call GroupRemoveUnitSimple( GetEnumUnit(), udg_EL_Seek_Group )
    else
        call DoNothing(  )
    endif
endfunction

function Trig_Bolt_Spell_Actions takes nothing returns nothing
    set udg_EL_Caster = GetTriggerUnit()
    set udg_EL_Owner = GetOwningPlayer(udg_EL_Caster)
    set udg_EL_Targeted_Location = GetSpellTargetLoc()
    set udg_EL_Ability_Level = GetUnitAbilityLevelSwapped(GetSpellAbilityId(), udg_EL_Caster)
    set udg_EL_Dummy_Ability_Ligtning = 'A00G'
    set udg_EL_Base_Number_of_Ligtnings = 2
    set udg_EL_Total_Number_of_Ligtnings = ( udg_EL_Base_Number_of_Ligtnings * udg_EL_Ability_Level )
    set udg_EL_Seek_Area_of_Effect = 500.00
    set udg_EL_Damage_Area_of_Effect = 275.00
    set udg_EL_Base_Damage = 100.00
    set udg_EL_Total_Damage = ( udg_EL_Base_Damage * I2R(udg_EL_Ability_Level) )
    set udg_EL_Damage_Group = GetUnitsInRangeOfLocMatching(udg_EL_Damage_Area_of_Effect, udg_EL_Targeted_Location, Condition(function Trig_Bolt_Spell_Func013002003))
    call AddSpecialEffectLocBJ( udg_EL_Targeted_Location, "Abilities\\Spells\\Human\\Thunderclap\\ThunderClapCaster.mdl" )
    call DestroyEffectBJ( GetLastCreatedEffectBJ() )
    call AddSpecialEffectLocBJ( udg_EL_Targeted_Location, "Abilities\\Spells\\Other\\Charm\\CharmTarget.mdl" )
    call DestroyEffectBJ( GetLastCreatedEffectBJ() )
    call ForGroupBJ( udg_EL_Damage_Group, function Trig_Bolt_Spell_Func018A )
    call DestroyGroup(udg_EL_Damage_Group)
    set bj_forLoopAIndex = 1
    set bj_forLoopAIndexEnd = udg_EL_Total_Number_of_Ligtnings
    loop
        exitwhen bj_forLoopAIndex > bj_forLoopAIndexEnd
        set udg_EL_Loop_Location = PolarProjectionBJ(udg_EL_Targeted_Location, 5.00, ( 360.00 / I2R(GetForLoopIndexA()) ))
        call AddSpecialEffectLocBJ( udg_EL_Loop_Location, "Abilities\\Spells\\Other\\Monsoon\\MonsoonBoltTarget.mdl" )
        call DestroyEffectBJ( GetLastCreatedEffectBJ() )
        set udg_EL_Seek_Group = GetUnitsInRangeOfLocMatching(udg_EL_Seek_Area_of_Effect, udg_EL_Targeted_Location, Condition(function Trig_Bolt_Spell_Func020Func004002003))
        call ForGroupBJ( udg_EL_Seek_Group, function Trig_Bolt_Spell_Func020Func005A )
        set udg_EL_Seek_Unit = GroupPickRandomUnit(udg_EL_Seek_Group)
        call CreateNUnitsAtLoc( 1, 'h008', udg_EL_Owner, udg_EL_Loop_Location, bj_UNIT_FACING )
        call UnitApplyTimedLifeBJ( 1.50, 'BTLF', GetLastCreatedUnit() )
        call UnitAddAbilityBJ( udg_EL_Dummy_Ability_Ligtning, GetLastCreatedUnit() )
        call SetUnitAbilityLevelSwapped( udg_EL_Dummy_Ability_Ligtning, GetLastCreatedUnit(), udg_EL_Ability_Level )
        call IssueTargetOrderBJ( GetLastCreatedUnit(), "chainlightning", udg_EL_Seek_Unit )
        call DestroyGroup(udg_EL_Seek_Group)
        call RemoveLocation(udg_EL_Loop_Location)
        set bj_forLoopAIndex = bj_forLoopAIndex + 1
    endloop
    call RemoveLocation(udg_EL_Targeted_Location)
endfunction

//===========================================================================
function InitTrig_Bolt_Spell takes nothing returns nothing
    set gg_trg_Bolt_Spell = CreateTrigger(  )
    call TriggerRegisterAnyUnitEventBJ( gg_trg_Bolt_Spell, EVENT_PLAYER_UNIT_SPELL_EFFECT )
    call TriggerAddCondition( gg_trg_Bolt_Spell, Condition( function Trig_Bolt_Spell_Conditions ) )
    call TriggerAddAction( gg_trg_Bolt_Spell, function Trig_Bolt_Spell_Actions )
endfunction

Get a variable first, sounds good!

Edit: added Jass tags

 

[Zwiebelchen]

Please wrap it in jass tags...

And after checking the JASS i'm positive your issue is caused by the nested group loop and the bugged IsUnitAlive.

 

[BPower]

I hope GUI doesn't generate "Do Nothing" by default.

Good Lord ... GUI
IsUnitAliveBJ --> IsUnitDeadBJ --> GetUnitState(whichUnit, UNIT_STATE_LIFE) <= 0.

I wonder why it does generate one time GetEnumUnit() and then GetFilterUnit(), while both functions are related to a group enumeration.

 

[Troll-Brain]

Please wrap it in jass tags...

And after checking the JASS i'm positive your issue is caused by the nested group loop and the bugged IsUnitAlive.

No, this native function is an ai one, so no the gui don't use it.
People tends to use it with the native function jasshelper support (you can do the very same if you import inside the map a custom common.j with this native function declaration added), but i prefer the IsUnitType(...DEAD) stuff.
Now maybe it's not bugged, but simply have different behaviors, which could make sense for an ai, i mean in some situations it could return false while it would be true with the IsUnitType stuff.
Like an unit which is being resurected and such. Clearly i've never used it, it's just a guess.

I wonder why it does generate one time GetEnumUnit() and then GetFilterUnit(), while both functions are related to a group enumeration.

GetEnumUnit -> ForForce
GetFilterUnit -> boolexpr filter

 

[Devalut]

So is it possible to fix this?

 

[IcemanBo]

Can you prepare your map a little bit, that we can easily test it and then attach it to your post?

 

[PurplePoot]

So is it possible to fix this?

Yes? If nothing else you can just use an if statement instead of a filter. However, I still suspect you're doing something strange, haven't looked at the complete code though.

 

[Troll-Brain]

Just use IsUnitType with UNIT_TYPE_DEAD ...

I don't know where it's in GUI, maybe in boolean condition.

 

[Zwiebelchen]

So is it possible to fix this?

Yeah, as I said, just do the unit counting before you do the ForGroup loop and replace the Is-unit-alive condition by an is-unit-type condition checking for the "dead" type.

No, this native function is an ai one, so no the gui don't use it.
People tends to use it with the native function jasshelper support (you can do the very same if you import inside the map a custom common.j with this native function declaration added), but i prefer the IsUnitType(...DEAD) stuff.
Now maybe it's not bugged, but simply have different behaviors, which could make sense for an ai, i mean in some situations it could return false while it would be true with the IsUnitType stuff.
Like an unit which is being resurected and such. Clearly i've never used it, it's just a guess.

Lol... Dude, I know that. I was referring to IsUnitAliveBJ, which simply gets the GetUnitState of the unit.

 

[Devalut]

Can you prepare your map a little bit, that we can easily test it and then attach it to your post?

Uploaded
Triggers under DEAD GROUP SPELL

 

[zv27]

Just use IsUnitType with UNIT_TYPE_DEAD ...

I don't know where it's in GUI, maybe in boolean condition.

as far as I know "UNIT_TYPE_DEAD" simply does not exist in the GUI version

 

[PurplePoot]

Your damage detection system is changing the units' life to 1 after they die, which is causing them to not get picked up by the life>0 checks to see if the units are alive.

The DDS appears to be badly bugged (I spent a few minutes trying to fix it, but the problems are too rampant to do a quick job) and I suggest you discard it in favour of a different one. I'll report it on the resource page.

 

[looking_for_help]

Your damage detection system is changing the units' life to 1 after they die, which is causing them to not get picked up by the life>0 checks to see if the units are alive.

Bug got fixed, further informations here.

 

[Devalut]

Awesome, This really drove me nuts thank you so much for all your hard work everyone!

 

[Uncle]

As I see with this system, heroes stop gaining experience if they kill the enemy with their abilities.

Are you sure you meant to post in this thread? It's 8 years old and has nothing to do with a system. Remember to check the date before reviving an old thread.