Snippet AutoFly

Azlier

Old World Ghost
Reaction score
461
Indeed. Especially because it's 11:20 am here.
 

Dest

New Member
Reaction score
26
Just one question, does this snippet works very well with spells, like, ie the hero casts a spell that makes the unit to fly back and then hits onto the ground.

Not requesting a spell, just being curious :p
 

Azlier

Old World Ghost
Reaction score
461
It's meant for spells. It's better than adding and removing Crow Form every time a spell is cast. :p
 

Jesus4Lyf

Good Idea™
Reaction score
397
How lame can I be...
Hm.
JASS:
        local integer i = 16
        call RegionAddRect(r, re)
        call TriggerRegisterEnterRegion(CreateTrigger(), r, Filter(function Actions))
        call RemoveRect(re)
        set re = null
        
        loop
            exitwhen i == 0
            set i = i - 1
            call GroupEnumUnitsOfPlayer(g, Player(i), Filter(function Actions))
        endloop

-->
JASS:
        local integer i = 15
        call RegionAddRect(r, re)
        call TriggerRegisterEnterRegion(CreateTrigger(), r, Filter(function Actions))
        call RemoveRect(re)
        set re = null
        
        loop
            call GroupEnumUnitsOfPlayer(g, Player(i), Filter(function Actions))
            exitwhen i == 0
            set i = i - 1
        endloop

That exitwhen comparison and subtract 1 should cut thy load time by 4.2 minutes. :rolleyes:

To be honest, I'd use yours and switch [LJASS]local integer i=16[/LJASS] with [LJASS]local integer i=bj_MAX_PLAYERS[/LJASS]. :)
It's just kind of nicer not to have magic numbers.

PS. Err, very nicely done. :) Good script.
PPS. You could use a static if to look for boolexpr utils for using the enum group. :)
Edit: This? Lol.
 

Azlier

Old World Ghost
Reaction score
461
Like this? Insanity.

JASS:
// Autofly - by Azlier
//  Requires a vJass preprocessor

// How to import:
//  1. Create a new trigger called AutoFly.
//  2. Convert the trigger to custom script, and replace all the code inside with this.

library AutoFly requires optional GroupUtils, optional GlobalGroup

private function Actions takes nothing returns boolean
    if UnitAddAbility(GetFilterUnit(), 'Amrf') then
        call UnitRemoveAbility(GetFilterUnit(), 'Amrf')
    endif
    return false
endfunction

private struct Hack extends array

    static method onInit takes nothing returns nothing
        local region r = CreateRegion()
        local rect re = GetWorldBounds()
        local group g
        local integer i = 15
        call RegionAddRect(r, re)
        call TriggerRegisterEnterRegion(CreateTrigger(), r, Filter(function Actions))
        call RemoveRect(re)
        set re = null
        
        static if LIBRARY_GroupUtils then
             set g = ENUM_GROUP
        elseif LIBRARY_GlobalGroup then
             set g = GROUP
        else
             set g = CreateGroup()
        endif
        loop
            call GroupEnumUnitsOfPlayer(g, Player(i), Filter(function Actions))
            exitwhen i == 0
            set i = i - 1
        endloop
        static not (if LIBRARY_GroupUtils or LIBRARY_GlobalGroup) then
             call DestroyGroup(g)
        endif
        set g = null

endmethod

endstruct

endlibrary
 

Romek

Super Moderator
Reaction score
963
> [ljass]static not (if LIBRARY_GroupUtils or LIBRARY_GlobalGroup) then[/ljass]
I'm sure that doesn't compile... :rolleyes:
 

Azlier

Old World Ghost
Reaction score
461
Well, duh. It doesn't make any sense. Did you know that [lJASS]then[/lJASS] can be omitted with static ifs? Unless Vex fixed that.
 

Azlier

Old World Ghost
Reaction score
461
I made it use library_once. Not that it matters much, but you can copy and paste it onto every spell or something without negative effects. Yes, I know that's evil. :p
 

Azlier

Old World Ghost
Reaction score
461
You tested that, Narks?

If it really does break Medivh, it's Romek's fault because I had it set up in a way that wouldn't break Medivh originally.
 

Narks

Vastly intelligent whale-like being from the stars
Reaction score
90
oh, wait, my mistake

damn you azlier, you have all your bases covered
 

Romek

Super Moderator
Reaction score
963
> If it really does break Medivh, it's Romek's fault because I had it set up in a way that wouldn't break Medivh originally.
You're welcome. :)
 

Troll-Brain

You can change this now in User CP.
Reaction score
85
I don't see how it could break the 'Amrf' ability since the ability is added/removed only when the unit enter the map and doesn't already have this ability.
 

Kenny

Back for now.
Reaction score
202
Can I ask why this doesn't use AIDS? Is there a specific reason or something? Otherwise, this could be done in like 10 lines.

Edit:

Something like this:

JASS:
library AutoFly
    private struct AutoFly extends array
        //! runtextmacro AIDS()
        private method AIDS_onCreate takes nothing returns nothing
            if UnitAddAbility(this.unit,'Amrf') then
                call UnitRemoveAbility(this.unit,'Amrf')
            endif
        endmethod
    endstruct
endlibrary
 

Kenny

Back for now.
Reaction score
202
Yeah fair enough. I just love how simple AIDS makes things, I've become use to using it all the time.

Off topic: Is there an ETA on an update for AIDS?
 
General chit-chat
Help Users
  • No one is chatting at the moment.

      The Helper Discord

      Members online

      Affiliates

      Hive Workshop NUON Dome World Editor Tutorials

      Network Sponsors

      Apex Steel Pipe - Buys and sells Steel Pipe.
      Top