[gekas]

PHP код:

dcmd_jail (playerid, params[])
{
    new giveid, time, str[128];
    if (!IsPlayerAdmin(playerid)) return SendClientMessage(playerid, -1, "You can't use this cmd.");
    if (sscanf(params, "ui", giveid, time)) return SendClientMessage(playerid, -1, "USAGE: /jail [ID] [TimeInMilliSeconds]");
    if (giveid == INVALID_PLAYER_ID) return SendClientMessage(playerid, -1, "Invalid player ID!");
    CagePlayer(giveid);
    format(str, sizeof(str), "You are now jailed for %i milliseconds!", time);
    SendClientMessage(giveid, -1, str);
    jailtimer = SetTimerEx("UnJailPlayer", time, false, "i", giveid);
    return 1;
} 


PHP код:

:\Documents and Settings\Arlind\Desktop\fsdf\filterscripts\Project.pwn(5980) : error 017: undefined symbol "jailtimer"
Pawn compiler 3.2.3664              Copyright (c) 1997-2006, ITB CompuPhase
1 Error. 


[Ciandlah]

Solution;

Код:

dcmd_jail (playerid, params[]) 
{ 
    new giveid, time, str[128], jailtimer; 
    if (!IsPlayerAdmin(playerid)) return SendClientMessage(playerid, -1, "You can't use this cmd."); 
    if (sscanf(params, "ui", giveid, time)) return SendClientMessage(playerid, -1, "USAGE: /jail [ID] [TimeInMilliSeconds]"); 
    if (giveid == INVALID_PLAYER_ID) return SendClientMessage(playerid, -1, "Invalid player ID!"); 
    CagePlayer(giveid); 
    format(str, sizeof(str), "You are now jailed for %i milliseconds!", time); 
    SendClientMessage(giveid, -1, str); 
    jailtimer = SetTimerEx("UnJailPlayer", time, false, "i", giveid); 
    return 1; 
}

[Vince]

Half baked solution. Even though that may compile, the variable and the value it contains will be lost as soon as the function ends.

[gekas]

PHP код:

C:\Documents and Settings\Arlind\Desktop\fsdf\filterscripts\Project.pwn(5980) : warning 204: symbol is assigned a value that is never used: "jailtimer"
Pawn compiler 3.2.3664              Copyright (c) 1997-2006, ITB CompuPhase


1 Warning. 


[Ciandlah]

Quick question, why do you even need to define the timer ?

Just simply go straight to it without defining it.

Код:

dcmd_jail (playerid, params[]) 
{ 
    new giveid, time, str[128], jailtimer; 
    if (!IsPlayerAdmin(playerid)) return SendClientMessage(playerid, -1, "You can't use this cmd."); 
    if (sscanf(params, "ui", giveid, time)) return SendClientMessage(playerid, -1, "USAGE: /jail [ID] [TimeInMilliSeconds]"); 
    if (giveid == INVALID_PLAYER_ID) return SendClientMessage(playerid, -1, "Invalid player ID!"); 
    CagePlayer(giveid); 
    format(str, sizeof(str), "You are now jailed for %i milliseconds!", time); 
    SendClientMessage(giveid, -1, str); 
    SetTimerEx("UnJailPlayer", time, false, "i", giveid); 
    return 1; 
}

Quote:

Originally Posted by Vince Half baked solution. Even though that may compile, the variable and the value it contains will be lost as soon as the function ends.

@Vince Half assed reply to a help topic

[Threshold]

His reply was more helpful than your code that you failed to proof-read.

[SyntaxQ]

Put new jailtimer; at top of your script (where you place your global variables)..

pawn Код:

// at top
new jailtimer;
///////////

dcmd_jail (playerid, params[]) 
{ 
    new giveid, time, str[128]; 
    if (!IsPlayerAdmin(playerid)) return SendClientMessage(playerid, -1, "You can't use this cmd."); 
    if (sscanf(params, "ui", giveid, time)) return SendClientMessage(playerid, -1, "USAGE: /jail [ID] [TimeInMilliSeconds]"); 
    if (giveid == INVALID_PLAYER_ID) return SendClientMessage(playerid, -1, "Invalid player ID!"); 
    CagePlayer(giveid); 
    format(str, sizeof(str), "You are now jailed for %i milliseconds!", time); 
    SendClientMessage(giveid, -1, str); 
    jailtimer = SetTimerEx("UnJailPlayer", time, false, "i", giveid); 
    return 1; 
}

forward UnJailPlayer (playerid);
public UnJailPlayer(playerid)
{
    UnCagePlayer(playerid);
    KillTimer(jailtimer);
    SendClientMessage(playerid, -1, "You are now unjailed!");
    return 1;
}

[GeekSiMo]

add define jailtimer