[Snippet] GetClosestWidget

[Ceday]

Because nobody uses And and Or natives.

 

[edo494]

the better question is: why dont you take code as argument and convert it into Condition?

 

[Bannar]

The answer is:
Back in time I haven't come up with such idea and now it would break compatibility.

I don't know if it's approriate to change argument type from boolexpr into code at this point.

@Ceday - you are right, DestroyBoolExpr might be unnecessary considering the occurance rate of And and Or methods.

 

[Bannar]

Updated - fixed issues similar to null-rect case. One of them involved replacing bj_mapInitialPlayableArea with GetWorldBounds().
I couldn't reproduce any other bugs. This is tested heavily - as with previous update, those "issues" could never occur, nonetheless I wanted this to have highest quality.

Also, textmacros + globals weren't doing what they were suppose to do. As of now, globals required for each module are hidden within private struct so after compilation is done, only those globals which are required become declared. Improved documentation, it's more readable. Also, added extra space within code to avoid code-blocks.

Demo code has been below snippets code. No API changes whatsoever.

 

[SanKakU]

function GetClosestDestructable takes real x, real y, boolexpr filter returns destructable

I don't understand why the boolexpr is even in there since you can't ask if it's a tree if you haven't got destructable yet. I don't think that is a usable boolexpr but if someone knows better, feel free to tell about it.

It looks like I have to use EnumDestructablesInRect but I don't understand the final part about code actionfunc.
I don't seem to be good at understanding enumeration, unfortunately. Ugh.
I have another function lying around somewhere that enumerates units and uses a group, but that's different, I can't put trees in groups and this function doesn't take that same entry code actionfunc. You cheated and used null in your demo/example so I don't know how to proceed.

 

[BPower]

The boolexpr is used for a filter function as such as you can use in EnumDestructableInRect.
Using null as argument means you do not filter out any destructables.

 

[FUJI]

I apologize for hijacking this thread but just wanted to share my experience recently as a user of this resource that: functions under the GROUP module could be avoided as the performance impact is very terrible when used periodically with filters on, code-wise: I don't know maybe it's because of the sorting algorithm used for it. I've just substitute to GroupEnum...(group) in conjunction with GetClosestUnitInGroup(group) instead and use it like how FirstOfGroup() loop works to enumerate a counted closest units which is naively performant than the available functions for it.

//globals: source, destination, picked
                    call GroupEnumUnitsInRange(source,x,y,range,filter)
                    loop
                        set picked = GetClosestUnitInGroup(x,y,source)
                        exitwhen picked == null or i == count
                        set i = i + 1
                        // actions (already sorted from closest to farthest)
                        // call GroupAddUnit(destination,picked)
                        call GroupRemoveUnit(source,picked)
                    endloop

// ^ is much more performant and works properly than these;
                   call GetClosestNUnitsInRange(x,y,range,count,source,filter)
                   call GetClosestNUnitsInGroup(x,y,count,source,destination)

The first method just remedied these performance drops I'm having, with proper results.
Note that these claims are only from my experience, didn't test those extensively yet.....

 

[Lt_Hawkeye]

I recall this library having a GetClosestDestructable(s) function, which would then provide an array of destructables where you can loop through.

Was that removed or am I thinking a different library?

 

[Bannar]

Hello @Clanzion , could you elaborate? The DESTS_MODULE boolean constant is still there. If initialized with 'true', GetClosestDestructable() will be added to the functions pool with help of textmacro.

 

[Lt_Hawkeye]

To elaborate, what I mean was that I recall there was an equivalent GetClosestNUnitsInGroup/GetClosestNUnitsInRange for destructible. @Bannar

 

[Bannar]

@Clanzion There are no native destructible groups exposed in Blizzard's interface. Maybe these were introduced in one of Reforged patches, not as fluent in changes they brought.

Because of this, don't believe functions you speak of were actually ever part of this library. Don't be afraid to challenge me on this, but some guidance i.e. pointing to some specific post would be required.

Crafting custom destructible group is probably is a lot of overhead. Target audience is also very niche.

 

[Lt_Hawkeye]

@Clanzion There are no native destructible groups exposed in Blizzard's interface.

I apologize for this late reply. As it turns out, it was actually line enumeration by icemanbo that had a destructable array.