[Trigger] Squad system problem:

[PaperJack]

I have been making my own squad system and I got a few questions:

I've using custom values (all members of a squad have the same custom value and so I can select them), is there another way?

When a squad moves, how can I get the offsets of where it should go ?
Like, I don't want them to mass up humping each other while trying to go at a single point, I want to go in a formation or something.
I think this can be accomplished only with JASS, though.

Can someone give me a list of all orders (the unprocessed version,like "move" or "attack") ?

Thanks!

 

[Cokemonkey11]

Use "point with polar offset", or if you're using Jass you can use trig.

 

[PaperJack]

I know that, it's not what I asked.

 

[Cokemonkey11]

You're right, you just said

humping each other while trying to go at a single point

Which means either I don't understand your question or you don't.

 

[PaperJack]

I want them to go in a formation or something.

Or perhaps you should quote the complete question.

 

[Cokemonkey11]

I want them to go in a formation or something.

You would do that by using point with polar offset or trig in jass.

 

[PaperJack]

You mean, I should use two indexed variables, one for the angle and the other for the position, and then refer to them in a for loop ?
I've tried and it didn't work because some units blocked off others and made a mess.

 

[Cokemonkey11]

No only 1 indexed variable.

Like:
funtion groupmove takes unit array u, integer unitsInGroup returns nothing
local real x
local real y
local real dist
local real angle
loop
exitwhen index>unitsInGroup
set dist=GetRandomReal(100,400)
set angle=index*(360/unitsInGroup)
set x=GetUnitX(u[index])+dist*Cos(angle*bjEGTORAD)
set y=GetUnitY(u[index])+dist*Sin(angle*bjEGTORAD)
call SetUnitX(u[index],x)
call SetUnitY(u[index],y)
set index=index+1
endloop
endfunction

I don't usually write code without an editor, so that of course is not real code that will work.

 

[Dr Super Good]

Try using hashtables, they may be quite effective as they allow the attachment of values to units in such a way that collisions via other systems can be impossiable. Also JASS probably would be best for your task.

 

[PaperJack]

No only 1 indexed variable.

Like:
funtion groupmove takes unit array u, integer unitsInGroup returns nothing
local real x
local real y
local real dist
local real angle
loop
exitwhen index>unitsInGroup
set dist=GetRandomReal(100,400)
set angle=index*(360/unitsInGroup)
set x=GetUnitX(u[index])+dist*Cos(angle*bjEGTORAD)
set y=GetUnitY(u[index])+dist*Sin(angle*bjEGTORAD)
call SetUnitX(u[index],x)
call SetUnitY(u[index],y)
set index=index+1
endloop
endfunction

What's bj ?

 

[Dr Super Good]

A part of the name of some functions / variables.
Those variables he listed do not exist though. How sloppy of him.

 

[PaperJack]

That's why I asked, I thought it was some kind of preset value.

 

[Cokemonkey11]

A part of the name of some functions / variables.
Those variables he listed do not exist though. How sloppy of him.

I don't have an underscore key, thanks for the comment though,

 

[PaperJack]

What is it though ?

 

[Cokemonkey11]

bj:DEGTORAD

(Replace : with underscore) is a blizzard constant that when multiplied to an angle represented by a real number, is converted to a radian represented by a real number.

 

[PaperJack]

Thanks, imma finish the trigger version of your code today and post the results

 

[Element of Water]

It must be an ass to code without an underscore button...

 

[PaperJack]

Didn't work. The units collide with each other and often can't reach their predistinated spot.

 

[Element of Water]

This should do it:

//g is the group you want to move
//targetX is the x point you want the group to move to
//targetY is the y point you want the group to move to
//dist is the distance from the target you want each unit to be

function GroupMove takes group g, real targetX, real targetY, real dist returns nothing
    local integer i = 0
    local unit u
    local real x
    local real y
    local real angle
    local integer numUnits = CountUnitsInGroup(g)
    local group g2 = CreateGroup()
    call GroupAddGroup(g, g2)
    loop
        set u = FirstOfGroup(g2)
        exitwhen u == null
        
        set angle=i*((2*bj_PI)/numUnits)
        set x=targetX+dist*Cos(angle)
        set y=targetY+dist*Sin(angle)
        call IssuePointOrder(u, "move", x, y)
        
        set i = i + 1
        call GroupRemoveUnit(g2, u)
    endloop
    
    set u = null
    call DestroyGroup(g2)
    set g2 = null
endfunction

It puts the units in a circle formation around the target X/Y

EDIT: Oops, declared a local that was taken as a parameter. Fixed.

 

[Mechanical Man]

About moving in formation, DSG already once told that is impossible in warcraft because of pathing implementation.

Not using custom value is possible. But it will require a little more time (it shouldn't be so demanding though). Just make a function which cycle through all groups in the array where are the groups stored and it returns the array position in which is the group containing that unit stored.

 

[PaperJack]

an array has limited slots though.
Also, I don't really know how to write JASS, I know a few other languages, but not JASS. I'm too lazy to open up the function library and start testing with stuff.

 

[Element of Water]

Look at my code. Call that function on a group and it should move it in a circle formation.

 

[PaperJack]

Is it in vjass ?

What the hell, the map makes warcraft 3 crash when I try ordering something.
Apart from the numerous leaks (I will fix them once this is finished), anyone knows why ?

 

[Element of Water]

No, it's not vJass. I might have got some syntax errors. Let me check it over.

EDIT: I just downloaded the map, I think the reason it gives errors is you put it in a trigger on its own. Unless you use JassNewGenPack it should go in the map header (click on the name of the map in the trigger editor, there is a section you can type into there)

EDIT2: I just looked over your code, and, quite frankly, it fails. It simply won't work at all.