Snippet GUI-Friendly Damage Detection

DuelPlayer

Member
Reaction score
21
If all spells in your map deal damage by triggers, you can temporarily set a boolean variable to true when dealing spell damage. Therefore, if the boolean is false when a unit takes damage, it must have come from an attack.

Otherwise, if you are comfortable triggering all buff-on-attack abilities, every unit can be given a hidden buff-on-attack ability which will let you determine whether the damage came from an attack by checking whether the damaged unit has that buff.

As far as I'm aware, these are the only reliable methods for discerning attack damage apart from spell damage in WC3.
This methods arent good...
And i tried once, i set the damage of cripple to 0
But the unit get knockback which is another trigger when it is casted
 

Weep

Godspeed to the sound of the pounding
Reaction score
401
I've been using the GDD damage detection in my spell, and referencing the GDD_DamageSource as "triggering unit" in my hashtable Key Handle. I didn't think it would work because the custom variable cannot be referenced in a Handle, but it seems to being working just fine.

Is this a bug, or was I misinformed, or what?
Well, not being able to use a variable in a hashtable seems to be a World Editor shortcoming.

It's not a bug that you can use Event Response - Triggering Unit in place of GDD_DamagedUnit (though unless you want it for a hashtable or after a wait, using the variable might be marginally faster :rolleyes:); I just haven't documented it. Also, FYI, Event Response - Damaged Unit doesn't work with GDD.
 

Weep

Godspeed to the sound of the pounding
Reaction score
401
Updated to 1.2.1, with two user-configuration functions, minor code cleanup, and expanded documentation. Happy users of 1.2.0 don't really need to update.
 

mapguy

New Member
Reaction score
46
I just have to copy + paste it in trigger editor?

that jass code box?
just it?

how to enable my locusts to DEAL damage.
 

hgkjfhfdsj

Active Member
Reaction score
55
question(s) (more learning purposes.)
JASS:
function GDD_LeaveMap takes nothing returns boolean
    local boolean cached = bj_slotControlUsed[5063]
    if(udg_GDD__Integers[2] < 64) then
        set udg_GDD__Integers[2] = udg_GDD__Integers[2]+1
    else
        set bj_slotControlUsed[5063] = true
        call ForGroup(udg_GDD__LeftMapGroup, function GDD_GroupRefresh)
        set udg_GDD__Integers[2] = 0
    endif
    call GroupAddUnit(udg_GDD__LeftMapGroup, GetFilterUnit())
    set bj_slotControlUsed[5063] = cached
    return false
endfunction


why set cached as bj_slotControlUsed instead of using bj_slotControlUsed directly, ie
JASS:

function GDD_GroupRefresh takes nothing returns nothing
    if (bj_slotControlUsed[5063] == true) then
        call GroupClear(udg_GDD__LeftMapGroup)
        set bj_slotControlUsed[5063] = false
    endif
    call GroupAddUnit(udg_GDD__LeftMapGroup, GetEnumUnit())
endfunction
// . .
function GDD_LeaveMap takes nothing returns boolean
    //local boolean cached = bj_slotControlUsed[5063]
    if(udg_GDD__Integers[2] < 64) then
        set udg_GDD__Integers[2] = udg_GDD__Integers[2]+1
    else
        set bj_slotControlUsed[5063] = true
        call ForGroup(udg_GDD__LeftMapGroup, function GDD_GroupRefresh)
        set udg_GDD__Integers[2] = 0
    endif
    call GroupAddUnit(udg_GDD__LeftMapGroup, GetFilterUnit())
    //set bj_slotControlUsed[5063] = cached  //why set it to original bj_slotControlUsed?
    return false
endfunction



ty:)
 

Weep

Godspeed to the sound of the pounding
Reaction score
401
It's because I'm co-opting a Blizzard global array instead of having to create my own global variable (it only matters without vJASS, which gives the freedom to declare globals in the code), and because I don't know whether some other system or spell might happen to use that exact same array variable (unlikely, but you never know ;)), I make sure to save its original value and put it back when I'm done.
 

the Immortal

I know, I know...
Reaction score
51
Setting a (surely empty) array's 5063rd member should allocate at least 5062 (surely empty) booleans in the game's memory from what I understand? Since arrays are consecutive in memory and blahblahblahboringtalk, my point is: you reset it anyway, so simply use the 0 index of the array. It is highly improbable that someone could use a 'variable changed value' event on this array member, so why waste these few bytes of memory creating 5k useless booleans! =D


It's a pointless optimization.. but hey, it is something!
 

Weep

Godspeed to the sound of the pounding
Reaction score
401
Setting a (surely empty) array's 5063rd member should allocate at least 5062 (surely empty) booleans in the game's memory from what I understand?
As far as I've read other people mention, WC3 allocates the entire array when declared, so this shouldn't make any difference. [edit] I guess it does...
 

Laiev

Hey Listen!!
Reaction score
188
100% leakless because this allow to add the event just one time, not 14923849273984 times for the same unit
 

Sevion

The DIY Ninja
Reaction score
424
There is no leakless.

But as far as you're concerned (and any users of this system) you'll be fine.
 

Sajin

User title under construction.
Reaction score
56
Would this system have a problem is the damage source and damaged unit are owned by the same player?
 

dianamod27

Member
Reaction score
1
Are there any conditions I can use to determine whether the damage taken by GDD_DamagedUnit is caused by a spell?
The problem is that I want certain triggers to fire only when a unit takes damage from a regular attack. I already have direct damage spells covered with

Trigger:
  • Or - Any (Conditions) are true
    • Conditions
      • (Current order of GDD_DamageSource) Equal to (Order(<Empty String>))
      • (Current order of GDD_DamageSource) Equal to (Order(smart))


but said triggers still fire when a unit takes periodic damage from Acid Bomb and similar spells.
I have this feeling that I'm overlooking something really obvious here but I just can't figure it out.
 

Ayanami

칼리
Reaction score
288
Are there any conditions I can use to determine whether the damage taken by GDD_DamagedUnit is caused by a spell?
The problem is that I want certain triggers to fire only when a unit takes damage from a regular attack. I already have direct damage spells covered with

Trigger:
  • Or - Any (Conditions) are true
    • Conditions
      • (Current order of GDD_DamageSource) Equal to (Order(<Empty String>))
      • (Current order of GDD_DamageSource) Equal to (Order(smart))


but said triggers still fire when a unit takes periodic damage from Acid Bomb and similar spells.
I have this feeling that I'm overlooking something really obvious here but I just can't figure it out.
I believe this System is only for detecting damage, and not damage type. You'd need to make your own damage system. For example, if you trigger all your spell-based damage, you can easily differentiate a physical attack and a spell attack.
 

dianamod27

Member
Reaction score
1
I believe this System is only for detecting damage, and not damage type. You'd need to make your own damage system. For example, if you trigger all your spell-based damage, you can easily differentiate a physical attack and a spell attack.
I was thinking about that as a measure of last resort but I guess I have to.
 
General chit-chat
Help Users
  • No one is chatting at the moment.
  • Varine Varine:
    I'll also probably need to set up a server at some point because these scans take up a huge amount of space. I'll worry about that when I have an office though
  • The Helper The Helper:
    I am sure Ghan could give you some solid advice on the Server stuff
  • The Helper The Helper:
    I hear that the latest IPhones and Samsungs have some of the best cameras out there.
  • jonas jonas:
    phone cameras are good for sending pics on whatsapp, but no good for macro photography
  • jonas jonas:
    I think you should buy a used macro lens, do you already have a system camera?
  • Varine Varine:
    No, I'm still looking into them and kind of leaning towards a Rebel T7 at the moment because it's pretty cheap and lenses seem pretty easy to come by. Used ones are only like 3-400 dollars, which is much cheaper than I was planning
  • jonas jonas:
    I can't really recommend you any specific lens or body due to lack of knowledge, but in my experience camera shops are very helpful, especially those that also sell used parts
  • Varine Varine:
    There's only one camera shop in my town and they seem to specialize in film, but I have time before I need it. I'm not in a rush so I'll keep researching them
  • jonas jonas:
    I think since you have such specific needs, you should find a way to give a try for different systems without having to commit to one
  • jonas jonas:
    It would suck to spend 400$ on a system that doesn't have the capability you need
  • jonas jonas:
    and it will probably take a couple of tries to get the settings right
  • Varine Varine:
    It would but I can also just make sure I can return them. The system I have works, it's pretty straightforward and I tested it with a camera I have. All it really is is a little cartridge that holds the transparency film and a white light source behind it that projects the image into the lens
  • Varine Varine:
    I'll have to fuck with the light a bit and try some different lighting methods to get color accuracy, but I have IT8 targets for that
    +1
  • The Helper The Helper:
    get that color accuracy :)
  • Varine Varine:
    They're really helpful for calibrating things because I'm really colorblind, so I'm not great at doing it by eye
    +2
  • Varine Varine:
    Well the long, slow slog of winter has officially begun here
  • Varine Varine:
    I have a bunch of community service I have to do, and I have a feeling there will be a lot of snow shoveling involved now
  • The Helper The Helper:
    don't know why people choose to live in that kind of weather :)
  • Varine Varine:
    Well
  • Varine Varine:
    My job is here
  • Varine Varine:
    I was born here man I didn't choose shit
  • Varine Varine:
    And also, I keep moving away and it doesn't get any better. I moved to San Antonio and shit froze there AND we had blackouts

    The Helper Discord

    Members online

    Affiliates

    Hive Workshop NUON Dome World Editor Tutorials

    Network Sponsors

    Apex Steel Pipe - Buys and sells Steel Pipe.
    Top