Snippet Trackable2

[Jesus4Lyf]

Hey, very nice. That's a lot better!

Hmmm. You don't null t or tr, but that doesn't matter since they're permanent anyway... So well done. I'm fairly impressed (but still can't rep you again lol). ^_^ (Even if a lot of this is my code, you seem to thoroughly understand how it works, so well done.)

PS. I didn't look in huge detail, but I assume you tested it and it works, and from what I've seen that should mean it's all correct.

 

[Azlier]

>You don't null t or tr, but that doesn't matter since they're permanent anyway
I foresaw that.

>you seem to thoroughly understand how it works, so well done
The only thing that baffles me are your boolexprs.

>I assume you tested it and it works
I did. Why would I release something I did not test? In fact, there was a certain problem that was exactly like the one in R&R. I shall fix R&R, next!

 

[Jesus4Lyf]

>The only thing that baffles me are your boolexprs.
XD What about 'em?

 

[Azlier]

They're probably simple, but I'm very tired and haven't looked at them.

 

[Azlier]

Update. Doubled instance limit with an estimated efficiency gain of 2%. Yes, I totally made up that estimate.

 

[Azlier]

 

[Azlier]

This hasn't been updated in a while, eh? Added AnyClick and AnyTrack events.

 

[Jesus4Lyf]

Kudos for that. That's nice functionality.
*Hasn't read the code.*

 

[Azlier]

Bump, I dare say!

 

[Dr.Jack]

Will be reviewed soon. Sorry for the delay!
If you feel you have other submissions that should be approved bump them as well!

 

[Dr.Jack]

Approved!
Thanks for contributing!

 

[Jesus4Lyf]

This was probably my favourite resource from Azlier...

Just because the reason I learned JASS in the first place was trackables, and at the time it was "impossible" to use them without local handle vars...

But between us we have it down to not even using H2I now, with this.

 

[Azlier]

I take credit for the original string of crappy versions.

 

[Anachron]

Fun fun, check what I have made here:
Window System.

Its alot like yours, but also gives a structure to make window systems for diffrent players without using other areas.

 

[Jesus4Lyf]

I don't understand how that is similar.

This is O(1) complexity H2I/GetHandleId free trackable attaching, and that is pretty much its entire point.

Yours is O(n) complexity H2I/GetHandleId trackable attaching.

Also, your events allow exactly one thing to be fired off. This provides a full WC3 style event that allows multiple triggers to fire off one event and supports dynamic triggers.

You'll have to excuse me, I tend to pick on code quality.

 

[luorax]

Okay, sorry for bumping this very old thread, but I found a bug:

You're missing these two lines from the "createForPlayer" method:

set this.onClickEv = Event.create()
set this.onTrackEv = Event.create()

 

[Azlier]

Fixed. Didn't fix it in the demo map, though. Don't have a properly equipped NewGen ready to do so.