function conjunction malfunction

[.pablo.]

http://www.rhymezone.com/
rhymes are fun (and they get stuck in people's heads easily)

anyway, heres the problem:
i have a function in which im trying to join together a foreach and an if-then into one process, but it isn't working.  the function is supposed to take an array of objects/men and a single man, and return which object/man is closest to the single man.  heres the code:

private ["_potentialtargets", "_man", "_closestdistance", "_closesttarget", "_distance"];

_potentialtargets = _this select 0;
_man = _this select 1;

_closestdistance = 999999;

{
_distance = _man distance _x;

If (_distance < _closestdistance) Then
  {
     _closestdistance = _distance;
     _closesttarget = _x;
  }
Else
  {
  };
};
ForEach _potentialtargets;

_closesttarget

i just started making functions yesturday so i don't completely understand how they work yet.  what'd i do wrong?

[deaddog]

I think you need to take a look at my answer to your other post.  That shows the way you use if-then with the foreach command.  

[.pablo.]

The _x in a foreach command has "local" scope.  You can have a _x variable in the other part of your script but it will not be affected by the _x inside the foreach command.

how does that tell me what i did wrong?

the error im getting is unknown operator, pointing at the _potentialtargets; i think im using the _x fine, but idk...

[LCD]

how do u pass potential targets ?

btw u dont need da whole else part

Else
  {
  };

delete em ;D

dere may b somin else - but its 2 late now

LCD OUT

[.pablo.]

_potentialtargets = _this select 0;
_man = _this select 1;

then i call the function with this line:

;_target = [_potentialtargets, _guy] call closesttarget

and before that i set:

_potentialtargets = [man1, a1, a2, a3]

and i originally didn't include the else part but it wasn't working so i thought "whatever"

[LCD]

try printing da _potentialtargets thingy w/ da format command

hint format ["%1",_potentialtargets]

in da function - checks if u realy got da array or somin strange else

LCD OUT

[LCD]

just figured wat deaddog meant (shudnt party dat much next time )

dis thing is wrong  :-X

Code: [Select]

{
_distance = _man distance _x;

If (_distance < _closestdistance) Then
  {
    _closestdistance = _distance;
    _closesttarget = _x;
  }
Else
  {
  };
};
ForEach _potentialtargets;
change da code 2 somin like dat

"If ((_man distance _x)< _closestdistance) Then
{
    _closestdistance = _man distance _x;
    _closesttarget = _x;
};"ForEach _potentialtargets;

give it a shot nd report

LCD OUT

[.pablo.]

"If ((_man distance _x)< _closestdistance) Then
{
    _closestdistance = _distance;
    _closesttarget = _x;
};"ForEach _potentialtargets;

works perfect now, just had to change _distance to _man distance _x in this line:

_closestdistance = _distance;

[LCD]

u c ? im genius (nd drunk ;D but nit alot )

just qoute me right (look @ ma post again )

so solve da topic now it works ;D

LCD OUT