checkPatrols sqf not working ?

[rainhardtgabriel]

if (_respawn) then {
[_x select 4 + (_x select 3), 3] execVM "Patrols\dela>
2016/05/05,  1:22:31   Error position: <+ (_x select 3), 3] execVM "Patrols\dela>
2016/05/05,  1:22:31   Error Generic error in expression
2016/05/05,  1:22:31 File mpmissions\__cur_mp.Takistan\Patrols\checkPatrols.sqf, line 45
2016/05/05,  1:26:59 "*** Tickets remaining: 80"
2016/05/05,  1:27:40 Error in expression < 3);

if (_respawn) then {
[_x select 4 + (_x select 3), 3] execVM "Patrols\dela>
2016/05/05,  1:27:40   Error position: <+ (_x select 3), 3] execVM "Patrols\dela>
2016/05/05,  1:27:40   Error Generic error in expression
2016/05/05,  1:27:40 File mpmissions\__cur_mp.Takistan\Patrols\checkPatrols.sqf, line 45
2016/05/05,  1:29:54 Client: Remote object 3:0 not found
2016/05/05,  1:29:56 EPE manager release (265|995|0)
2016/05/05,  1:29:56 Number of joints in scene after release: 56
2016/05/05,  1:29:56 Number of actors in scene after release: 325
2016/05/05,  1:29:56 EPE manager release (0|325|0)
2016/05/05,  1:29:56 Number of joints in scene after release: 56
2016/05/05,  1:29:56 Number of actors in scene after release: 60
2016/05/05,  1:29:58 EPE manager release (56|60|0)
2016/05/05,  1:29:58 Number of actors in scene after release: 60
2016/05/05,  1:29:58 EPE manager release (0|60|0)

[MisfitFred]

Der zugehörige Code könnte man etwas übersichtlicher gestallten und ein paar Kommentare wären auch nicht schlecht!

Vorschlag (So ähnlich könnte das aussehen)

//_typ
//_Patrol_Type = {inf,wheeld,tracked} 
//psydo enum: PATROL_TYPE
#define PATROLS_TYPE 0
#define PATROLS_TYPE_INFANTRY 0
#define PATROLS_TYPE_WHEELD   1
#define PATROLS_TYPE_TRACKED  2

#define PATROLS_TIMER 1
//_group
#define PATROLS_GROUPHANDLE 2
//_convoy
#define PATROLS_LIST_OF_CONVOY_VEHICLES
//_inf 
#define PATROLS_LIST_OF_CONVOY_INFANTRY_UNITS 4
//_start
#define PATROLS_START_MARKER 5
//_end
#define PATROLS_END_MARKER 6

//Zweckfrei?
Patrols = Patrols;
{   
    [(_x select PATROLS_LIST_OF_CONVOY_INFANTRY_UNITS) -  units (_x select PATROLS_GROUPHANDLE), 300] execVM "Patrols\delayeddeleteVehicles.sqf";
    _respawn = true;
    {
        if !(alive _x) then {
            [[_x], 300] execVM "Patrols\delayeddeleteVehicles.sqf";
        };
    } forEach units (_x select PATROLS_GROUPHANDLE) + (_x select PATROLS_LIST_OF_CONVOY_VEHICLES);

    {
        if (speed _x < 3) then {
            sleep 10;
            if (speed _x < 3) then {
                [(crew _x) + [_x], 0] execVM "Patrols\delayeddeleteVehicles.sqf";
            };
        };
    } forEach (_x select PATROLS_LIST_OF_CONVOY_VEHICLES);

    {
        if !(isNull _x) then {
            _respawn = false;
        };
    } forEach (_x select PATROLS_LIST_OF_CONVOY_VEHICLES);

    if (_respawn) then {
        [_x select PATROLS_LIST_OF_CONVOY_INFANTRY_UNITS + (_x select PATROLS_LIST_OF_CONVOY_VEHICLES), 3] execVM "Patrols\delayeddeleteVehicles.sqf";
        PatrolMarkers = PatrolMarkers + [_x select PATROLS_START_MARKER, _x select PATROLS_END_MARKER];
        Patrols = Patrols - [_x];
        [_x select PATROLS_TYPE, _x select PATROLS_TIMER] execVM "Patrols\delayedcreatePatrol.sqf";
    };
} forEach Patrols;

(Die Kommentare fehlen noch)

[MisfitFred]

Um die Frage zu beantworten: Klammern vergessen! Richtig: [(_x select 4) + (_x select 3), 3] execVM "Patrols\delayeddeleteVehicles.sqf