[Solved] BlzGetTriggerPlayerMouse problems

[xorkatoss]

alright guys so I'm having problems with these:

native BlzGetTriggerPlayerMouseX
native BlzGetTriggerPlayerMouseY
native BlzGetTriggerPlayerMousePosition

Triggers for testing:

Spoiler: GUI Test 1

• Test
  • Events
    • Unit - A unit Starts the effect of an ability
  • Conditions
    • (Ability being cast) Equal to Blink
  • Actions
    • Custom script: set udg_Point = BlzGetTriggerPlayerMousePosition()
    • Special Effect - Create a special effect at Point using Abilities\Spells\NightElf\Blink\BlinkTarget.mdl
    • Special Effect - Destroy (Last created special effect)
    • Unit - Move (Triggering unit) instantly to Point
    • Custom script: call RemoveLocation(udg_Point)

I guess I must be using it wrong?

Because this doesn't work either:

Spoiler: GUI Test 2

• Test 2
  • Events
    • Unit - A unit Starts the effect of an ability
  • Conditions
    • (Ability being cast) Equal to Blink 2
  • Actions
    • Custom script: set udg_X = BlzGetTriggerPlayerMouseX()
    • Custom script: set udg_Y = BlzGetTriggerPlayerMouseY()
    • Set VariableSet Point = (Point(X, Y))
    • Special Effect - Create a special effect at Point using Abilities\Spells\NightElf\Blink\BlinkTarget.mdl
    • Special Effect - Destroy (Last created special effect)
    • Unit - Move (Triggering unit) instantly to Point
    • Custom script: call RemoveLocation(udg_Point)

They fail to get mouse position or X/Y and instead return center of playable map.

Only when using it with a mouse detection system I can get it to work:

Spoiler: GUI Test 3

• Detect Mouse
  • Events
    • Player - Player 1 (Red) issues Mouse Move event
  • Conditions
  • Actions
    • Set VariableSet PlayerMouse_X = (Mouse Position X for Triggered Mouse Event)
    • Set VariableSet PlayerMouse_Y = (Mouse Position Y for Triggered Mouse Event)

• Test 3
  • Events
    • Unit - A unit Starts the effect of an ability
  • Conditions
    • (Ability being cast) Equal to Blink 3
  • Actions
    • Set VariableSet Point = (Point(PlayerMouse_X, PlayerMouse_Y))
    • Special Effect - Create a special effect at Point using Abilities\Spells\NightElf\Blink\BlinkTarget.mdl
    • Special Effect - Destroy (Last created special effect)
    • Unit - Move (Triggering unit) instantly to Point
    • Custom script: call RemoveLocation(udg_Point)

So I guess it's an easy work-around to have a system save the X/Y of the mouse and just use that when you have to but I was wondering if there is a way to just get the X/Y only when needed.

I'm also afraid of performance issues when saving X/Y of all players when there is a mouse move because that's pretty much all the time when playing.

 

[apsyll]

it's how blizz always handled their things, it's like to try to get triggering unit by a multieboard button press event will return nothing too.
to the performance I don't really think this will have to much of an impact to change 2 real values...
If you still too concerned about it you could add buffer like min distance or refresh interval, I would start a timer that sets a in the move event checked Boolean back or something like this.

 

[Dr Super Good]

I guess I must be using it wrong?

Yes. The mouse event responses make no sense for that event. There was no mouse event issued for it to return the X/Y/Position of.

They fail to get mouse position or X/Y and instead return center of playable map.

The mouse event responses make no sense for that event. There was no mouse event issued for it to return the X/Y/Position of.

Only when using it with a mouse detection system I can get it to work:

Yes because then the event response makes sense. It is designed to go with the mouse event to return data about the event.

The reason for this is because the mouse coordinates have to be synchronized. The event handles all that.

 

[xorkatoss]

@apsyll 2 reals is not a big deal but with full players it's 24x2 = 48 reals getting updated every millisecond (give or take)

@Dr Super Good yeah I figured that out, at first I thought the native was supposed to work like e.g "GetUnitLoc" which fetches the unit's location
but instead it only works with mouse events which is inconvenient >_<

 

[Drake53]

I think you're looking for these:

constant native GetSpellTargetLoc           takes nothing returns location
constant native GetSpellTargetX				takes nothing returns real
constant native GetSpellTargetY				takes nothing returns real

 

[apsyll]

2 reals is not a big deal but with full players it's 24x2 = 48 reals getting updated every millisecond (give or take)

True and I'm not that much into all the behind the scenes to know how often the xy gets updated but I'm still pretty confident that the computer we use nowadays are more then good to handle the worst case of 48 reals variables.

I understand that more efficient and streamlined is wishful but in the end having something working is the best result and the most of the times people breaking their head over the less important point.
For me have nice code doesn't mean have the fastest executive one it's for me have the one that is the best human and machine interpretationable one, where you can build on top of or even find a better solution and refactore it after you gained more knowledge.

 

[xorkatoss]

@Drake53 I tried it and it doesnt work either, what you showed me must be the equivalent to this:

• Set VariableSet Point = (Target point of ability being cast)

@apsyll yes the problem is not the 48 reals, its the fact that having a lot of systems like this eventually add up and become a lot for the pc to handle