[Unte99]

1)

pawn Код:

#define MAX_DROPPED_GPARTS 10
static GunParts[MAX_DROPPED_GPARTS][GunPart];

which is basically

pawn Код:

static GunParts[10][GunPart];

now in the command:

pawn Код:

for(new i = 0; i < sizeof(crateLocations); i++)
            {
                if(IsPlayerInRangeOfPoint(playerid, 7.0, crateLocations[i][gPosx], crateLocations[i][gPosy], crateLocations[i][gPosz]))
                {

you made a loop with sizeof(crateLocations), which is

pawn Код:

static crateLocations[][GunPartsCoords] = {
    {2834.8857, -2386.6113, 17.5051}, // 0
    {2842.3608, -2500.4163, 17.5051}, // 1
    {2823.5750, -2474.1826, 12.0983}, // 2
    {2837.7878, -2449.3562, 19.9220}, // 3
    {2846.3259, -2447.9080, 19.9220}, // 4
    {2842.2053, -2500.4648, 17.5051} // 5
};

6, because crateLocations has 6 rows of coordinates you made for picking up parts.

So the loop will be repeated only 6 times, not 10 times as the GunParts[10] first dimension index is.

Fix this code so the loop checks all available coordinates for picking up the parts and also checks all available indexes of GunParts:

pawn Код:

CMD:takegunpart(playerid, params[])
{
    /*if(pInfo[playerid][pFamily] > 0)
    {
        if(pInfo[playerid][pRank] >= 5)
        {*/
            new pickedmsg[30];
            for(new i = 0; i < sizeof(crateLocations); i++)
            {
                if(IsPlayerInRangeOfPoint(playerid, 7.0, crateLocations[i][gPosx], crateLocations[i][gPosy], crateLocations[i][gPosz]))
                {
                    if(if(GunParts[i][gDropped]==0)
                    {
                        format(pickedmsg, sizeof(pickedmsg), "Skipped because GunParts[%i][gDropped]==0",i); // I added this message for debugging purposes
                        SendClientMessage(playerid, -1, pickedmsg);
                    }
                    if(GunParts[i][gDropped] == 0) continue;
                    new gunpartamount = random (6);
                    //PlayerInfo[playerid][Gunparts] += gunpartamount;
                    format(pickedmsg, sizeof(pickedmsg), "You've picked %i gun parts.", gunpartamount);
                    SendClientMessage(playerid, -1, pickedmsg);
                    GunParts[i][gDropped] = 0;
                    return 1;
                }

            }
        /*}
    }*/
    return 1;
}

2) Don't create variables in loops.