Snippet AutocastOrderEvent

Azlier

Old World Ghost
I won't be able to upgrade this. I lost my computer last week. If someone is nice enough to create a version that works with specific ID's, I would be happy to replace the one in the first post and give credits.
 

Azlier

Old World Ghost
Revamped after all this time. Should be much, much faster. And it now recognizes Searing Arrows and friends.

Alas, I had to sacrifice the most useless function. Breaks backwards compatibility for those who for some reason used it.

If I missed any autocast orders, please tell me.

EDIT: Quick change in data storage. No longer uses a hashtable, because I discovered the perfect 'offset' for attachment to order ID's, much like 0x100000 for handles. But this is safe because order ID's are static.
 

Kenny

Back for now.
Hahaha wow!! Your new method reminds me of how I did this back when you first released this. Nice work keeping this hashtable free too. Hastables seem like a bit to much for something like this.

I'll see if I can find mine for comparison, it is freakishly similar...

Edit:

Found it hiding away somewhere:

JASS:
library AutocastEvent initializer Init requires Event
    
    globals 
        private Event         AutoOn   = 0
        private Event         AutoOff  = 0
        private integer array OrderIds
        private integer array ArrayIds
    endglobals

    function TriggerRegisterAutocastOnEvent takes trigger whichTrigger returns nothing
        call AutoOn.register(whichTrigger)
    endfunction

    function TriggerRegisterAutocastOffEvent takes trigger whichTrigger returns nothing
        call AutoOff.register(whichTrigger)
    endfunction

    private function Actions takes nothing returns boolean
        local integer i = ArrayIds[S2I(SubString(I2S(GetIssuedOrderId()),3,6))]
        
        if i == 1 then
            call AutoOn.fire()
        elseif i == 2 then
            call AutoOff.fire()
        endif
        
        return false
    endfunction

    private function Init takes nothing returns nothing
        local trigger trig = CreateTrigger()
        local integer i    = 15
        
        set AutoOn  = Event.create()
        set AutoOff = Event.create()
        
        loop
            call TriggerRegisterPlayerUnitEvent(trig,Player(i),EVENT_PLAYER_UNIT_ISSUED_ORDER,null)
            exitwhen i == 0
            set i = i - 1
        endloop
        
        call TriggerAddCondition(trig,Condition(function Actions))
        
        set OrderIds[0]  = 25  // Repair
        set OrderIds[1]  = 41  // Kaboom
        set OrderIds[2]  = 51  // Get Corpse
        set OrderIds[3]  = 64  // Heal
        set OrderIds[4]  = 67  // Inner Fire
        set OrderIds[5]  = 76  // Slow
        set OrderIds[6]  = 102 // Bloodlust
        set OrderIds[7]  = 133 // Abolish Magic
        set OrderIds[8]  = 150 // Faerie Fire
        set OrderIds[9]  = 158 // Recharge
        set OrderIds[10] = 162 // Renew
        set OrderIds[11] = 174 // Searing Arrows
        set OrderIds[12] = 191 // Curse
        set OrderIds[13] = 198 // Raise Dead
        set OrderIds[14] = 203 // Restore
        set OrderIds[15] = 212 // Web
        set OrderIds[16] = 244 // Cold Arrows
        set OrderIds[17] = 255 // Poison Arrows
        set OrderIds[18] = 458 // Frost Armor
        set OrderIds[19] = 484 // Spell Steal
        set OrderIds[20] = 540 // Orb of Annihilation
        set OrderIds[21] = 546 // Essence of Blight
        set OrderIds[22] = 549 // Spirit Touch
        set OrderIds[23] = 552 // Carrion Beetles
        set OrderIds[24] = 578 // Black Arrow
        set OrderIds[25] = 602 // Parasite
    
        loop
            exitwhen i == 26
            set ArrayIds[OrderIds<i>] = 1
            set ArrayIds[OrderIds<i> + 1] = 2
            set i = i + 1
        endloop
    endfunction

endlibrary
</i></i>


I suppose:

JASS:
local integer i = ArrayIds[S2I(SubString(I2S(GetIssuedOrderId()),3,6))]


Could be replaced with:

JASS:
local integer i = ArrayIds[GetIssuedOrderId() - 852000]


To make it a bit faster..
 

Azlier

Old World Ghost
That is indeed freakishly similar.

Wow. But, mine is better :p. Marginally.

And mine doesn't leak.

EDIT: Made even faster. On initialization, of course, where it doesn't really matter.
 

Kenny

Back for now.
Mind explaining where mine leaks? (I might actually use it later on).

Oh, and also, in what ways is yours marginally better? To me they look pretty similar.. Except instead of 26 function calls (each with 3 sets, before your update), I have 26 sets and 1 loop with 2 sets per iteration.

Personally I think my way is neater, as it doesn't use unwarrented function calls.
 

Azlier

Old World Ghost
Mine is slower on initialization. Yours is slower at runtime and leaks strings.
 

Kenny

Back for now.
>Mine is slower on initialization. Yours is slower at runtime and leaks strings.

Not if I switch to [ljass]local integer i = ArrayIds[GetIssuedOrderId() - 852000][/ljass] or even [ljass]set I = ArrayIds[GetIssuedOrderId() - 852000][/ljass] and use a global integer. :)
 

Azlier

Old World Ghost
Then you get something (possibly) slightly faster than mine, only at initialization. Yours trades 26 or so function calls for 52 array reads, 26 variable sets, and 26 exitwhen statements. Somehow, function calls sound better.

But why use 852000 over the perfect 0xD0000?
 

Kenny

Back for now.
>But why use 852000 over the perfect 0xD0000?

Because 0xD0000 equates to something like 851968 and the first autocast order starts at 852025.

So in hex format 0xD0000 looks perfect, but in decimal format 852000 looks perfect. :D

But yeah, they both seem to work, and the speed difference is negligible and not worth worrying about.

Anywho, once again, nice work with this. I think it will come in handy for many mappers.
 

Azlier

Old World Ghost
>I think it will come in handy for many mappers.

Not likely. However, this certainly has some very good uses.
 

Kenny

Back for now.
I think you are missing the [ljass]"incineratearrowon"[/ljass] and [ljass]"incineratearrowoff"[/ljass] orders. I don't have your print order library, but the order ids for them are:

- [ljass]852671[/ljass] - [ljass]0xD02BF[/ljass] - On
- [ljass]852672[/ljass] - [ljass]0xD02C0[/ljass] - Off
 

Azlier

Old World Ghost
I've never heard of such a spell!

Very well. Added.
 
General chit-chat
Help Users
  • No one is chatting at the moment.
  • C Cherry.Grove:
    My one game was basically Among Us but you sabotaged an entire medieval city simulated drastically inefficiently with dynamic NPCs :S
  • C Cherry.Grove:
    of course I never finished it x D
  • jonas jonas:
    xD
  • jonas jonas:
    I think you can still run the original game, just there's a chance your map won't work with the newer patches
  • jonas jonas:
    what development framework have you been looking at for mobile development?
  • jonas jonas:
    I'm currently creating (as a hobby) a space invaders like game for mobile using Xamarin, after starting a few larger projects that I also may not have finished :rolleyes::p
  • The Helper The Helper:
    Awesome! you should look at the Atari VCS platform it does not have a huge user base but the one it has is super active at around 11k and they have no games in there store
  • The Helper The Helper:
    basically it is linux
  • The Helper The Helper:
    OMG it is BanLord as Blackveiled on the forum
  • The Helper The Helper:
    what is up buddy
  • jonas jonas:
    thanks for the tip, that sounds cool. Never heard about the VCS before but seems like it should be a good match
  • tom_mai78101 tom_mai78101:
    The winter vacation left a toll on me. I missed staying up late and sleeping in late.
  • C Cherry.Grove:
    i haven't looked into it yet
  • C Cherry.Grove:
    If I could use something Python compatible it would save me some learning
  • C Cherry.Grove:
    But Unity might be easiest
  • C Cherry.Grove:
    looking at old posts here makes me happy. Makes me be all like "wow I was pretty smart in 2009"
  • C Cherry.Grove:
    mostly failing at life through adulthood really demotivated me for a while
  • C Cherry.Grove:
    ....what I don't get is why the fuck they would force us to permanently convert to a version of the game everyone fucking hated : \
  • C Cherry.Grove:
    Raid Shadow Legends texture pack smh
  • jonas jonas:
    Most people fail at life throughout their adulthood. Sometimes we get to learn from it. Usually it just feels like crap.
    +1
  • The Helper The Helper:
    What does not kill you makes you stronger is what they say
  • C Cherry.Grove:
    for real! :cool:
  • Varine Varine:
    Are we still on the topic of failing? I failed lots!
  • jonas jonas:
    If you ever want to talk about it we're here. The way I see it, you're still around. That required a few very difficult successes if you ask me:)

    Members online

    No members online now.

    Affiliates

    Hive Workshop NUON Dome
    Top