[DerickClark]

I got errors from code.

pawn Код:

error 017: undefined symbol "AddARepairPickups"
warning 203: symbol is never used: "ARepairPickups"
warning 203: symbol is never used: "IsPlayerAtRepair"
Pawn compiler 3.2.3664          Copyright (c) 1997-2006, ITB CompuPhase


1 Error.

pawn Код:

ARepairPickups(Float:x, Float:y, Float:z)
{
    for (new i; i < sizeof(ARepairPickups); i++)
    {
        if (ARepairPickups[i][PickupID] == 0)
        {
            ARepairPickups[i][PickupID] = CreatePickup(27, 1, x, y, z, 0);
            ARepairPickups[i][pux] = x;
            ARepairPickups[i][puy] = y;
            ARepairPickups[i][puz] = z;
            Create3DTextLabel(TXT_PickupRepair, 0xFFFFFFFF, x, y, z + 0.8, 30.0, 0);
            CreateDynamicMapIcon(x, y, z, 56, 0, 0, 0, -1, 300.0);
            break;
        }
    }
}
bool:IsPlayerAtRepair(playerid)
{

    for(new i; i != sizeof(ARepairPickups); i++)
        if(IsPlayerInRangeOfPoint(playerid, 2.5, ARepairPickups[i][pux], ARepairPickups[i][puy], ARepairPickups[i][puz]))
            return true;

    return false;
}
CMD:createrepair(playerid,params[])
{
   new Float:x, Float:y, Float:z;
   GetPlayerPos(playerid, x, y, z);
   AddARepairPickups(x, y, z);
   return 1;
}
CMD:repairtest(playerid, params[])
{
    if(GetPlayerVehicleSeat(playerid) != 0) SendClientMessage(playerid, 0xFF0000AA, "You are not a driver!");
    else if(!IsPlayerAtGasStation(playerid)) SendClientMessage(playerid, 0xFF0000AA, "You not at the repair place!");
    else{
        SetTimerEx("Repair", 5000, false, "i", playerid);
        TogglePlayerControllable(playerid, 0);
        GameTextForPlayer(playerid, TXT_PickupRepair, 3000, 4);
    }
    return 1;
}

[Rifa4life]

Код:

AddARepairPickups

Dude, you're using a function that you haven't defined, I mean, the error message is self-explanatory. Perhaps you made a mistake and wrote

Код:

 ARepairPickups(Float:x, Float:y, Float:z)

instead of

Код:

AddARepairPickups(x, y, z);

since the former one seems to do exactly what the name of the latter suggests.


As regards the warnings, add "stock", so change your function headings to the following, compile and see if still gives you the same warning.

Код:

stock ARepairPickups(Float:x, Float:y, Float:z)
stock IsPlayerAtRepair(playerid)

[DerickClark]

Thanks, but I get 1 errors

pawn Код:

(4507) : error 017: undefined symbol "AddARepairPickups"
Pawn compiler 3.2.3664          Copyright (c) 1997-2006, ITB CompuPhase


1 Error.

pawn Код:

CMD:createrepair(playerid,params[])
{
   new Float:x, Float:y, Float:z;
   GetPlayerPos(playerid, x, y, z);
   AddARepairPickups(x, y, z);
   return 1;
}

[Rifa4life]

Have you defined that function? Or is this:

Код:

ARepairPickups(Float:x, Float:y, Float:z)
{
    for (new i; i < sizeof(ARepairPickups); i++)
    {
        if (ARepairPickups[i][PickupID] == 0)
        {
            ARepairPickups[i][PickupID] = CreatePickup(27, 1, x, y, z, 0);
            ARepairPickups[i][pux] = x;
            ARepairPickups[i][puy] = y;
            ARepairPickups[i][puz] = z;
            Create3DTextLabel(TXT_PickupRepair, 0xFFFFFFFF, x, y, z + 0.8, 30.0, 0);
            CreateDynamicMapIcon(x, y, z, 56, 0, 0, 0, -1, 300.0);
            break;
        }
    }
}

Supposed to be it? Because in the code you have shown, you haven't defined AddARepairPickups as such, but instead, you defined ARepairPickups. Plus, the latter is a variable, and you are using it as a function name.

Replace this:

Код:

ARepairPickups(Float:x, Float:y, Float:z)
{
    for (new i; i < sizeof(ARepairPickups); i++)
    {
        if (ARepairPickups[i][PickupID] == 0)
        {
            ARepairPickups[i][PickupID] = CreatePickup(27, 1, x, y, z, 0);
            ARepairPickups[i][pux] = x;
            ARepairPickups[i][puy] = y;
            ARepairPickups[i][puz] = z;
            Create3DTextLabel(TXT_PickupRepair, 0xFFFFFFFF, x, y, z + 0.8, 30.0, 0);
            CreateDynamicMapIcon(x, y, z, 56, 0, 0, 0, -1, 300.0);
            break;
        }
    }
}

With this:

Код:

AddARepairPickups(Float:x, Float:y, Float:z)
{
    for (new i; i < sizeof(ARepairPickups); i++)
    {
        if (ARepairPickups[i][PickupID] == 0)
        {
            ARepairPickups[i][PickupID] = CreatePickup(27, 1, x, y, z, 0);
            ARepairPickups[i][pux] = x;
            ARepairPickups[i][puy] = y;
            ARepairPickups[i][puz] = z;
            Create3DTextLabel(TXT_PickupRepair, 0xFFFFFFFF, x, y, z + 0.8, 30.0, 0);
            CreateDynamicMapIcon(x, y, z, 56, 0, 0, 0, -1, 300.0);
            break;
        }
    }
}

[DerickClark]

it tought was a funtion

[Rifa4life]

Код:

ARepairPickups

Is used in your code as a variable, and hence should be defined at the start of the gamemode.

enum aInfo
{
PickupID,
Float: pux,
Float: puy,
Float: puz,
Text3D: TextLabel
};
new ARepairPickups[MAX_REPAIR_PICKUPS][aInfo];

You don't have to use the last variable (TextLabel), but if you do, then exchange this:

Код:

Create3DTextLabel(TXT_PickupRepair, 0xFFFFFFFF, x, y, z + 0.8, 30.0, 0);

For this:

Код:

ARepairPickups[i][TextLabel] = Create3DTextLabel(TXT_PickupRepair, 0xFFFFFFFF, x, y, z + 0.8, 30.0, 0);

It will allow you to edit or delete the 3D label later on, if you need to. Also, you obviously need to #define MAX_REPAIR_PICKUPS at the start of your script.

Problem solved?

[DerickClark]

Quote:

Originally Posted by Rifa4life Код: ARepairPickups Is used in your code as a variable, and hence should be defined at the start of the gamemode. enum aInfo { PickupID, Float: pux, Float: puy, Float: puz, Text3D: TextLabel }; new ARepairPickups[MAX_REPAIR_PICKUPS][aInfo]; You don't have to use the last variable (TextLabel), but if you do, then exchange this: Код: Create3DTextLabel(TXT_PickupRepair, 0xFFFFFFFF, x, y, z + 0.8, 30.0, 0); For this: Код: ARepairPickups[i][TextLabel] = Create3DTextLabel(TXT_PickupRepair, 0xFFFFFFFF, x, y, z + 0.8, 30.0, 0); It will allow you to edit or delete the 3D label later on, if you need to. Also, you obviously need to #define MAX_REPAIR_PICKUPS at the start of your script. Problem solved?

icon now showing

[Rifa4life]

So it creates a pickup but somewhere else?

Change:

Код:

CMD:createrepair(playerid,params[])
{
   new Float:x, Float:y, Float:z;
   GetPlayerPos(playerid, x, y, z);
   AddARepairPickups(x, y, z);
   return 1;
}

To:

Код:

CMD:createrepair(playerid,cmdtext[])
{
   #pragma unused cmdtext
   new Float:x, Float:y, Float:z;
   GetPlayerPos(playerid, x, y, z);
   AddARepairPickups(x, y, z);
   return 1;
}

Also, check this: https://sampwiki.blast.hk/wiki/Pickup_IDs because it seems that the pickup ID that you are trying to create doesn't exist so no pickup is displayed in the player's place.

[DerickClark]

Quote:

Originally Posted by Rifa4life So it creates a pickup but somewhere else? Change: Код: CMD:createrepair(playerid,params[]) { new Float:x, Float:y, Float:z; GetPlayerPos(playerid, x, y, z); AddARepairPickups(x, y, z); return 1; } To: Код: CMD:createrepair(playerid,cmdtext[]) { #pragma unused cmdtext new Float:x, Float:y, Float:z; GetPlayerPos(playerid, x, y, z); AddARepairPickups(x, y, z); return 1; } Also, check this: https://sampwiki.blast.hk/wiki/Pickup_IDs because it seems that the pickup ID that you are trying to create doesn't exist so no pickup is displayed in the player's place.

Thank you for help, glad you helped me out.

[Rifa4life]

Works?